Closed Bug 743090 Opened 12 years ago Closed 12 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: 12 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: