RTC NPM Resource Changed bug PR-4.4-dev Pending

User tests: Successful: Unsuccessful:

avatar AkameOuO
AkameOuO
7 Oct 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

avatar AkameOuO AkameOuO - open - 7 Oct 2024
avatar AkameOuO AkameOuO - change - 7 Oct 2024
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 7 Oct 2024
Category JavaScript Repository NPM Change
avatar richard67
richard67 - comment - 7 Oct 2024

@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.

avatar hans2103
hans2103 - comment - 7 Oct 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

avatar hans2103 hans2103 - test_item - 7 Oct 2024 - Tested successfully
avatar hans2103
hans2103 - comment - 7 Oct 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.

avatar richard67
richard67 - comment - 7 Oct 2024

@hans2103 Thanks for checking and testing.

avatar TLWebdesign TLWebdesign - test_item - 7 Oct 2024 - Tested successfully
avatar TLWebdesign
TLWebdesign - comment - 7 Oct 2024

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.

avatar richard67 richard67 - change - 7 Oct 2024
Status Pending Ready to Commit
Labels Added: NPM Resource Changed bug PR-4.4-dev
avatar richard67
richard67 - comment - 7 Oct 2024

RTC


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

avatar MacJoom MacJoom - change - 7 Oct 2024
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2024-10-07 10:34:49
Closed_By MacJoom
Labels Added: RTC
avatar MacJoom MacJoom - close - 7 Oct 2024
avatar MacJoom MacJoom - merge - 7 Oct 2024
avatar MacJoom
MacJoom - comment - 7 Oct 2024

Thank you

Add a Comment

Login with GitHub to post a comment