adwaita-qt icon indicating copy to clipboard operation
adwaita-qt copied to clipboard

Spinboxes

Open AlexB67 opened this issue 3 years ago • 1 comments

Okay, slider height slightly overdone, but much improved, tweak as you like.

AlexB67 avatar Jan 03 '22 20:01 AlexB67

One final look EDIT: This is where the issue starts, you call

int frameWidth(pixelMetric(PM_SpinBoxFrameWidth, option, widget)); if (!flat) { size = expandSize(size, frameWidth); }

you are expanding your frame too much in height as a result, with changing the above to

int frameWidth(pixelMetric(PM_SpinBoxFrameWidth, option, widget)); if (!flat) { size = expandSize(size, frameWidth, 0); } gives the same height as per my previous change, so whatever way you cook it. You decide, but calling expandsize adds too much to the height. With the new change the result looks the same as per my previous change, but you can now call

size.setHeight(qMax(size.height(), int(Metrics::SpinBox_MinHeight)));

as before, and still get the improved result. having a careful look your other widgets are too tall also versus gtk, but negligible.

I'll check in my final changes tomorrow and leave it at that. I also did the slider ticks as per suggestion. Sorry not doing a new issue for it. Time is up.

Cheers .

AlexB67 avatar Jan 07 '22 02:01 AlexB67