Attempting to fix >1px height in linked combobox-button

Still not perfect
More will come once I unify button (image, text, image-text, popup, radio) paddings
pull/663/head
Khurshid Alam 2017-02-15 12:46:53 +05:30
parent ec299d4047
commit f52633e976
1 changed files with 8 additions and 6 deletions

View File

@ -43,9 +43,9 @@
} }
%button { %button {
min-height: 16px; //min-height: 16px;
min-width: 16px; //min-width: 16px;
padding: $spacing ($spacing + 2px); padding: $spacing ($spacing + 1);
border-width: 1px; border-width: 1px;
border-style: solid; border-style: solid;
border-radius: $roundness; border-radius: $roundness;
@ -525,8 +525,8 @@
button.combo { button.combo {
// otherwise the arrow placement is not symmetric // otherwise the arrow placement is not symmetric
min-width: 0; min-width: 0;
padding-left: $spacing + 2px; //padding-left: $spacing + 1px;
padding-right: $spacing + 2px; //padding-right: $spacing + 1px;
} }
arrow { arrow {
@ -537,9 +537,11 @@
box button, box entry { box button, box entry {
@extend %linked_button; @extend %linked_button;
padding: $spacing ($spacing + 2px);
} }
//hack to get rid of extra +1px height in linked combobox
//.linked:not(.vertical) button.combo { padding: 4px; }
.linked:not(.vertical) > & > box > button.combo { .linked:not(.vertical) > & > box > button.combo {
// the combo is a composite widget so the way we do button linked doesn't // the combo is a composite widget so the way we do button linked doesn't
// work, special case needed. See // work, special case needed. See