Skip to content

Responsive mobile nav#2417

Merged
nur-alam merged 18 commits into4.0.0-devfrom
dynamic-mobile-nav
Mar 10, 2026
Merged

Responsive mobile nav#2417
nur-alam merged 18 commits into4.0.0-devfrom
dynamic-mobile-nav

Conversation

@nur-alam
Copy link
Collaborator

@nur-alam nur-alam commented Mar 6, 2026

No description provided.

@nur-alam nur-alam added the 4.0.0 Tutor v4.w0w label Mar 6, 2026
@nur-alam nur-alam requested a review from sazedul-haque March 6, 2026 05:03
@nur-alam nur-alam changed the title Dynamic mobile nav Responsive mobile nav Mar 6, 2026
width: 220px;
width: 230px;
padding: $tutor-spacing-4;
z-index: 999999 !important;
Copy link
Collaborator

Choose a reason for hiding this comment

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

Why?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

The text ‘became an instructor’ is breaking into two lines.

width: 220px;
width: 230px;
padding: $tutor-spacing-4;
z-index: 999999 !important;
Copy link
Collaborator

Choose a reason for hiding this comment

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

Use design token $tutor-z-highest

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

#wpadminbar has a higher z-index than $tutor-z-highest

Copy link
Collaborator

Choose a reason for hiding this comment

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

We may use calc($tutor-z-highest + 1).

@nur-alam nur-alam merged commit d949783 into 4.0.0-dev Mar 10, 2026
10 of 13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4.0.0 Tutor v4.w0w

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants