Opened 9 years ago

Closed 9 years ago

#11528 closed enhancement (fixed)

Mouse layout optimization (easy)

Reported by: humdinger Owned by: axeld
Priority: normal Milestone: R1
Component: Preferences/Mouse Version: R1/Development
Keywords: Cc:
Blocked By: #8391 Blocking:
Platform: All

Description

This is hrev48403.

My skills with the layout management are apparently too weak, but generally, this ticket can be considered "easy"...

The GUI elements in the Mouse preferences aren't pixel perfectly aligned. Here's a screenshot where I marked where the left/right distance to the next element isn't quite right:

mouse screenshot

The green areas are the differences that make the distance to the left/right bigger than on the other side of the element.

Besides, I think making the text "Double-click test area" center aligned would look nicer. And there could be a little more vertical space between the slider settings, i.e. between the "Double-click speed", "Mouse speed" and "Mouse acceleration" sections.

Attachments (5)

mouse_layout.png (19.4 KB ) - added by humdinger 9 years ago.
mouse screenshot
mouse_layout_fix.png (193.1 KB ) - added by MrPingouin 9 years ago.
mouse_layout_fix2.png (36.6 KB ) - added by MrPingouin 9 years ago.
Vertical space between sliders with B_USE_DEFAULT_SPACING
mouse_layout_fix3.png (21.2 KB ) - added by MrPingouin 9 years ago.
new layout, without slider bar labels
0001-SettingsView-Layout-update-fixes-11528.patch (4.3 KB ) - added by MrPingouin 9 years ago.
Layout update (sliders label commented out, insets more homogeneous)

Download all attachments as: .zip

Change History (17)

by humdinger, 9 years ago

Attachment: mouse_layout.png added

mouse screenshot

comment:1 by pulkomandy, 9 years ago

This could be useful to find the right spacing constants for everything: http://pulkomandy.tk/drop/haiku_spacing.png

comment:2 by MrPingouin, 9 years ago

I've started checking the source code, the part about centering "Double-click test area" is correct (SetAlignment is called with the good B_CENTER flag).

There might be an issue with BTextControl class.

comment:3 by pulkomandy, 9 years ago

Blocked By: 8391 added

comment:4 by MrPingouin, 9 years ago

I've submitted a patch for #8391 that might do the trick.

I keep working on #11528.

by MrPingouin, 9 years ago

Attachment: mouse_layout_fix.png added

comment:5 by MrPingouin, 9 years ago

You'll find in attached files "mouse_layout_fix.png".

Will it be enough ? Note : in the french version, the translation of "click to focus and raise" is huge, and it makes the application window width pretty big (at least 1.5x of the english version), not a problem though.

comment:6 by MrPingouin, 9 years ago

For a reason I ignore, the image is not viewable online. But once downloaded it's ok. Can someone fix this issue please ?

comment:7 by humdinger, 9 years ago

The vertical space between the three sliders seems a bit much. Is it possible that the middle one has 2 * B_USE_DEFAULT_SPACING to the one above/below?

You're uploaded PNG isn't displayed, bacause it's not a PNG, but a WonderBrush file. Apparently you have saved, not exported it. :) Now, if Web+ would use Translators...

Version 0, edited 9 years ago by humdinger (next)

by MrPingouin, 9 years ago

Attachment: mouse_layout_fix2.png added

Vertical space between sliders with B_USE_DEFAULT_SPACING

comment:8 by MrPingouin, 9 years ago

In fact, the original vertical spacing was set to 5 (first screenshot). Then, in my version, I tried an arbitrary value of 27, waiting for your opinion.

In those two versions, the spacing was equal, the sensation of a different spacing could be some kind of "optical illusion" (different text, etc).

In mouse_layout_fix2.png, I set the spacing to B_USE_DEFAULT_SPACING, it's like 1px more between each slider, which imho is enough to be seem less "compact" than the first one, and less "space-wasting" than the second.

What I think would help the most, is to be able to define different fonts for the BSliders widgets (font size, font weight), etc. But I haven't seen anything for it in Slider.h.

by MrPingouin, 9 years ago

Attachment: mouse_layout_fix3.png added

new layout, without slider bar labels

comment:9 by MrPingouin, 9 years ago

patch: 01

comment:10 by MrPingouin, 9 years ago

Please note that the screenshot you saw have been using the patched version of libbe.so I provided for #8391.

by MrPingouin, 9 years ago

Layout update (sliders label commented out, insets more homogeneous)

comment:11 by MrPingouin, 9 years ago

I replaced the patch, this one removes the label definition lines instead of commenting them out.

comment:12 by pulkomandy, 9 years ago

Resolution: fixed
Status: newclosed

... and I forgot to close this when I made the changes a few weeks ago.

Note: See TracTickets for help on using tickets.