Open Bug 305014 Opened 19 years ago Updated 15 years ago

Scrollbars of maximized window should be flush with screen edge

Categories

(Other Applications :: ChatZilla, enhancement)

enhancement
Not set
normal

Tracking

(Not tracked)

ASSIGNED

People

(Reporter: tuukka.tolvanen, Assigned: tuukka.tolvanen)

Details

(Keywords: polish, Whiteboard: [cz-0.9.69])

Attachments

(2 obsolete files)

Decent window manager themes [1] don't have borders on maximized windows. That
allows the application's scrollbar to be right at the screen edge, which makes
the scrollbar extremely easy to acquire. Firefox and Thunderbird both take
advantage of this. Chatzilla has a bit of empty space between the window edge
and the content scrollframe.

[1] for metacity, such themes would be e.g. quiet-purple, SphereCrystal, Simple,
    Atlanta, Esco, Glider, Industrial, keramik-Gyellow, mcblue, Metabox, Mist
This is the minimal fix. Do you want me to eliminate left hand side space, for
symmetry?
Assignee: rginda → tuukka.tolvanen
Status: NEW → ASSIGNED
Attachment #192991 - Flags: review?(silver)
Comment on attachment 192991 [details] [diff] [review]
patch1: remove right-side borders and margins

r- because it is non-symmetric. I will also point out that this 'bug' will
return when we get real draggable views, so it is not worth fixing IMHO.
Attachment #192991 - Flags: review?(silver) → review-
OS: Linux → All
Hardware: PC → All
now with both side margins/borders eliminated on the content pane -- whether
the userlist should get the same treatment is a matter of taste I guess...

> return when we get real draggable views, so it is not worth fixing IMHO.

well, there's when, and then there's now :)
Attachment #192991 - Attachment is obsolete: true
Attachment #198297 - Flags: review?(silver)
Comment on attachment 198297 [details] [diff] [review]
[checked-in] patch2: remove side borders & margins on content pane

Sorry for the delay.

r=silver

This patch will do for the moment. I think it might be good to remove the entire margin on #tabpanels-contents-box in the future.
Attachment #198297 - Flags: review?(silver) → review+
checked in.
leaving open for further work
Attachment #198297 - Attachment description: patch2: remove side borders & margins on content pane → [checked-in] patch2: remove side borders & margins on content pane
Attachment #198297 - Attachment is obsolete: true
Whiteboard: [cz-0.9.69]
Keywords: polish
QA Contact: samuel → chatzilla
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: