Opened 3 years ago

Closed 14 months 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: Unscheduled
Component: Applications/WebPositive Version: R1/Development
Keywords: Cc:
Blocked By: Blocking:
Has a Patch: no 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 21 months ago.
snapshot of the settings window
before.png (42.1 KB ) - added by Janus 14 months ago.
after.png (43.9 KB ) - added by Janus 14 months ago.
border.png (45.3 KB ) - added by Janus 14 months ago.
noborder.png (44.6 KB ) - added by Janus 14 months ago.
experiment.png (45.3 KB ) - added by Janus 14 months ago.
web+_fonts.png (27.0 KB ) - added by humdinger 14 months ago.
another mockup
web+_fonts-2.png (27.1 KB ) - added by humdinger 14 months ago.
mockup2
after2.png (42.8 KB ) - added by -Meanwhile- 14 months ago.
mockup2413.png (42.9 KB ) - added by Janus 14 months ago.

Download all attachments as: .zip

Change History (33)

by vchrombie, 21 months ago

snapshot of the settings window

comment:1 by vchrombie, 21 months 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, 21 months 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, 19 months 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, 14 months ago

Attachment: before.png added

by Janus, 14 months ago

Attachment: after.png added

comment:4 by Janus, 14 months ago

RFC

Before

After

comment:5 by humdinger, 14 months 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, 14 months 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, 14 months ago

Attachment: border.png added

by Janus, 14 months ago

Attachment: noborder.png added

comment:7 by Janus, 14 months ago

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

comment:8 by Janus, 14 months 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 14 months ago by Janus (previous) (diff)

by Janus, 14 months ago

Attachment: experiment.png added

comment:9 by Janus, 14 months ago

Another experiment

by humdinger, 14 months ago

Attachment: web+_fonts.png added

another mockup

comment:10 by humdinger, 14 months ago

Maybe something like:

another mockup

comment:11 by Janus, 14 months 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 14 months ago by Janus (previous) (diff)

comment:12 by humdinger, 14 months ago

Or:

mockup2

Last edited 14 months ago by humdinger (previous) (diff)

by humdinger, 14 months ago

Attachment: web+_fonts-2.png added

mockup2

by -Meanwhile-, 14 months ago

Attachment: after2.png added

comment:13 by -Meanwhile-, 14 months 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 14 months ago by -Meanwhile- (previous) (diff)

comment:14 by -Meanwhile-, 14 months ago

Version 0, edited 14 months ago by -Meanwhile- (next)

by Janus, 14 months ago

Attachment: mockup2413.png added

comment:16 by vidrep, 14 months ago

I like the last one the best

comment:17 by humdinger, 14 months 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, 14 months ago

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

comment:19 by -Meanwhile-, 14 months 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 14 months ago by -Meanwhile- (previous) (diff)

in reply to:  19 comment:20 by Janus, 14 months 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-, 14 months ago

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

comment:22 by Janus, 14 months 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, 14 months ago

Resolution: fixed
Status: newclosed

Merged in hrev52405.

Note: See TracTickets for help on using tickets.