Closed Bug 743090 Opened 13 years ago Closed 13 years ago

Allow per-comment wrapping of quoted text

Categories

(bugzilla.mozilla.org :: General, defect)

Production
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: guijoselito, Assigned: glob)

References

Details

Attachments

(1 file)

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
Attached patch patch v1Splinter Review
Attachment #617073 - Flags: review?(dkl)
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
See Also: → 688205
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: