Wrong label/accesskey is used in Options > Compose > General for entity Size

RESOLVED FIXED in Thunderbird 65.0

Status

defect
--
minor
RESOLVED FIXED
10 months ago
10 months ago

People

(Reporter: nONoNonO, Assigned: Paenglab)

Tracking

unspecified
Thunderbird 65.0

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Under Options > Compose > General the entities size.label and size.accesskey are used and they should be read from mail/chrome/messenger/preferences/compose.dtd.
They are read from mail/chrome/messenger/preferences/display.dtd however, because both files use the same entity names and both files are included in mail/components/preferences/aboutPreferences.xul:

https://dxr.mozilla.org/comm-central/source/comm/mail/components/preferences/aboutPreferences.xul#16

https://dxr.mozilla.org/comm-central/source/comm/mail/components/preferences/doctype.inc#12-15

This poses a problem for localization where there's a difference between both labels or accesskeys.
The "offending" lines are:

https://dxr.mozilla.org/comm-central/source/comm/mail/locales/en-US/chrome/messenger/preferences/display.dtd?q=file%3Adisplay.dtd&redirect_type=single#15-16
<!ENTITY size.label                       "Size:">
<!ENTITY size.accesskey                   "z">
https://dxr.mozilla.org/comm-central/source/comm/mail/locales/en-US/chrome/messenger/preferences/compose.dtd#21-22
<!ENTITY size.label                           "Size:">
<!ENTITY size.accesskey                       "z">

I suppose one pair of entities should be renamed, e.g. in compose.dtd rename them to fontSize.label and fontSize.accesskey (or htmlSize.label and htmlSize.accesskey), but if you rename them to htmlSize, it would probably make sense to also rename font.label and font.accesskey tp htmlFont.label and htmlFont.accesskey.

Either way: if you rename one of these entities, you should check with l10n if it is possible and wise to rename those entities for all locales automatically. At the moment for Dutch locale, there is a difference in font.accesskey in both files, but the wrong accesskey is shown in Options > Compose > General.
Richard, can you take care of this.
Flags: needinfo?(richard.marti)
Like this?
Assignee: nobody → richard.marti
Status: NEW → ASSIGNED
Flags: needinfo?(richard.marti)
Attachment #9021586 - Flags: review?(jorgk)
Comment on attachment 9021586 [details] [diff] [review]
1502913-prefs-label.patch

I think so, but let's ask the reporter.
Attachment #9021586 - Flags: review?(jorgk)
Attachment #9021586 - Flags: review+
Attachment #9021586 - Flags: feedback?(o.e.ekker)
Yes, that should do the trick... New entities require new translations, so this fix can only make it in next release (tb68?).
There's no secure way to do this sooner, because with this change the for all locales that have different labels/accesskeys in both files, now the specified labels/accesskeys are used and they can conflict with other accesskeys.
(For Dutch however the current accesskey is conflicting, but I guess we have to wait a bit...)
Attachment #9021586 - Flags: feedback?(o.e.ekker) → feedback+
On TB 60 there should be no problem as the prefs in tab aren't the default and not many know about it.
Keywords: checkin-needed
Ah great! I hadn't noticed that the problem only occurred with inContent prefs and I forgot I toggled that option on sometime...
Pushed by mozilla@jorgk.com:
https://hg.mozilla.org/comm-central/rev/0216225d5fba
Rename compose size.label/size.accesskey to distinguish from display in prefs. r=jorgk
Status: ASSIGNED → RESOLVED
Closed: 10 months ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 65.0
You need to log in before you can comment on or make changes to this bug.