Skip to content

Commit

Permalink
Avoid that the bottom bar has a line break on mobile which breaks the…
Browse files Browse the repository at this point in the history
… layout as it pushes the content to the top and thus behind the app header
  • Loading branch information
HDinger authored and oliverguenther committed Sep 9, 2024
1 parent 71be3d3 commit 15cbc63
Show file tree
Hide file tree
Showing 4 changed files with 9 additions and 2 deletions.
4 changes: 4 additions & 0 deletions app/components/work_packages/split_view_component.sass
Original file line number Diff line number Diff line change
Expand Up @@ -7,3 +7,7 @@
@media screen and (max-width: $breakpoint-lg)
// Unfortunately, we have to enforce this style via !important as the resizer writes the width directly on the element as inline style
min-width: unset !important

@media screen and (max-width: $breakpoint-sm)
.work-packages--details-toolbar-container
@include hide-button-texts
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@

wpSingleContextMenu
[wpSingleContextMenu-workPackage]="workPackage">
<span class="button--text" [textContent]="text.button_more"></span>
<span class="button--text button--text_without_icon" [textContent]="text.button_more"></span>
<op-icon icon-classes="button--dropdown-indicator"></op-icon>
</button>
</div>
2 changes: 1 addition & 1 deletion frontend/src/global_styles/layout/_toolbar_mobile.sass
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@
margin: 0 0 0 10px

// Hide toolbar button texts on mobile
.button--text:not(.button--text_without_icon),
@include hide-button-texts
.icon-pulldown,
.spot-icon_dropdown
display: none
Expand Down
3 changes: 3 additions & 0 deletions frontend/src/global_styles/openproject/_mixins.sass
Original file line number Diff line number Diff line change
Expand Up @@ -274,6 +274,9 @@ $scrollbar-size: 10px
font: inherit
text-align: inherit

@mixin hide-button-texts
.button--text:not(.button--text_without_icon)
display: none

@mixin board-header-editable-toolbar-title
line-height: normal !important
Expand Down

0 comments on commit 15cbc63

Please sign in to comment.