If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

improve legibility of about:buildconfig

RESOLVED FIXED in mozilla1.9.3a5

Status

()

Toolkit
Build Config
RESOLVED FIXED
8 years ago
7 years ago

People

(Reporter: Steffen Wilberg, Assigned: Steffen Wilberg)

Tracking

Trunk
mozilla1.9.3a5
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

8 years ago
Created attachment 420974 [details] [diff] [review]
patch

In about:buildconfig, it looks much better if the table headings "Compiler", "Version", "Complier flags" are left-aligned.

The ugly large fonts go away when changing the Doctype from HTML 4.01 Transitional, which triggers Quirks mode, to HTML5 (just <!DOCTYPE html>).

Oh, shouldn't we add a license to this file?
Attachment #420974 - Flags: review?(mano)
Comment on attachment 420974 [details] [diff] [review]
patch

>-<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
> #filter substitution
>+<!DOCTYPE html>

Without being a real reviewer here, can we keep the <!DOCTYPE ...> as the first line in the file, will help autodiscovery of file type by actual editors. (Anything MANO says overrides me)
Feel free to add a licence if you are changing the file for other reasons. Please get it from here:
http://www.mozilla.org/MPL/boilerplate-1.1/

Gerv
Doesn't seem like there's anything in that file worth licensing, but I defer to Gerv.
(Assignee)

Comment 4

8 years ago
Created attachment 423234 [details] [diff] [review]
patch v1.1

1. Kept the <!DOCTYPE ...> in the first line per comment 1.

2. Added a license header per comment 2. about:buildconfig was created by "hacker formerly known as seawood@netscape.com <netscape@seawood.org>" in bug 140034 in February 2003. I assume seawood was a Netscape employee at that time, so the initial developer is Netscape Communications Corporation.
Attachment #420974 - Attachment is obsolete: true
Attachment #423234 - Flags: review?(mano)
Attachment #420974 - Flags: review?(mano)
(Assignee)

Updated

8 years ago
Attachment #423234 - Flags: review?(mano) → review?(gavin.sharp)
(Assignee)

Comment 5

8 years ago
Comment on attachment 423234 [details] [diff] [review]
patch v1.1

Basically getting rid of "HTML 4.01 Transitional" plus adding 3 lines of css.
Attachment #423234 - Flags: review?(gavin.sharp) → review+
(Assignee)

Comment 6

8 years ago
http://hg.mozilla.org/mozilla-central/rev/b24fcc30c53c
Status: ASSIGNED → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9.3a5
Depends on: 600133
You need to log in before you can comment on or make changes to this bug.