Opened 7 years ago

Closed 5 years ago

Last modified 4 years ago

#13165 closed bug (fixed)

[WebPositive] align menus to the right size in Settings window (easy)

Reported by: diver Owned by: pulkomandy
Priority: normal Milestone: R1/beta2
Component: Applications/WebPositive Version: R1/Development
Keywords: Cc:
Blocked By: Blocking:
Platform: All

Description

Menus in fonts tab of Settings window should align to the right side

Attachments (10)

VirtualBox_Haiku_25_03_2018_14_12_36.png (65.0 KB ) - added by vchrombie 6 years ago.
snapshot of the settings window
before.png (42.1 KB ) - added by Janus 5 years ago.
after.png (43.9 KB ) - added by Janus 5 years ago.
border.png (45.3 KB ) - added by Janus 5 years ago.
noborder.png (44.6 KB ) - added by Janus 5 years ago.
experiment.png (45.3 KB ) - added by Janus 5 years ago.
web+_fonts.png (27.0 KB ) - added by humdinger 5 years ago.
another mockup
web+_fonts-2.png (27.1 KB ) - added by humdinger 5 years ago.
mockup2
after2.png (42.8 KB ) - added by -Meanwhile- 5 years ago.
mockup2413.png (42.9 KB ) - added by Janus 5 years ago.

Download all attachments as: .zip

Change History (34)

by vchrombie, 6 years ago

snapshot of the settings window

comment:1 by vchrombie, 6 years ago

I want to work on this ticket.

I see that the Settings window has the extra space which is not the same for the Downloads window. I have to fix that. I attached the snapshot of the Settings window and Downloads window, above.

Kindly tell me if I am wrong.

comment:2 by pulkomandy, 6 years ago

Ask humdinger on the IRC channel about the spacing rules, because I'm not sure what should be done. For this ticket, just align the label of the menu fields to the right instead of to the left as they are now.

comment:3 by pulkomandy, 6 years ago

Summary: [WebPositive] align menus to the right size in Settings window[WebPositive] align menus to the right size in Settings window (easy)

by Janus, 5 years ago

Attachment: before.png added

by Janus, 5 years ago

Attachment: after.png added

comment:4 by Janus, 5 years ago

RFC

Before

After

comment:5 by humdinger, 5 years ago

Not sure I like that too much, tbh.
Could it be more like the Appearance prefs, with the size options to the rigt of the font pop-ups?

comment:6 by Janus, 5 years ago

In Appearance each font type has its size. There is only one size for the font standard/serif/sans... what is your idea?

by Janus, 5 years ago

Attachment: border.png added

by Janus, 5 years ago

Attachment: noborder.png added

comment:7 by Janus, 5 years ago

Working with grids is very time consuming here some unfinished ideas:

comment:8 by Janus, 5 years ago

I think the label "Default standard font size:" should be "Default font size:" because it applies to standard/serif/sans. In the mockup the size PopUpMenu should look smaller in both cases.

Last edited 5 years ago by Janus (previous) (diff)

by Janus, 5 years ago

Attachment: experiment.png added

comment:9 by Janus, 5 years ago

Another experiment

by humdinger, 5 years ago

Attachment: web+_fonts.png added

another mockup

comment:10 by humdinger, 5 years ago

Maybe something like:

another mockup

comment:11 by Janus, 5 years ago

I personally don't like the "legend" effect. Repeat the information with a sort of *.

And using the grid I don't think the BBox will align with the 3 labels.

Last edited 5 years ago by Janus (previous) (diff)

comment:12 by humdinger, 5 years ago

Or:

mockup2

Last edited 5 years ago by humdinger (previous) (diff)

by humdinger, 5 years ago

Attachment: web+_fonts-2.png added

mockup2

by -Meanwhile-, 5 years ago

Attachment: after2.png added

comment:13 by -Meanwhile-, 5 years ago

The original 'after.png' added boxes around the example sentences, making a comparison with 'before.png' harder and diverting from the original issue.

Last edited 5 years ago by -Meanwhile- (previous) (diff)

comment:14 by -Meanwhile-, 5 years ago

Version without boxes:

Last edited 5 years ago by -Meanwhile- (previous) (diff)

by Janus, 5 years ago

Attachment: mockup2413.png added

comment:16 by vidrep, 5 years ago

I like the last one the best

comment:17 by humdinger, 5 years ago

Yep, the last one's best. Maybe don't use bold for the labels. The Appearance fonts tab looks very nice without it. Also, how about removing the labels of the sizes or do it like Appearance and just name them "Size:"?

comment:18 by vidrep, 5 years ago

Just "Font size" works for me; simple and to the point

comment:19 by -Meanwhile-, 5 years ago

In line with appearances/fonts the labels shouldn't have to be bold.

If 'Default font size' would change into 'Size' (preferable over 'Font size'; again for consistency with appearances/fonts), then the user may get the idea that only 'Standard fonts' have different sizes to choose from. The indication 'default' is needed to avoid that idea and indicate that the 'Serif' and 'Sans-serif' fonts will follow the size selected for 'Standard font'.

For consistency within this settings window, the word 'default' is better also used for the 'Fixed font' category. So: 'Standard font' gets 'Default font size' and 'Fixed font' gets 'Default fixed font size'.

You could shorten 'Default font size' into 'Default size' but then you'd have to shorten the label for 'Fixed font' as well (again to be consistent) and that doesn't work ('Default fixed size' is confusing), so the longer versions have to be used, just like they already are in mockup2413.png.

Last edited 5 years ago by -Meanwhile- (previous) (diff)

in reply to:  19 comment:20 by Janus, 5 years ago

Replying to -Meanwhile-:

In line with appearances/fonts the labels shouldn't have to be bold.

If 'Default font size' would change into 'Size' (preferable over 'Font size'; again for consistency with appearances/fonts), then the user may get the idea that only 'Standard fonts' have different sizes to choose from. The indication 'default' is needed to avoid that idea and indicate that the 'Serif' and 'Sans-serif' fonts will follow the size selected for 'Standard font'.

The initial string was "Default standard font size" confusing as well.

comment:21 by -Meanwhile-, 5 years ago

Yes, agreed. That didn't enter the brain smoothly.

comment:22 by Janus, 5 years ago

I leave the original labels as they are, because other browsers use the same strings.

Pulkomandy will have the final word...

comment:23 by waddlesplash, 5 years ago

Resolution: fixed
Status: newclosed

Merged in hrev52405.

comment:24 by nielx, 4 years ago

Milestone: UnscheduledR1/beta2

Assign tickets with status=closed and resolution=fixed within the R1/beta2 development window to the R1/beta2 Milestone

Note: See TracTickets for help on using tickets.