Skip to content

Adapt navbar to support RTL#2012

Closed
mboukhalfa wants to merge 1 commit intogoogle:mainfrom
mboukhalfa:Adapt-navbar-to-support-RTL/mboukhalfa
Closed

Adapt navbar to support RTL#2012
mboukhalfa wants to merge 1 commit intogoogle:mainfrom
mboukhalfa:Adapt-navbar-to-support-RTL/mboukhalfa

Conversation

@mboukhalfa
Copy link
Contributor

@mboukhalfa mboukhalfa commented May 19, 2024

Replace bootstrap classes flex-column flex-md-row and ms-md-auto by w-100 and justify-content-end which have better support for RTL

Fix: #2011

TODO:

  • Improve small-device responsivity

Signed-off-by: Mohammed Boukhalfa <mohammed.boukhalfa@est.tech>
@mboukhalfa mboukhalfa force-pushed the Adapt-navbar-to-support-RTL/mboukhalfa branch from 1d85264 to f4a16d9 Compare May 19, 2024 01:28
@mboukhalfa
Copy link
Contributor Author

/hold
when switching to small device the behavior of the logo changed

@chalin chalin closed this in #2018 May 28, 2024
@chalin chalin added this to the 24Q2 milestone Jan 9, 2025
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.

The menu does not switch sides when the site changes to RTL

2 participants