Closed
Bug 743090
Opened 13 years ago
Closed 13 years ago
Allow per-comment wrapping of quoted text
Categories
(bugzilla.mozilla.org :: General, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: guijoselito, Assigned: glob)
References
Details
Attachments
(1 file)
2.08 KB,
patch
|
dkl
:
review+
|
Details | Diff | Splinter Review |
User Agent: Mozilla/5.0 (Windows NT 5.1; rv:11.0) Gecko/20100101 Firefox/11.0
Build ID: 20120312181643
Steps to reproduce:
Go to https://bugzilla.mozilla.org/show_bug.cgi?id=742438
Actual results:
Show/Hide CC Changes and the description of the bug overlap each other (I think it happens because the word pasted is very long "js::workers::WorkerParent::addChild(js::workers::Worker*)’:" and didn't go to the next line)
Expected results:
The long word should go to the next line.
this is related to upstream bug 524915 and bug 688205.
rather than wait for upstream to debate and decide, we could add a per-comment action to wrap text (next to [reply]).
i have a working patch, and from a ux perspective it works quite well.
Assignee: nobody → glob
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Summary: "Show/Hide CC Changes" and description of a bug occupies same space → Allow per-comment wrapping of quoted text
Attachment #617073 -
Flags: review?(dkl)
Comment 3•13 years ago
|
||
Comment on attachment 617073 [details] [diff] [review]
patch v1
Review of attachment 617073 [details] [diff] [review]:
-----------------------------------------------------------------
Looks good and works as expected. r=dkl
Attachment #617073 -
Flags: review?(dkl) → review+
Committing to: bzr+ssh://bjones%40mozilla.com@bzr.mozilla.org/bmo/4.0/
modified extensions/BMO/web/styles/edit_bug.css
modified skins/standard/global.css
modified template/en/default/bug/comments.html.tmpl
Committed revision 8165.
Committing to: bzr+ssh://bjones%40mozilla.com@bzr.mozilla.org/bmo/4.2/
modified extensions/BMO/web/styles/edit_bug.css
modified skins/standard/global.css
modified template/en/default/bug/comments.html.tmpl
Committed revision 8146.
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•