----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125240/ -----------------------------------------------------------
(Updated Sept. 16, 2015, 11:02 p.m.) Status ------ This change has been marked as submitted. Review request for Plasma. Changes ------- Submitted with commit 7453beb9ceac0fe28561ec99778fcebc564c3af7 by David Edmundson to branch master. Repository: plasma-framework Description ------- QQC button heights are max(backgroundHeight, label + margins). QQC lineedits heights are set from the background, ignoring the margins for size. Only using it for the offset of the TextInput within the frame. We need to take the margins into consideration when deciding how high the textbox should be otherwise we're effectively just ignoring the SVG theme. This means button and textbox on the login screen are now the same height \o/. I had always assumed till now it was deliberate / the SVG hints were wrong. Diffs ----- src/declarativeimports/plasmastyle/TextFieldStyle.qml 8daaa0b8355947cba920106f9d1cae3f3adb0200 tests/components/button.qml 55d7dfbe9424736db56a1ebce5771a4592e20a98 Diff: https://git.reviewboard.kde.org/r/125240/diff/ Testing ------- Looked at some applets and stuff that used textboxes... all looked 2px higher. Shouldn't have any visible breakages as all existing code should have handled different themes with different heights anyway. Thanks, David Edmundson
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
