Skip to content
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.

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

Issue with navigation menu #37926 #40259

Merged
merged 11 commits into from Apr 3, 2023
Merged

Issue with navigation menu #37926 #40259

merged 11 commits into from Apr 3, 2023

Conversation

Mohammadkhan18
Copy link

Pull Request for Issue #37926
Issue with navigation menu and breakpoints
Clean installation of joomla 4.1.4 without any changes.
I noticed that on screens with resolutions between 768px (medium) and 992px (large), the navigation menu does not collapse to hamburger menu.
so i have change the bootstrap class it will automatically turn into a hamburger menu at smaller screen
related to #37926
Summary of Changes
After changing the class now the nav is responsive

Actual result BEFORE applying this Pull Request

No hamburger menu is created

Expected result AFTER applying this Pull Request
Automatically hamburger menu will created when screen size is reduced

Link to documentations

Please select:

  • Documentation link for docs.joomla.org:

  • No documentation changes for docs.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • No documentation changes for manual.joomla.org needed

@joomla-cms-bot joomla-cms-bot added NPM Resource Changed This Pull Request can't be tested by Patchtester PR-4.4-dev labels Mar 31, 2023
@laoneo
Copy link
Member

laoneo commented Mar 31, 2023

I think you forgot to commit your changes.

@Mohammadkhan18
Copy link
Author

I think you forgot to commit your changes.

Actually I haven't done any changes, just change my branch from 4.3 to 4.4 because in previous PR I have the branch 4.3 while doing pull request

@joomla-cms-bot joomla-cms-bot added the Language Change This is for Translators label Apr 1, 2023
obuisard and others added 4 commits April 2, 2023 02:44
Signed-off-by: Olivier Buisard <olivier.buisard@simplifyyourweb.com>
…sts" and checked out Guided Tours module when updating from 4.2.9 or older (#40286)
@obuisard obuisard requested a review from laoneo as a code owner April 2, 2023 16:02
@laoneo laoneo merged commit 6c135dd into 4.4-dev Apr 3, 2023
@laoneo
Copy link
Member

laoneo commented Apr 3, 2023

This here is accidentally used as upmerge pr, but I did the upmerge from 4.3 manually. Just a fyi.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Language Change This is for Translators NPM Resource Changed This Pull Request can't be tested by Patchtester PR-4.4-dev
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

9 participants