Make settings dialog full screen on mobile with a nav toggle #111

Open

miro wants to merge 1 commit from claude/mobile-settings-sidebar-toggle-2f98a5 into main

pull from: claude/mobile-settings-sidebar-toggle-2f98a5

merge into: miro:main

miro:main

miro:renovate/non-major-dependencies

miro:claude/dev-env-tui

miro:claude/app-logo-cloud-icon-525f5d

miro:renovate/major-github-artifact-actions

miro:claude/login-page-oidc-407a35

Conversation 0 Commits 1 Files changed 1 +29 -5

miro commented 2026-10-15 20:10:39 +00:00

Owner

Copy link

The settings dialog's nav sidebar was hidden below md with no way to open it.

• Add a mobile-only toggle button in the dialog header that opens the nav as an overlay; selecting an item closes it.
• Make the dialog full screen on mobile; desktop layout is unchanged.

The settings dialog's nav sidebar was hidden below `md` with no way to open it. - Add a mobile-only toggle button in the dialog header that opens the nav as an overlay; selecting an item closes it. - Make the dialog full screen on mobile; desktop layout is unchanged.

miro added 1 commit 2026-10-15 20:10:39 +00:00

Make settings dialog full screen on mobile with a nav toggle f031e21539

All checks were successful

Hide all checks

CI / lint (pull_request) Successful in 32s

Details

CI / format (pull_request) Successful in 30s

Details

CI / knip (pull_request) Successful in 45s

Details

CI / typecheck (pull_request) Successful in 54s

Details

CI / docker (arm64, docker-arm64) (pull_request) Successful in 27s

Details

CI / build (pull_request) Successful in 47s

Details

CI / docker (amd64, docker-amd64) (pull_request) Successful in 20s

Details

CI / manifest (pull_request) Has been skipped

Details

CI / deploy-test (pull_request) Has been skipped

Details

E2E / playwright (pull_request) Successful in 2m19s

Details

CI / test (pull_request) Successful in 48s

Details

This pull request has changes conflicting with the target branch.

• apps/app/src/components/settings-dialog.tsx

View command line instructions

Manual merge helper

Use this merge commit message when completing the merge manually.

Merge commit title Make settings dialog full screen on mobile with a nav toggle (#111)

Merge commit body Reviewed-on: https://git.rauhala.dev/miro/files/pulls/111

Make settings dialog full screen on mobile with a nav toggle (#111) Reviewed-on: https://git.rauhala.dev/miro/files/pulls/111 Copy merge commit message

Checkout

From your project repository, check out a new branch and test the changes.

git fetch -u origin claude/mobile-settings-sidebar-toggle-2f98a5:claude/mobile-settings-sidebar-toggle-2f98a5

git switch claude/mobile-settings-sidebar-toggle-2f98a5

Merge

Merge the changes and update on Forgejo.

git switch main

git merge --no-ff claude/mobile-settings-sidebar-toggle-2f98a5

git switch claude/mobile-settings-sidebar-toggle-2f98a5

git rebase main

git switch main

git merge --ff-only claude/mobile-settings-sidebar-toggle-2f98a5

git switch claude/mobile-settings-sidebar-toggle-2f98a5

git rebase main

git switch main

git merge --no-ff claude/mobile-settings-sidebar-toggle-2f98a5

git switch main

git merge --squash claude/mobile-settings-sidebar-toggle-2f98a5

git switch main

git merge --ff-only claude/mobile-settings-sidebar-toggle-2f98a5

git switch main

git merge claude/mobile-settings-sidebar-toggle-2f98a5

git push origin main

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels

No items

No labels

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

1 participant

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

miro/files!111

No description provided.