[#73831] Sprints column scroll bar overlaps content#23119
Open
myabc wants to merge 2 commits intobug/74753-backlog-sprints-columns-full-heightfrom
Open
[#73831] Sprints column scroll bar overlaps content#23119myabc wants to merge 2 commits intobug/74753-backlog-sprints-columns-full-heightfrom
myabc wants to merge 2 commits intobug/74753-backlog-sprints-columns-full-heightfrom
Conversation
Removes `_master_backlog.sass` rules tied to UI primitives that were deleted in #22740 but were not cleaned up alongside them: `.op-backlogs-header` and its modifiers, `.op-backlogs-container`, `.op-backlogs-lists`, and the `backlogsListsContainer` container queries that targeted the old header layout.
Contributor
There was a problem hiding this comment.
Pull request overview
Adjusts Backlogs “Sprint planning” layout styling to prevent the sprints column scrollbar from overlapping content by tweaking page padding and simplifying/retargeting sprint planning container styles.
Changes:
- Extends Backlogs backlog show page content to the right side (
extended-content--right) to remove right padding constraints. - Refactors sprint header and sprint planning layout styles to focus on
.op-sprint-*selectors. - Updates the side-by-side divider styling for sprint planning columns in container queries.
myabc
force-pushed
the
bug/73831-sprints-column-scroll-overlap
branch
from
efa3345 to
85adc61
Compare
myabc
commented
May 7, 2026
Comment on lines
+36
to
+42
| // Additional overrides of overrrides to make scrolling and margins work well | ||
| // when the details (split) pane is shown. | ||
| // see frontend/src/global_styles/layout/_base.sass | ||
| #content:has(#content-bodyRight > *) | ||
| // we remove the 1rem padding on #content-body, but leave #content-header. | ||
| #content-body | ||
| padding-right: 0 |
Contributor
Author
There was a problem hiding this comment.
@HDinger @bsatarnejad this is very brittle and I'm certain it will break in the future. Let's figure out if we can refactor this.
myabc
force-pushed
the
bug/73831-sprints-column-scroll-overlap
branch
from
85adc61 to
b203f07
Compare
myabc
commented
May 7, 2026
|
|
||
| .controller-backlogs\/backlog.action-show | ||
| .controller-backlogs\/backlog.action-show, | ||
| .controller-backlogs\/backlog.action-details |
Contributor
Author
There was a problem hiding this comment.
FYI, we need to apply this to both action-details and action-show. Turbo-loaded pages do not set the correct page class.
(I do not understand why this wasn't a problem for #23118 however)
|
Caution The provided work package version does not match the core version Details:
Please make sure that:
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Note
This PR is based on #23118. Please review/merge that PR first.
Ticket
https://community.openproject.org/wp/73831
What are you trying to accomplish?
Fix padding so that scrollbars do not overlap content.
Screenshots
What approach did you choose and why?
Important
This PR also cherry-picks 9071b9b from #22936 (now merged into
dev)Merge checklist