Opened 2 years ago

Closed 7 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 14 months ago.
snapshot of the settings window
before.png (42.1 KB) - added by Janus 8 months ago.
after.png (43.9 KB) - added by Janus 8 months ago.
border.png (45.3 KB) - added by Janus 8 months ago.
noborder.png (44.6 KB) - added by Janus 8 months ago.
experiment.png (45.3 KB) - added by Janus 8 months ago.
web+_fonts.png (27.0 KB) - added by humdinger 8 months ago.
another mockup
web+_fonts-2.png (27.1 KB) - added by humdinger 8 months ago.
mockup2
after2.png (42.8 KB) - added by -Meanwhile- 8 months ago.
mockup2413.png (42.9 KB) - added by Janus 8 months ago.

Download all attachments as: .zip

Change History (33)

Changed 14 months ago by vchrombie

snapshot of the settings window

comment:1 Changed 14 months ago by vchrombie

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 Changed 14 months ago by pulkomandy

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 Changed 13 months ago by pulkomandy

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

Changed 8 months ago by Janus

Attachment: before.png added

Changed 8 months ago by Janus

Attachment: after.png added

comment:4 Changed 8 months ago by Janus

RFC

Before

After

comment:5 Changed 8 months ago by humdinger

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 Changed 8 months ago by Janus

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

Changed 8 months ago by Janus

Attachment: border.png added

Changed 8 months ago by Janus

Attachment: noborder.png added

comment:7 Changed 8 months ago by Janus

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

comment:8 Changed 8 months ago by Janus

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

Changed 8 months ago by Janus

Attachment: experiment.png added

comment:9 Changed 8 months ago by Janus

Another experiment

Changed 8 months ago by humdinger

Attachment: web+_fonts.png added

another mockup

comment:10 Changed 8 months ago by humdinger

Maybe something like:

another mockup

comment:11 Changed 8 months ago by Janus

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

comment:12 Changed 8 months ago by humdinger

Or:

mockup2

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

Changed 8 months ago by humdinger

Attachment: web+_fonts-2.png added

mockup2

Changed 8 months ago by -Meanwhile-

Attachment: after2.png added

comment:13 Changed 8 months ago by -Meanwhile-

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 8 months ago by -Meanwhile- (previous) (diff)

comment:14 Changed 8 months ago by -Meanwhile-

Version without boxes:

Last edited 8 months ago by -Meanwhile- (previous) (diff)

Changed 8 months ago by Janus

Attachment: mockup2413.png added

comment:15 Changed 8 months ago by Janus

comment:16 Changed 8 months ago by vidrep

I like the last one the best

comment:17 Changed 8 months ago by humdinger

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 Changed 8 months ago by vidrep

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

comment:19 Changed 8 months ago by -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'.

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 8 months ago by -Meanwhile- (previous) (diff)

comment:20 in reply to:  19 Changed 8 months ago by Janus

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 Changed 8 months ago by -Meanwhile-

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

comment:22 Changed 8 months ago by Janus

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

Pulkomandy will have the final word...

comment:23 Changed 7 months ago by waddlesplash

Resolution: fixed
Status: newclosed

Merged in hrev52405.

Note: See TracTickets for help on using tickets.