Closed Bug 2006673 Opened 9 months ago Closed 7 months ago

Use getFormatedString for message window titles

Categories

(SeaMonkey :: MailNews: Message Display, task)

Tracking

(seamonkey2.53+ fixed)

RESOLVED FIXED
Future
Tracking Status
seamonkey2.53 + fixed

People

(Reporter: iannbugzilla, Assigned: iannbugzilla)

Details

(Whiteboard: SM2.53.24)

Attachments

(1 file)

At the moment we format the folder/newsgroup name and server part of the message window in a not very locale friendly way, we make assumptions what comes about what should be the start, middle and end. We should use a formatted string instead and let localisers decide the best order.

[Approval Request Comment]
Regression caused by (bug #): n/a
User impact if declined: message window titles not always localised well
Testing completed (on m-c, etc.): 2.53.24pre
Risk to taking this patch (and alternatives if risky): low
String changes made by this patch: 2 removals, 2 additions

Requesting feedback from flod that this is the correct approach.

Attachment #9533692 - Flags: review?(frgrahl)
Attachment #9533692 - Flags: feedback?(francesco.lodolo)
Attachment #9533692 - Flags: approval-comm-release?

Comment on attachment 9533692 [details] [diff] [review]
2006673-message-titles-25324.patch

Not perfect since you are hardcoding the separator and spacing for macOS, but it's also something we started doing in Firefox to simplify the code.

Overall, it makes sense.

Attachment #9533692 - Flags: feedback?(francesco.lodolo) → feedback+

Comment on attachment 9533692 [details] [diff] [review]
2006673-message-titles-25324.patch

LGTM r/a+

Attachment #9533692 - Flags: review?(frgrahl)
Attachment #9533692 - Flags: review+
Attachment #9533692 - Flags: approval-comm-release?
Attachment #9533692 - Flags: approval-comm-release+

Pushed by frgrahl@gmx.net:
https://hg.mozilla.org/comm-central/rev/110cb23cae70
Use getFormatedString for message window titles. f=flod r=frg DONTBUILD

Status: ASSIGNED → RESOLVED
Closed: 7 months ago
Resolution: --- → FIXED
Whiteboard: SM2.53.24
Target Milestone: --- → Future
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: