Closed Bug 960095 Opened 6 years ago Closed 6 years ago
Style Sheet .title entity appears 3 times in browser/devtools/styleeditor .properties
Landed in bug 950921 https://hg.mozilla.org/mozilla-central/diff/a7073df6fe23/browser/locales/en-US/chrome/browser/devtools/styleeditor.properties Also wondering about access keys conflict (a lot of C).
I don't know what happened, as the patch attached to bug 950921 doesn't add any saveStyleSheet.title entities. Maybe a landing issue? Those keys will clash.
I'm looking at the patch and see the string twice, plus one already existing, so I think the problem is not with the landing.
Forgive me, I was looking at the wrong properties file. This takes the two extra title strings out, and changes the access key to a unique one. I didn't change the string name because this hasn't made it to Aurora yet, but let me know if I should change it regardless.
Assignee: nobody → fayearthur
Attachment #8360774 - Flags: review?(dcamp)
In general you need to change the string ID if the string has already landed on mozilla-central, and therefore exposed to the localization process. Access keys don't really need that, consider that each locale has different labels, so this kind of change is OK. I'll send a message to dev-l10n to notify them of the change, just to stay on the safe side. The same string+ak is used in browser/devtools, no idea what other commands are displayed together and if there are more conflicts (haven't seen the menu yet).
Can we get traction on this bug? Should be an easy fix and we're not far away from merge day.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 29
You need to log in before you can comment on or make changes to this bug.