Opened 10 years ago

Closed 4 years ago

#5230 closed bug (fixed)

Position of Defaults/Revert buttons and spacing in Preferences is inconsistent

Reported by: damoklas Owned by: Janus
Priority: normal Milestone: R1
Component: Preferences Version: R1/Development
Keywords: Cc:
Blocked By: Blocking:
Has a Patch: yes Platform: All

Description

Non standart look in preferences applets GUI: the same buttons in different places; different distances from the edges.

Attachments (3)

standardpositionofbuttons.jpg (10.3 KB) - added by damoklas 10 years ago.
I think that is gut standard position for buttons, some like that.
0001-Fix-button-layout-consistency-issues-in-preflets.patch (3.0 KB) - added by KapiX 5 years ago.
0002-Fix-button-layout-consistency-issues-in-preflets.patch (3.7 KB) - added by KapiX 5 years ago.
fixed patch

Download all attachments as: .zip

Change History (17)

comment:1 Changed 10 years ago by damoklas

lack of style

comment:2 Changed 10 years ago by stippi

Owner: changed from nobody to stippi
Status: newassigned

Right you are. The new BControlLook class has methods for retrieving standard spacing and insets, it should be used in every preflet. It's probably not even much work, since most preflets are already converted to use layout management. The ticket falls in my area, I suppose, but I am not working on it just now.

Changed 10 years ago by damoklas

I think that is gut standard position for buttons, some like that.

comment:3 Changed 10 years ago by kirilla

In the cases where an Apply button is missing it's because settings are meant to be live, taking effect immediately. This has been the preferred way for the panels to work. When this live quality isn't possible, Apply and Revert are used. For some cases there are only partial meaningful defaults, or none at all, like for Time, Email and Screen, and thus not good to have a Defaults button. Full consistency here - same 3 buttons in all preflets - is not a desired thing IMO, but placement, distances, etc can definitely be improved.

comment:4 Changed 10 years ago by damoklas

to kirilla: Not that I have in mind. I mean the standard order of buttons and a standard location. It is clear that the number of buttons needed as it is required.

comment:5 Changed 5 years ago by KapiX

Has a Patch: set

I checked all preferences windows and fixed all the problems I have noticed.

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

comment:6 Changed 5 years ago by stippi

Thanks for the patch! It seems fine, except for the TouchpadPrefView.cpp change. Do you perhaps mean "B_USE_SMALL_ITEM_SPACING" instead of hard-coding "5" pixels?

comment:7 Changed 5 years ago by KapiX

I suggested myself with Mouse preflet, changed it there as well.

Changed 5 years ago by KapiX

fixed patch

comment:8 Changed 5 years ago by stippi

Thanks, patch is applied. However, I opened up all preflets, and the patch does not seem to address all inconsistencies. I can sometimes see normal spacing between the "default" and "revert" button, sometimes smaller spacing (for example the Mouse preflet). I would prefer the normal spacing throughout. Since it's still not 100% consistent, I am not closing the ticket, yet. Thanks for your work in any case!

comment:9 in reply to:  8 Changed 5 years ago by KapiX

Replying to stippi:

I can sometimes see normal spacing between the "default" and "revert" button, sometimes smaller spacing (for example the Mouse preflet). I would prefer the normal spacing throughout.

Should preflets use B_USE_SMALL_SPACING (as in Mouse preflet), B_USE_ITEM_SPACING or something different (e.g. B_USE_DEFAULT_SPACING) then? I am not sure what are the uses for each of these values.

comment:10 Changed 5 years ago by stippi

IMHO it should be B_USE_DEFAULT_SPACING everywhere, but there should be additional space forced between the buttons on the left side of the window and those on the right. Usually, adding the glue should be enough, since then there are two spacings added:

[left 1]<-default spacing->[left 2]<-default spacing->[glue]<-default spacing->[right]

comment:11 Changed 5 years ago by pulkomandy

Summary: Preferences GUI bugPosiiton of Defaults/Revert buttons and spacing in Preferences is inconsistent

comment:12 Changed 4 years ago by pulkomandy

Owner: changed from stippi to Janus
Summary: Posiiton of Defaults/Revert buttons and spacing in Preferences is inconsistentPosition of Defaults/Revert buttons and spacing in Preferences is inconsistent

comment:13 Changed 4 years ago by Janus

Should be fixed in hrev49056 and with my old commits. I don't close this now for feedbacks. Please let me know if something is off.

comment:14 Changed 4 years ago by Janus

Resolution: fixed
Status: assignedclosed

Fixed in hrev49084.

Note: See TracTickets for help on using tickets.