Skip to content

[v12] feat(ui-drawer-layout): rework DrawerLayout#2401

Open
ToMESSKa wants to merge 1 commit intov12from
INSTUI-4914-drawer-layout-rework
Open

[v12] feat(ui-drawer-layout): rework DrawerLayout#2401
ToMESSKa wants to merge 1 commit intov12from
INSTUI-4914-drawer-layout-rework

Conversation

@ToMESSKa
Copy link
Contributor

@ToMESSKa ToMESSKa commented Feb 11, 2026

INSTUI-4914

ISSUE:

  • DrawerLayout needs to be migrated to the new theming system DrawerLayout

TEST PLAN:

  • compare the component to the Figma design
  • check if all tokens are used in styles.ts
  • check if the component works the same as before

@ToMESSKa ToMESSKa self-assigned this Feb 11, 2026
@github-actions
Copy link

github-actions bot commented Feb 11, 2026

PR Preview Action v1.8.1

QR code for preview link

🚀 View preview at
https://instructure.design/pr-preview/pr-2401/

Built to branch gh-pages at 2026-02-11 15:03 UTC.
Preview will be ready when the GitHub Pages deployment is complete.

Copy link
Collaborator

@matyasf matyasf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code looks good, but can you please fix the examples when using the dark theme? The sample content should be also light text on dark background

@ToMESSKa ToMESSKa force-pushed the INSTUI-4914-drawer-layout-rework branch from e6c99a3 to a608844 Compare February 11, 2026 14:59
@ToMESSKa ToMESSKa changed the title feat(ui-drawer-layout): rework DrawerLayout [v12] feat(ui-drawer-layout): rework DrawerLayout Feb 11, 2026
@ToMESSKa
Copy link
Contributor Author

Code looks good, but can you please fix the examples when using the dark theme? The sample content should be also light text on dark background

Thanks, I updated the examples.

@ToMESSKa ToMESSKa requested a review from matyasf February 11, 2026 15:12
Copy link
Collaborator

@matyasf matyasf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice work

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants