Skip to content

Commit

Permalink
fix #945 Select inner divs overlap outer divs height
Browse files Browse the repository at this point in the history
  • Loading branch information
vegegoku committed Jul 26, 2024
1 parent 6f96c7b commit 79e3cac
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,7 @@
import java.util.Optional;
import java.util.function.Consumer;
import jsinterop.base.Js;
import org.dominokit.domino.ui.grid.flex.FlexAlign;
import org.dominokit.domino.ui.grid.flex.FlexItem;
import org.dominokit.domino.ui.grid.flex.FlexLayout;
import org.dominokit.domino.ui.style.Color;
Expand Down Expand Up @@ -211,14 +212,15 @@ public T setFieldStyle(FieldStyle fieldStyle) {

private void layout() {

fieldInnerContainer = FlexLayout.create();
fieldInnerContainer = FlexLayout.create().css("field-inner-cntr");
fieldInnerContainer.appendChild(
inputContainer.css("field-input-cntr").setFlexGrow(1).appendChild(inputElement));

fieldGroup.appendChild(
fieldContainer.appendChild(
fieldInnerContainer.apply(
self -> {
self.setAlignItems(FlexAlign.STRETCH);
mandatoryAddOn = createMandatoryAddOn();
if (nonNull(mandatoryAddOn)) {
self.appendChild(DominoElement.of(mandatoryAddOn).css("field-mandatory-addon"));
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -151,10 +151,15 @@
color: #F44336;
}

.field-inner-cntr {
height: 100%;
}

.field-group .field-input-cntr {
position: relative;
background-color: inherit;
width: 100%;
height:100%;
}

.field-cntr .flex-layout,
Expand All @@ -176,7 +181,7 @@
.field-group.lined.d-select .field-cntr .flex-layout {
position: relative;
bottom: 4px;
align-items: center;
align-items: stretch;
}

.table-cm-filter .field-group.lined.dui-datebox .field-cntr .flex-layout,
Expand Down Expand Up @@ -244,7 +249,7 @@
}

.field-group .select-button {
height: 35px;
height: 100%;
line-height: 35px;
}

Expand Down

0 comments on commit 79e3cac

Please sign in to comment.