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

Status: RESOLVED FIXED

(MailNews Core :: Localization, defect)

Description

[Marek Wawoczny]

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

Comment 1

[Marek Wawoczny]

Attachment: Localize "All" and "Unread" label in message views

Comment 2

[Mitchell Stoltz (not reading bugmail)]

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.

Comment 3

[Daniel Veditz [:dveditz]]

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

sr=dveditz

Comment 4

[Marek Wawoczny]

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

Comment 5

[Asa Dotzler [:asa]]

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?

Comment 6

[(not reading, please use [e-mail] instead)]

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.

Comment 7

[(not reading, please use [e-mail] instead)]

re-assigning to [e-mail]

Comment 8

[(not reading, please use [e-mail] instead)]

fixed on trunk (so not part of 1.3 alpha, but will be part of 1.3 beta)

thanks for the fix, Marek.

Comment 9

[Marek Wawoczny]

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

Comment 10

[Asa Dotzler [:asa]]

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

Comment 11

[marina]

unable to verify yet... there is no localized build