Closed
Bug 610340
Opened 14 years ago
Closed 14 years ago
Truncate form validation message only if they are content specified (Port Bug 606817)
Categories
(SeaMonkey :: General, defect)
SeaMonkey
General
Tracking
(Not tracked)
RESOLVED
FIXED
seamonkey2.1b2
People
(Reporter: philip.chee, Assigned: philip.chee)
References
Details
Attachments
(1 file)
10.66 KB,
patch
|
neil
:
review+
|
Details | Diff | Splinter Review |
No description provided.
Comment 1•14 years ago
|
||
You only need to remove the cut to 256 characters in your browser.js. The truncature is now done by the content so you will received an already truncated string if it needs to be truncated.
Assignee | ||
Comment 2•14 years ago
|
||
We also need to port the changes in the tests. But thanks for the information.
Assignee | ||
Comment 3•14 years ago
|
||
ToDo:
1. Port UI and test changes from Bug 606817.
2. Port test changes from:
2.1 [Bug 600155] Let the authors override validation message with x-mozerrormessage attribute.
2.2 http://hg.mozilla.org/mozilla-central/rev/b343269c3aa4
Merging to tip. a=blocking-b7
Assignee | ||
Comment 4•14 years ago
|
||
Ports Bug 606817, plus roll-up fix for all the patches from Firefox for browser_bug561636.js
Attachment #488870 -
Flags: review?(neil)
Assignee | ||
Comment 5•14 years ago
|
||
Note: passes all tests in browser_bug561636.js using:
$ TEST_PATH=suite/browser/test/browser_bug561636.js make -C ../objdir-sm/ mochitest-browser-chrome
Updated•14 years ago
|
Attachment #488870 -
Flags: review?(neil) → review+
Assignee | ||
Comment 6•14 years ago
|
||
Pushed to comm-central
http://hg.mozilla.org/comm-central/rev/7c770ed05601
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Updated•14 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•