Opened 9 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 9 years ago.
ScreenSaver_modules_tab.png (23.3 KB ) - added by deejam 9 years ago.
patch.patch (2.3 KB ) - added by xray7224 9 years ago.
Changes names
namingfix.patch (1.4 KB ) - added by xray7224 9 years ago.
Correct patch (Last submit was wrong)
namingfix.2.patch (1.5 KB ) - added by xray7224 9 years ago.
New patch updated with latest comments now "General" & "Screensavers" ad the tab names

Download all attachments as: .zip

Change History (9)

by deejam, 9 years ago

Attachment: ScreenSaver_fade_tab.png added

by deejam, 9 years ago

Attachment: ScreenSaver_modules_tab.png added

by xray7224, 9 years ago

Attachment: patch.patch added

Changes names

comment:1 by xray7224, 9 years ago

Has a Patch: set

by xray7224, 9 years ago

Attachment: namingfix.patch added

Correct patch (Last submit was wrong)

comment:2 by humdinger, 9 years ago

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.

in reply to:  2 comment:3 by deejam, 9 years ago

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"?

by xray7224, 9 years ago

Attachment: namingfix.2.patch added

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

comment:4 by humdinger, 8 years ago

Resolution: fixed
Status: newclosed

Applied the patch with a few additions. Thanks xray!

Note: See TracTickets for help on using tickets.