Unnecessary requirement that two strings be the same

RESOLVED FIXED in mozilla1.2alpha

Status

SeaMonkey
MailNews: Message Display
RESOLVED FIXED
17 years ago
10 years ago

People

(Reporter: mkaply, Unassigned)

Tracking

Trunk
mozilla1.2alpha
x86
All

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

7.73 KB, patch
timeless
: review+
Scott MacGregor
: superreview+
Benjamin Smedberg
: approval1.8b3+
Details | Diff | Splinter Review
(Reporter)

Description

17 years ago
Search messages requires that searchButton.label in SearchDialog.dtd and 
labelForSearchButton in search.properties be exactly the same.

This puts an unnecessary burden on translation.

Updated

17 years ago
Status: NEW → ASSIGNED
Target Milestone: --- → mozilla1.2
mass re-assign.
Assignee: naving → sspitzer
Status: ASSIGNED → NEW
Product: Browser → Seamonkey

Updated

13 years ago
Assignee: sspitzer → mail
(Reporter)

Comment 2

13 years ago
Created attachment 186817 [details] [diff] [review]
Fix for problem

This patch adds code to the startup of each search dialog to initialize the
values from the properties file rather than using the DTD information.
(Reporter)

Updated

13 years ago
Attachment #186817 - Flags: review?(timeless)

Updated

13 years ago
Attachment #186817 - Flags: review?(timeless) → review+
(Reporter)

Comment 3

13 years ago
Comment on attachment 186817 [details] [diff] [review]
Fix for problem

Note this is a thunderbird and mail/news patch
Attachment #186817 - Flags: superreview?(mscott)

Updated

13 years ago
Attachment #186817 - Flags: superreview?(mscott) → superreview+
(Reporter)

Comment 4

13 years ago
Comment on attachment 186817 [details] [diff] [review]
Fix for problem

I'd like this because it affects l10n - makes things a little happier.
Attachment #186817 - Flags: approval1.8b3?

Comment 5

13 years ago
Comment on attachment 186817 [details] [diff] [review]
Fix for problem

a=bsmedberg for checkin on 6/30 only
Attachment #186817 - Flags: approval1.8b3? → approval1.8b3+
(Reporter)

Comment 6

13 years ago
fix checked in.
Status: NEW → RESOLVED
Last Resolved: 13 years ago
Resolution: --- → FIXED

Updated

10 years ago
Component: MailNews: Search → MailNews: Message Display
QA Contact: laurel → search
You need to log in before you can comment on or make changes to this bug.