Closed
Bug 978558
Opened 10 years ago
Closed 9 years ago
Column Size has tooltip "Click to sort by size" instead of "Sort by size" in normal message list and in message search results list
Categories
(Thunderbird :: Folder and Message Lists, defect)
Thunderbird
Folder and Message Lists
Tracking
(Not tracked)
RESOLVED
FIXED
Thunderbird 41.0
People
(Reporter: aryx, Assigned: thomas8)
Details
Attachments
(1 file, 3 obsolete files)
4.27 KB,
patch
|
thomas8
:
review+
thomas8
:
ui-review+
|
Details | Diff | Splinter Review |
Latest Earlybird and Daily on Windows 8.1 Pro 64 bit When searching for messages using menu Edit > Find > Search Messages and after adding the Size column, moving the mouse over the column header shows the tooltip "Click to sort by size" (sizeColumnTooltip from messenger.properties) instead of "Sort by size" (sizeColumn2.tooltip introduced by bug 881073). The function which does this is SetNewsFolderColumns ( http://mxr.mozilla.org/comm-aurora/source/mail/base/content/commandglue.js#69 ) and this happens despite its name also when a mail inbox is selected.
Reporter | ||
Updated•10 years ago
|
Summary: Message search: Column Size has tooltip "Click to sort by size" instead of "Sort by size" → Column Size has tooltip "Click to sort by size" instead of "Sort by size" in normal message list and in message search results list
Zach, as you fixed bug 881073, would you like to finish this one too?
Flags: needinfo?(zach.x.nickell)
Comment 2•10 years ago
|
||
(In reply to :aceman from comment #1) > Zach, as you fixed bug 881073, would you like to finish this one too? Hi Aceman, I'd be happy to work on this bug:)
Flags: needinfo?(zach.x.nickell)
So you need to update the strings sizeColumnTooltip and linesColumnTooltip in the files /mail/locales/en-US/chrome/messenger/messenger.properties /suite/locales/en-US/chrome/mailnews/messenger.properties
Comment 4•10 years ago
|
||
Attachment #8391914 -
Flags: review?(bwinton)
Comment 5•10 years ago
|
||
Comment on attachment 8391914 [details] [diff] [review] Corrects inconsistency in sizeColumnTooltip and linesColumnTooltip Review of attachment 8391914 [details] [diff] [review]: ----------------------------------------------------------------- So, I like where this has started from, but… 1) I think this is a material change in the strings, and so we'll need to give them new IDs so that the localizers notice. 2) I'm not a reviewer for the change in /suite/, so you'll need to get someone to review those parts. (Standard8 or Neil have reviewed other changes to that file.) Thanks, Blake.
Attachment #8391914 -
Flags: review?(bwinton) → review-
Comment 6•10 years ago
|
||
I figured I'd have to change the localization keys. Sorry about that, I should have asked first
Attachment #8391914 -
Attachment is obsolete: true
Attachment #8391936 -
Flags: review?(standard8)
Attachment #8391936 -
Flags: review?(bwinton)
Comment 7•10 years ago
|
||
Comment on attachment 8391936 [details] [diff] [review] Corrects inconsistency in sizeColumnTooltip and linesColumnTooltip (with new localization keys ) Review of attachment 8391936 [details] [diff] [review]: ----------------------------------------------------------------- Yep, seems good to me. Thanks!
Attachment #8391936 -
Flags: review?(bwinton) → review+
Comment 8•10 years ago
|
||
nit: prefer adding 2 instead of 1 as 1 an l are very similar in many fonts.
Updated•10 years ago
|
Attachment #8391936 -
Flags: review?(standard8) → review?(neil)
Comment 9•10 years ago
|
||
Comment on attachment 8391936 [details] [diff] [review] Corrects inconsistency in sizeColumnTooltip and linesColumnTooltip (with new localization keys ) Yes, you should use 2 for consistency with sizeColumn2.tooltip etc.
Attachment #8391936 -
Flags: review?(neil) → review+
Comment 10•10 years ago
|
||
Updated•10 years ago
|
Attachment #8392388 -
Flags: feedback?(mkmelin+mozilla)
Comment 11•10 years ago
|
||
Comment on attachment 8392388 [details] [diff] [review] Corrects inconsistency in sizeColumnTooltip and linesColumnTooltip (with new localization keys) v1.1 Review of attachment 8392388 [details] [diff] [review]: ----------------------------------------------------------------- Didn't test it but looks ok to me!
Attachment #8392388 -
Flags: feedback?(mkmelin+mozilla) → feedback+
Assignee | ||
Comment 12•10 years ago
|
||
Unfortunately nobody requested checkin-needed on this little patch, so it's already starting to bitrot... Zach, thanks for the patch. Next time, when you have all the reviews (as you do for this bug), don't forget to set the keyword checkin-needed, otherwise it won't get included into Trunk (the main code branch). Pls update the patch, at least for the two messenger.properties files the line numbers have changed.
Attachment #8391936 -
Attachment is obsolete: true
Comment 13•10 years ago
|
||
Zach, will you fix the patch per comment 12?
Assignee: nobody → zach.x.nickell
Status: NEW → ASSIGNED
Flags: needinfo?(zach.x.nickell)
Comment 14•9 years ago
|
||
(In reply to :aceman from comment #13) > Zach, will you fix the patch per comment 12? I reached out to Zach but no reply
Status: ASSIGNED → NEW
Flags: needinfo?(zach.x.nickell)
Whiteboard: [patchlove]
Updated•9 years ago
|
Assignee: zach.x.nickell → nobody
Severity: normal → minor
Assignee | ||
Updated•9 years ago
|
Assignee: nobody → bugzilla2007
OS: Windows 8.1 → All
Hardware: x86_64 → All
Assignee | ||
Comment 15•9 years ago
|
||
Status: NEW → ASSIGNED
Assignee | ||
Comment 16•9 years ago
|
||
Comment on attachment 8624492 [details] [diff] [review] unbitrotted patch - polish size and lines column tooltips r=bwinton from comment 7 r=neil from comment 9 ui-r=ThomasD
Attachment #8624492 -
Flags: ui-review+
Attachment #8624492 -
Flags: review+
Assignee | ||
Updated•9 years ago
|
Keywords: checkin-needed
Whiteboard: [patchlove]
Assignee | ||
Updated•9 years ago
|
Attachment #8392388 -
Attachment is obsolete: true
Updated•9 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 41.0
You need to log in
before you can comment on or make changes to this bug.
Description
•