Skip to content

Commit

Permalink
fix(ui5-combobox, ui5-multicombobox): center ValueState text (#2733)
Browse files Browse the repository at this point in the history
ValueState text is now vertically centered in the Popover's header.

Fixes: #2274
  • Loading branch information
ndeshev committed Jan 27, 2021
1 parent 418034e commit b2c770a
Show file tree
Hide file tree
Showing 4 changed files with 5 additions and 6 deletions.
2 changes: 1 addition & 1 deletion packages/main/src/ComboBox.js
Original file line number Diff line number Diff line change
Expand Up @@ -814,7 +814,7 @@ class ComboBox extends UI5Element {
suggestionPopoverHeader: {
"display": this._listWidth === 0 ? "none" : "inline-block",
"width": `${this._listWidth}px`,
"padding": "0.5625rem 1rem",
"padding": "0.9125rem 1rem",
},
};
}
Expand Down
3 changes: 1 addition & 2 deletions packages/main/src/MultiComboBox.js
Original file line number Diff line number Diff line change
Expand Up @@ -872,9 +872,8 @@ class MultiComboBox extends UI5Element {
return {
popoverValueStateMessage: {
"width": `${this._listWidth}px`,
"min-height": "2.5rem",
"padding": "0.5625rem 1rem",
"display": this._listWidth === 0 ? "none" : "inline-block",
"padding": "0.9125rem 1rem",
},
popoverHeader: {
"width": `${this._inputWidth}px`,
Expand Down
2 changes: 1 addition & 1 deletion packages/main/src/themes/ResponsivePopover.css
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
}

.ui5-responsive-popover-header {
height: var(--_ui5-responnsive_popover_header_height);
height: var(--_ui5-responsive_popover_header_height);
display: flex;
justify-content: space-between;
align-items: center;
Expand Down
4 changes: 2 additions & 2 deletions packages/main/src/themes/base/sizes-parameters.css
Original file line number Diff line number Diff line change
Expand Up @@ -70,7 +70,7 @@
--_ui5_textarea_padding: 0.5625rem 0.6875rem;

/* Responsive Popover */
--_ui5-responnsive_popover_header_height: 2.75rem;
--_ui5-responsive_popover_header_height: 2.75rem;

/* Side Navigation */
--ui5_side_navigation_item_height: 2.75rem;
Expand Down Expand Up @@ -210,7 +210,7 @@
--_ui5_radiobutton_min_width: var(--_ui5_radiobutton_min_width_compact);

/* Responsive Popover */
--_ui5-responnsive_popover_header_height: 2.5rem;
--_ui5-responsive_popover_header_height: 2.5rem;

/* Side Navigation */
--ui5_side_navigation_item_height: 2rem;
Expand Down

0 comments on commit b2c770a

Please sign in to comment.