Closed Bug 170924 Opened 22 years ago Closed 14 years ago

Preview sound should be "listen" or play

Categories

(SeaMonkey :: Preferences, defect)

defect
Not set
trivial

Tracking

(Not tracked)

RESOLVED FIXED
seamonkey2.1b3

People

(Reporter: bugzilla, Assigned: ewong)

Details

Attachments

(1 file, 6 obsolete files)

In the prefs for mail sound we current have a button named "Preview". Preview a sound. Why not label it for what it actually does. "Listen" 20020924
'Listen' is a bad choice because it changes the object of the sentence from the sound, to the user. e.g., You don't 'Listen' (verb) a sound (object), you (object) listen *to* (verb) a sound. Generally, action should relate directly to the object, so 'Preview' (verb) the 'sound' (object) works ok from that point of view. 'Play' would be an less-ambigous alternative, though.
I like 'Play'.
How about a graphic button instead? The 'play' arrow has to be one of the most easily recognized symbols, and it would also mean one less word to translate. Windows has this for previewing system sounds.
Product: Browser → Seamonkey
does 3 votes for play morph the bug to "play"? :)
Component: MailNews: Notification → MailNews: Message Display
QA Contact: stephend → search
Henrik, are you still seeing this? Shredder/Thunderbird on OS X has had a button labeled "Play button for quite a while now?
Assignee: mscott → mail
Assignee: mail → nobody
QA Contact: search → message-display
WFM per comment 5
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → WORKSFORME
Should you move this bug to Thunderbird then since in SeaMonkey 2.1b3pre, it is still "Preview"?
I goofed :(
Severity: normal → trivial
Status: RESOLVED → REOPENED
Component: MailNews: Message Display → Preferences
QA Contact: message-display → preferences
Resolution: WORKSFORME → ---
Summary: Preview should be "listen" → Preview sound should be "listen" or play
Edmund, you seem to like simple preference bugs ;-)
Status: REOPENED → NEW
Assignee: nobody → ewong
Status: NEW → ASSIGNED
Attachment #519302 - Flags: review?(iann_bugzilla)
OS: Windows XP → All
Hardware: x86 → All
Comment on attachment 519302 [details] [diff] [review] Changed the button's lable from "Preview" to "Listen". If we are going to change it here, we should change it in the downloads pane too. Need to update the help pages too. r- due to need to review changes to new files.
Attachment #519302 - Flags: review?(iann_bugzilla) → review-
Attachment #519302 - Attachment is obsolete: true
Attachment #519655 - Flags: review?(iann_bugzilla)
Comment on attachment 519655 [details] [diff] [review] Changed the Preview Sound to "Listen To" for Mail Notification and Download Notification. Why the change to "Listen to"? It was fine with "Listen". We only have "Browse" not "Browse for", so I see no reason for "Listen to" If you're not going to do the help here, please create a bug for it instead. r=me with that explained/fixed and some information about what is happening with the help.
Attachment #519655 - Flags: review?(iann_bugzilla) → review+
Attached patch Changed 'Preview' to 'Listen' (obsolete) (deleted) — Splinter Review
Attachment #519655 - Attachment is obsolete: true
Attachment #520374 - Flags: review?
Attachment #520374 - Flags: review? → review?(iann_bugzilla)
Attachment #520374 - Flags: review?(iann_bugzilla) → review+
Attachment #520374 - Flags: ui-review?(neil)
Comment on attachment 520374 [details] [diff] [review] Changed 'Preview' to 'Listen' I don't think Listen is right. I'd be happy with Play though, with or without an icon.
Attachment #520374 - Flags: ui-review?(neil) → ui-review-
Attachment #520374 - Attachment is obsolete: true
Attachment #521492 - Flags: review?(iann_bugzilla)
Comment on attachment 521492 [details] [diff] [review] Changed the Preview Sound to "Play" for Mail Notification and Download Notification. Good apart from the fact you have forgotten the popup prefs and the help pages in this patch. r- so I can look at complete patch.
Attachment #521492 - Flags: review?(iann_bugzilla) → review-
Attachment #521492 - Attachment is obsolete: true
Attachment #522203 - Flags: review?(iann_bugzilla)
Attachment #522203 - Attachment is obsolete: true
Attachment #522205 - Flags: review?(iann_bugzilla)
Attachment #522203 - Flags: review?(iann_bugzilla)
Comment on attachment 522205 [details] [diff] [review] Changed the Preview Sound to "Play" for Popups, Mail Notification and Download Notification and their respective help. (v6) >+++ b/suite/common/pref/pref-popups.xul >+ <button id="playSound" >+ label="&playSound.label;" >+ accesskey="&playSound.accesskey;" > preference="pref.advanced.popups.disable_button.preview_sound" >+ oncommand="PlaySound();"/> >+++ b/suite/locales/en-US/chrome/common/pref/pref-popups.dtd > <!ENTITY playSound.accesskey "P"> > <!ENTITY systemSound.label "System beep"> > <!ENTITY systemSound.accesskey "S"> > <!ENTITY customSound.label "Custom sound file"> > <!ENTITY customSound.accesskey "C"> > > <!ENTITY selectSound.label "Browseâ¦"> > <!ENTITY selectSound.accesskey "o"> >+<!ENTITY playSound.label "Play"> >+<!ENTITY playSound.accesskey "l"> These clash with the label/accesskey for the checkbox, perhaps playSoundButton.label/accesskey r=me with that fixed.
Attachment #522205 - Flags: review?(iann_bugzilla) → review+
Attachment #522277 - Flags: ui-review?(neil)
Attachment #522277 - Flags: review+
Attachment #522205 - Attachment is obsolete: true
Comment on attachment 522277 [details] [diff] [review] Changed 'Preview buton' to "Play" button. (v7) [Checkin: comment 23] >+<!ENTITY playButton.label "Play"> >+<!ENTITY playButton.accesskey "l"> I guess we don't have much of a choice of access keys...
Attachment #522277 - Flags: ui-review?(neil) → ui-review+
Keywords: checkin-needed
Comment on attachment 522277 [details] [diff] [review] Changed 'Preview buton' to "Play" button. (v7) [Checkin: comment 23] http://hg.mozilla.org/comm-central/rev/d06d41cd2ccf
Attachment #522277 - Attachment description: Changed 'Preview buton' to "Play" button. (v7) → Changed 'Preview buton' to "Play" button. (v7) [Checkin: comment 23]
Status: ASSIGNED → RESOLVED
Closed: 14 years ago14 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.1b3
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: