Opened 8 years ago

Closed 8 years ago

#7327 closed enhancement (fixed)

ScreenSaver - Unclear tab titles

Reported by: deejam Owned by: axeld
Priority: normal Milestone: R1
Component: Preferences/ScreenSaver Version: R1/Development
Keywords: Cc:
Blocked By: Blocking:
Has a Patch: yes Platform: All

Description

Tabs in the ScreenSaver preflet have unclear titles. I would suggest that they are named in this manner;

  • Fade tab is renamed to Settings
  • Run module is renamed to Run screensaver

  • Modules tab is renamed to Screensavers
  • Module settings is renamed to Screensaver settings

I do not think that users understands what a module is. For them it is a screensaver.

GCC2 Hybrid hrev40735.

Attachments (5)

ScreenSaver_fade_tab.png (17.8 KB) - added by deejam 8 years ago.
ScreenSaver_modules_tab.png (23.3 KB) - added by deejam 8 years ago.
patch.patch (2.3 KB) - added by xray7224 8 years ago.
Changes names
namingfix.patch (1.4 KB) - added by xray7224 8 years ago.
Correct patch (Last submit was wrong)
namingfix.2.patch (1.5 KB) - added by xray7224 8 years ago.
New patch updated with latest comments now "General" & "Screensavers" ad the tab names

Download all attachments as: .zip

Change History (9)

Changed 8 years ago by deejam

Attachment: ScreenSaver_fade_tab.png added

Changed 8 years ago by deejam

Attachment: ScreenSaver_modules_tab.png added

Changed 8 years ago by xray7224

Attachment: patch.patch added

Changes names

comment:1 Changed 8 years ago by xray7224

Has a Patch: set

Changed 8 years ago by xray7224

Attachment: namingfix.patch added

Correct patch (Last submit was wrong)

comment:2 Changed 8 years ago by humdinger

Since these preferences are all "settings", how about naming the first tab "General" and the second "Screensavers"? I don't think "Run Screensavers" from your patch is the best description.

comment:3 in reply to:  2 Changed 8 years ago by deejam

Replying to humdinger:

Since these preferences are all "settings", how about naming the first tab "General" and the second "Screensavers"? I don't think "Run Screensavers" from your patch is the best description.

I agree. And maybe "Settings" instead of "Screensaver settings"?

Changed 8 years ago by xray7224

Attachment: namingfix.2.patch added

New patch updated with latest comments now "General" & "Screensavers" ad the tab names

comment:4 Changed 8 years ago by humdinger

Resolution: fixed
Status: newclosed

Applied the patch with a few additions. Thanks xray!

Note: See TracTickets for help on using tickets.