Closed
Bug 134016
Opened 22 years ago
Closed 21 years ago
[PATCH] long anchor text in jprof output causes horizontal jumping
Categories
(Core :: Layout, enhancement, P3)
Tracking
()
RESOLVED
FIXED
Future
People
(Reporter: morrison, Assigned: morrison)
References
Details
Attachments
(1 file, 1 obsolete file)
2.56 KB,
patch
|
dbaron
:
review+
|
Details | Diff | Splinter Review |
Many of the symbol names in jprof output are very long. When these are used as the text for HTML anchors, it results in very long link names. As you click on one link after another, the page scrolls horizontally as well as vertically, which makes it very confusing.
Assignee | ||
Comment 1•22 years ago
|
||
A patch to be applied after the one attached to bug http://bugzilla.mozilla.org/show_bug.cgi?id=132071 which uses different text for the anchor text. It's not nearly as nice on the eye as the original output (there are preceeding underscores, e.g.), but it does produce a web page which doesn't flip back and forth horizontally and confusingly as links are followed.
Comment 2•22 years ago
|
||
Marking NEW for patch loving./
Assignee: asa → attinasi
Status: UNCONFIRMED → NEW
Component: Browser-General → Layout
Ever confirmed: true
QA Contact: doron → petersen
Summary: long anchor text in jprof output causes horizontal jumping → [PATCH] long anchor text in jprof output causes horizontal jumping
Updated•22 years ago
|
Priority: -- → P3
Updated•22 years ago
|
Target Milestone: --- → Future
Comment 3•21 years ago
|
||
David, would you be a good person to review this patch? Assuming it still applies?
Comment 4•21 years ago
|
||
Unfortunately, this linkifies the leading spaces....
Attachment #76600 -
Attachment is obsolete: true
Comment 5•21 years ago
|
||
Comment on attachment 140560 [details] [diff] [review] Merged to tip David, what do you think?
Attachment #140560 -
Flags: review?(dbaron)
Attachment #140560 -
Flags: review?(dbaron) → review+
Updated•21 years ago
|
Status: NEW → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•