bugzilla.mozilla.org has resumed normal operation. Attachments prior to 2014 will be unavailable for a few days. This is tracked in Bug 1475801.
Please report any other irregularities here.

tweak entity names so that all locales pick up the removal of ellipsis from reply & forward

RESOLVED FIXED in Thunderbird 3.0b2

Status

Thunderbird
Mail Window Front End
P3
normal
RESOLVED FIXED
10 years ago
10 years ago

People

(Reporter: davida, Assigned: philor)

Tracking

Trunk
Thunderbird 3.0b2
Bug Flags:
blocking-thunderbird3 +

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [has l10n impact])

Attachments

(1 attachment)

(Reporter)

Description

10 years ago
this was done too late for b1, so doing it in a separate bug.  The substantive change is the removal of ellipses.  Changing the IDs on the last two is primarily to make it easier to find the mapping b/w widgets and strings.

likely additions: 

 <!ENTITY hdrReplyButton.label "reply">
 <!ENTITY hdrForwardButton.label "forward">
 <!ENTITY hdrJunkButton.label "mark as junk">
 <!ENTITY hdrTrashButton.tooltiptext "delete">

Note: while doing so,remove the following after making sure they're not used by anyone else:

  <!ENTITY replyButton.label "reply…">
  <!ENTITY forwardButton.label "forward…">
  <!ENTITY junkButton.label "mark as junk">
  <!ENTITY trashButton.tooltiptext "delete">
Flags: blocking-thunderbird3+
(Reporter)

Updated

10 years ago
Assignee: nobody → david.ascher

Comment 1

10 years ago
The revised Entities clarify what they apply to. I assume these are related to what Theme Devs would use to skin the buttons, so less confusion with Toolbar entities.

Comment 2

10 years ago
In addition to removing the ellipses, I'd also suggest title casing the button captions, so the button style matches the rest of the TB UI (toolbars, options dialog, account dialog, etc.).  The buttons would then read: Reply, Forward, Mark as Junk, Delete, etc.
(Reporter)

Comment 3

10 years ago
likely won't get to this for b2.
Priority: -- → P3
Target Milestone: Thunderbird 3.0b2 → Thunderbird 3.0b3

Comment 4

10 years ago
With Windows builds the ellipses have been gone for a few days.
(Reporter)

Comment 5

10 years ago
neat!
Status: NEW → RESOLVED
Last Resolved: 10 years ago
Resolution: --- → FIXED
(Assignee)

Comment 6

10 years ago
Only if you also think it's neat to take people on long hikes and then run off into the woods leaving them there: http://mxr.mozilla.org/l10n-central/search?string=replyButton.label&find=msgHdrViewOverlay.dtd

Bug 451995 removed the ellipses while adding the archive button, but didn't rev the entity names, so now we have a few locales that thought en-US was being silly with lowercase and/or the ellipsis in the first place and never did use them, a few locales that look at everything in a patch which changes any entities, so they picked up the change despite not having an entity name change, and 35 locales who thought a hike in the woods sounded great, right up until the time when we ran off and abandoned them.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
(Reporter)

Comment 7

10 years ago
i suspect dmose wants this as part of owning the message reader
Assignee: david.ascher → dmose
(Assignee)

Comment 8

10 years ago
Stealing to do this and bug 476555 in one fell swoop.
Assignee: dmose → philringnalda
Status: REOPENED → ASSIGNED

Updated

10 years ago
Summary: remove ellipses from reply and forward in message reader → tweak entity names so that all locales pick up the removal of ellipsis from reply & forward
(Assignee)

Comment 9

10 years ago
Created attachment 360440 [details] [diff] [review]
Fix

Much as I tried, I couldn't come up with any excuse to rename trashButton.tooltiptext (which at least is a string only used in the one place), but this gets the key change that bug 451995 should have made while removing the ellipses, and in the process makes it easier to tell these from the labels for the toolbar buttons, and does the "mark as junk" -> "junk" for bug 476555.
Attachment #360440 - Flags: review?(mkmelin+mozilla)

Comment 10

10 years ago
I don't know if this is intentional or if there is a good reason for it, but the button captions are still lower case in this patch.  This is different from the rest of the button captions in TB (at least my build of it - see the main toolbar buttons, options dialog, about box, account wizard, etc.).

Comment 11

10 years ago
Comment on attachment 360440 [details] [diff] [review]
Fix

r=mkmelin, as long as we're reasonably sure we want the button labels lower case, and don't have to change it once more later. 

I don't have a strong opinion about those, but there are other places like the "other actions" dropdown and the header names where i do think we should make it to Upper case.
Attachment #360440 - Flags: review?(mkmelin+mozilla) → review+

Updated

10 years ago
Target Milestone: Thunderbird 3.0b3 → Thunderbird 3.0b2

Updated

10 years ago
Whiteboard: [has l10n impact]

Updated

10 years ago
Keywords: push-needed
Keywords: push-needed
Whiteboard: [has l10n impact] → [has l10n impact][philor to checkin]
(Assignee)

Comment 12

10 years ago
http://hg.mozilla.org/comm-central/rev/adae126ae987
Status: ASSIGNED → RESOLVED
Last Resolved: 10 years ago10 years ago
Resolution: --- → FIXED
Whiteboard: [has l10n impact][philor to checkin] → [has l10n impact]
You need to log in before you can comment on or make changes to this bug.