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

[4.4] Fix the actions menu in the admin isn't working on mobile #44199

Merged
merged 1 commit into from
Oct 7, 2024

Conversation

AkameOuO
Copy link
Contributor

@AkameOuO AkameOuO commented Oct 7, 2024

Pull Request for Issue #44198 .

Summary of Changes

Fix the actions menu in the admin isn't working on mobile

Testing Instructions

Select one or more articles / users from the list. Then click in the top right corner to open the menu and click on 'Actions'.

Actual result BEFORE applying this Pull Request

The whole menu is closed.

Expected result AFTER applying this Pull Request

Open the submenu to give you a choice

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 Oct 7, 2024
@richard67
Copy link
Member

@hans2103 Could you have a look on this PR here and the corresponding issue #44198 ? It seems to me that it was introduced with your PR #43891 , or at least this PR modifies what you had introduced with your PR.

@hans2103
Copy link
Contributor

hans2103 commented Oct 7, 2024

It seems that the issue at hand is introduced with my PR for issue #43891, solving overlapping dropdowns.
My fix was closing all data-bs-toggle="collapse" when opening a data-bs-toggle="dropdown".
This fix seems to introduce an issue on small screens where the Actions dropdown is revealed after clicking the reveal button top right. Since the latter has data-bs-toggle="collapse" and the Actions has data-bs-toggle="dropdown" it will be closed when trying to open the dropdown.

Adding the check as proposed in this PR solves that issue. Good catch @AkameOuO

@hans2103
Copy link
Contributor

hans2103 commented Oct 7, 2024

I have tested this item ✅ successfully on 43c6243


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/44199.

@richard67
Copy link
Member

@hans2103 Thanks for checking and testing.

@richard67 richard67 added the bug label Oct 7, 2024
@TLWebdesign
Copy link

I have tested this item ✅ successfully on 43c6243

Tested successfully on 4.4.8


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/44199.

@richard67
Copy link
Member

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/44199.

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Oct 7, 2024
@MacJoom MacJoom added this to the Joomla! 4.4.9 milestone Oct 7, 2024
@MacJoom MacJoom merged commit 36729f6 into joomla:4.4-dev Oct 7, 2024
4 checks passed
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Oct 7, 2024
@MacJoom
Copy link
Contributor

MacJoom commented Oct 7, 2024

Thank you

richard67 added a commit that referenced this pull request Oct 7, 2024
[5.2] Upmerge 2024-10-07: Changes from PR #44199 .
richard67 added a commit that referenced this pull request Oct 8, 2024
[5.3] Upmerge 2024-10-08:
- [4.4] Update NPM developer dependencies braces, fill-range, micromatch and rollup #44159 
- [4.4] Fix the actions menu in the admin isn't working on mobile #44199 
- [5.2][GuidedTours] replace deprecated access of input property #44195 
- [5.2] Translation Update #44191 
- [5.2] SEF: Don't remove trailingslash on hp in a subdir #44205
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 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.

6 participants