Opened 14 years ago

Closed 14 years ago

Last modified 13 years ago

#15021 closed (fixed)

Debug page code double-spaced, kinda

Reported by: Karen Tracey Owned by: nobody
Component: Core (Other) Version: dev
Severity: Keywords: blocker
Cc: Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

Since r14844, the debug page code appears double-spaced in Firefox. The actual content of lines of code appear on the line below the line with the line number. Chrome looks better, but even there the ellipsis at the end of the current line of code is placed on its own line, so there's a funny almost-blank line below the current line of code in the traceback portion of the page. I haven't tried any other browsers to see how they look.

Attachments (1)

cross_browser_debug.diff​ (1.6 KB ) - added by EmilStenstrom 14 years ago.
Update: Use pixels instead of em:s.

Download all attachments as: .zip

Change History (8)

comment:1 by Russell Keith-Magee, 14 years ago

Component: Uncategorized → Core framework
Triage Stage: Unreviewed → Accepted

Oh, I do so love CSS.

This is a change in trunk, so it's a release blocker.

comment:2 by Russell Keith-Magee, 14 years ago

Keywords: blocker added

comment:4 by EmilStenstrom, 14 years ago

Has patch: set

Added a quick patch that fixes the double spacing issue. My first patch, so let me know if I'm "doing it wrong".

Here's how it looks in (in order): IE7, IE8, IE9, Firefox 4b8, Chrome 10.0.612.3 dev, Safari 5.0.3. All on Windows: ​http://twitpic.com/3nbrh6/full

Needs more testing on more browsers, and I'm counting on your help with testing this.

(IMHO: The real solution would be to rewrite the CSS and HTML with a proper CSS reset in place. But that would not make it in time for 1.3)

by EmilStenstrom, 14 years ago

Attachment: cross_browser_debug.diff​ added

Update: Use pixels instead of em:s.

comment:5 by Ramiro Morales, 14 years ago

Triage Stage: Accepted → Ready for checkin

Verified that the latest patch also corrects the issue on Chrome 6.x, Firefox 3.5.x and KDE's Konqueror 4.5.x all on Linux.

comment:6 by Russell Keith-Magee, 14 years ago

Resolution: → fixed
Status: new → closed

(In [15294]) Fixed #15021 -- Corrected CSS for the layout of stack traces on the server error page. Thanks to EmilStenstrom for the patch.

comment:7 by Jacob, 13 years ago

milestone: 1.3

Milestone 1.3 deleted

Note: See TracTickets for help on using tickets.
Back to Top