Skip to content

fix(composer): centre the column vertically #205

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
PouyanJay merged 1 commit into from
Aug 1, 2026
Merged

fix(composer): centre the column vertically #205

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
11 changes: 10 additions & 1 deletion apps/web/src/components/configurator/IdleCourseSetup.module.css
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,8 +8,17 @@
display: flex;
flex-direction: column;
align-items: center;
/* `safe` is doing real work here, not decoration. A plain `center` in a scroll container clips
the top of anything taller than the viewport and puts it out of reach: with a long build
history the heading would be unscrollable. `safe center` centres while the content fits and
falls back to start alignment the moment it does not. A browser without `safe` drops the whole
declaration and lands on start alignment, which is the old behaviour rather than a broken one,
so there is deliberately no plain `center` fallback above this. */
justify-content: safe center;
gap: var(--space-10);
padding: var(--space-10) var(--space-5) var(--space-12);
/* Symmetric, so "centred" reads as centred. These are the minimum edges once the content is tall
enough to scroll. */
padding: var(--space-10) var(--space-5);
}

/* Every direct child takes the measure, so nothing has to remember to set its own. */
Expand Down
Loading