296301 - "Report Broken Website" tool has broken layout on high-DPI displays

Status: RESOLVED WONTFIX

(Other Applications Graveyard :: Reporter, defect)

Description

[Felix Miata]

The Deer Park Alpha 1 privacy policy statement pane is not big enough to see
anything in (Linux), or too short to easily read (OS/2), and so totally useless
unless the window is resized (possible on Linux, impossible on OS/2). It needs
to be considerably bigger by default.

Comment 1

[Felix Miata]

Attachment: Linux screenshot (120 DPI)

Nothing showing to read.

Comment 2

[Felix Miata]

Attachment: OS/2 screenshot (120 DPI)

Three lines showing is not sufficient.

Comment 3

[Vladimir Vukicevic [:vlad] [:vladv] (needinfo me, slow to respond)]

The scrolling text box just needs a minimum height set.

Comment 4

[Vladimir Vukicevic [:vlad] [:vladv] (needinfo me, slow to respond)]

Duping bug 296304 to this; there are a lot of bad offenders like this throughout
the code (e.g. the new preferences dialog by default has a chunk cut off at
higher dpi), but the report broken website tool has a lot of them.

Comment 5

[Vladimir Vukicevic [:vlad] [:vladv] (needinfo me, slow to respond)]

*** Bug 296304 has been marked as a duplicate of this bug. ***

Comment 6

[Felix Miata]

Note bug 133627. DPI dependence is apparently a broken way to design
non-resizable windows.

The window/dialog problem here is little different than that in bug 238137,
except that here it is the main content that is given an inexcusably miniscule
amount of screen space, exactly like the itty bitty M$ OS dialogs. If the
primary purpose of a window is to show an important document, that document
should be allotted more than 2% of the display area, and certainly the major
portion of the window in which displayed.

Comment 7

[Asa Dotzler [:asa]]

Robert, can you dig into this for us? 

Comment 8

[Robert Accettura [:raccettura]]

I tried 125DPI on my mac with the recent nightly and can't reproduce.

Comment 9

[Felix Miata]

Attachment: DPa2 OS/2 120 DPI screenshot of "Fill out Report" form

DPa2: The privacy policy now opens in a regular browser window, but the form
window is still too small, its input area is much too small, and the inputs use
menu text size instead of browser default text size.

Comment 10

[Felix Miata]

Attachment: DPa2 Linux "new" profile privacy policy 120 DPI screenshot

The privacy policy opens in normal browser window when clicking the link in the
report form, but in this window if the profile has not previously been used to
open reporter.

Comment 11

[Felix Miata]

Attachment: DPa2 OS/2 "new" profile privacy policy 120 DPI screenshot

Comment 12

[Felix Miata]

Attachment: DPa2 Linux report form 120 DPI screenshot

The textarea and input text is menu font size, which is too small, not as it
should be, the browser default size. On OS/2, the cursor is missing from the
textarea when trying to input there.

Comment 13

[Robert Accettura [:raccettura]]

Attachment: 120 DPI W32

This is what I see at 120 DPI

Comment 14

[Robert Accettura [:raccettura]]

Attachment: 120 DPI W32 #2

Another Screenshot

Comment 15

[Felix Miata]

Attachment: Linux 168 DPI SM privacy policy screenshot

Comment 16

[Felix Miata]

Attachment: Linux 168 DPI SM report form screenshot

Comment 17

[Robert Accettura [:raccettura]]

Gavin: want this one?

Comment 18

[Felix Miata]

Attachment: W32 144 DPI SM privacy policy screenshot

Comment 19

[Felix Miata]

Attachment: W32 144 DPI SM report form screenshot

This is the same problem as bug 296695 and bug 293427 among several others older. Note that all attachments to this bug are within 6px in width of each other even though 3 platforms are represented. That's because the size of these dialogs is set without regard to DPI. When DPI goes up, font size goes up. When font size goes up, space required goes up regardless whether scrolling is provided. Heights and widths of these dialogs and their components need to be set relative to text size, which means in ems. High DPI is not an isolated condition, and apparently is destined to become a lot more common before too long if this article is any indication: http://chronicle.com/free/2005/11/2005111602t.htm

Comment 20

[Sylvestre Ledru [:Sylvestre]]

Reporter isn't a maintained project. Closing!