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)

defect
Not set
minor

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 41.0

People

(Reporter: aryx, Assigned: thomas8)

Details

Attachments

(1 file, 3 obsolete files)

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.
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)
(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 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-
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 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+
nit: prefer adding 2 instead of 1 as 1 an l are very similar in many fonts.
Attachment #8391936 - Flags: review?(standard8) → review?(neil)
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+
Attachment #8392388 - Flags: feedback?(mkmelin+mozilla)
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+
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
Zach, will you fix the patch per comment 12?
Assignee: nobody → zach.x.nickell
Status: NEW → ASSIGNED
Flags: needinfo?(zach.x.nickell)
(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]
Assignee: zach.x.nickell → nobody
Severity: normal → minor
Assignee: nobody → bugzilla2007
OS: Windows 8.1 → All
Hardware: x86_64 → All
Status: NEW → ASSIGNED
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+
Keywords: checkin-needed
Whiteboard: [patchlove]
Attachment #8392388 - Attachment is obsolete: true
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.

Attachment

General

Creator:
Created:
Updated:
Size: