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

[damoklas]

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

[damoklas]

lack of style

[stippi]

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.

[damoklas]

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

[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.

[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.

[KapiX]

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

[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?

[KapiX]

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

[KapiX]

fixed patch

[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!

[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.

[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]

[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.

[Janus]

Fixed in hrev49084.