Need to localize "All" and "Unread" label in message views

RESOLVED FIXED

Status

RESOLVED FIXED
16 years ago
10 years ago

People

(Reporter: mwawoczny, Assigned: mwawoczny)

Tracking

Bug Flags:
blocking1.3a -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

16 years ago
Need to localize "All" and "Unread" label in message views - attaching patch,
this should be fixed for 1.3 Alpha because it is highly visible option/menu
(Assignee)

Comment 1

16 years ago
Created attachment 108586 [details] [diff] [review]
Localize "All" and "Unread" label in message views
(Assignee)

Updated

16 years ago
Attachment #108586 - Flags: superreview?(sspitzer)
Attachment #108586 - Flags: review?(mstoltz)
Attachment #108586 - Flags: approval1.3a?
Flags: blocking1.3a?

Updated

16 years ago
Flags: blocking1.3a? → blocking1.3a-
Comment on attachment 108586 [details] [diff] [review]
Localize "All" and "Unread" label in message views

I have no idea why this was given to me to review, but it looks OK, so
r=mstoltz.
Attachment #108586 - Flags: review?(mstoltz) → review+
Comment on attachment 108586 [details] [diff] [review]
Localize "All" and "Unread" label in message views

sr=dveditz
Attachment #108586 - Flags: superreview?(sspitzer) → superreview+
(Assignee)

Comment 4

16 years ago
Please apply this patch to trunk tree (I don't have CVS write account) - thnx
again :)

Comment 5

16 years ago
Comment on attachment 108586 [details] [diff] [review]
Localize "All" and "Unread" label in message views

a=asa for checkin to 1.3a. Who can land this for us?
Attachment #108586 - Flags: approval1.3a? → approval1.3a+
Comment on attachment 108586 [details] [diff] [review]
Localize "All" and "Unread" label in message views

sr=sspitzer

thanks for the fix.  I'll land it.
re-assigning to Marek.Wawoczny@GmbH.rubikon.pl
Assignee: rchen → Marek.Wawoczny
fixed on trunk (so not part of 1.3 alpha, but will be part of 1.3 beta)

thanks for the fix, Marek.
Status: NEW → RESOLVED
Last Resolved: 16 years ago
Resolution: --- → FIXED
(Assignee)

Comment 9

16 years ago
Thnx :) Btw. My patch has "approval1.3a+" flag, so this patch can also land in
1.3a branch, yes?

Comment 10

16 years ago
this didn't land in time for 1.3a. There was no branch. 

Comment 11

16 years ago
unable to verify yet... there is no localized build
Product: MailNews → Core
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.