Feature Unit/System Tests Language Change Composer Dependency Changed NPM Resource Changed PR-6.0-dev PR-5.3-dev Pending

User tests: Successful: 0 Unsuccessful: 0

avatar LadySolveig
LadySolveig
21 Nov 2024

Pull Request for Issue # .

Summary of Changes

This PR adds the js-grid-item-is-checked class and remove onClick in the HtmlHelper for Grid as it was originally prepared in the script.

target.querySelectorAll('.js-grid-item-is-checked').forEach((element) => element.addEventListener('click', applyIsChecked));

Testing Instructions

  • install blog sample data
    BACKEND
  • go to articles list view
  • try out if you can mark single articles bei checkbox and by clicking on the row
  • try out if you can mark multiple articles
  • try out if you can select and deselect all articles
  • try if you can mark articles and use the batch processes
    FRONTEND
  1. go to Sample Layouts -> Category List
    • check if ordering works as expected
      grafik
  2. create menu item with menu item type -> List All Tags
    • check if filter works as expected
  3. create menu item with menu item type -> Featured Contacts
    • check if ordering works as expected

Install this PR
Do the same again.

Actual result BEFORE applying this Pull Request

All should work.

Expected result AFTER applying this Pull Request

All should work as before.

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: WIP

  • No documentation changes for manual.joomla.org needed

avatar LadySolveig LadySolveig - open - 21 Nov 2024
avatar LadySolveig LadySolveig - change - 21 Nov 2024
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 21 Nov 2024
Category Libraries
5b16e75 21 Nov 2024 avatar LadySolveig cs
avatar LadySolveig LadySolveig - change - 21 Nov 2024
Labels Added: PR-5.3-dev
avatar LadySolveig LadySolveig - change - 21 Nov 2024
The description was changed
avatar LadySolveig LadySolveig - edited - 21 Nov 2024
avatar Quy Quy - test_item - 21 Nov 2024 - Tested successfully
avatar Quy
Quy - comment - 21 Nov 2024

I have tested this item ✅ successfully on 5b16e75


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

avatar dgrammatiko
dgrammatiko - comment - 22 Nov 2024

FWIW I had to revert this some time ago as it broke 3rd PD code: #40192
But would be cool to have CSP strict OOTB at some point

avatar LadySolveig LadySolveig - change - 9 Feb 2025
Labels Added: Feature
avatar LadySolveig LadySolveig - change - 9 Feb 2025
The description was changed
avatar LadySolveig LadySolveig - edited - 9 Feb 2025
avatar joomla-cms-bot joomla-cms-bot - change - 9 Feb 2025
Category Libraries Unit Tests Repository SQL Administration com_admin Postgresql com_users Language & Strings Modules JavaScript NPM Change Front End com_finder com_tags External Library Composer Change Installation Libraries
avatar LadySolveig LadySolveig - change - 9 Feb 2025
Title
[5.3] Add the js-grid-item-is-checked class and remove onClick HtmlHelper Grid
[6.0] Add the js-grid-item-is-checked class and remove onClick HtmlHelper Grid
avatar LadySolveig LadySolveig - edited - 9 Feb 2025
avatar Bodge-IT
Bodge-IT - comment - 12 Feb 2025

Hi @LadySolveig, how can we help you to get this over the line?

avatar bembelimen
bembelimen - comment - 12 Feb 2025

Hi @LadySolveig, how can we help you to get this over the line?

image

I guess by fixing the conflict and make a PR against @LadySolveig repo :)

avatar LadySolveig LadySolveig - change - 12 Feb 2025
Labels Added: Unit/System Tests Language Change Composer Dependency Changed NPM Resource Changed PR-6.0-dev
avatar joomla-cms-bot joomla-cms-bot - change - 12 Feb 2025
Category Libraries Unit Tests Repository SQL Administration com_admin Postgresql com_users Language & Strings Modules JavaScript NPM Change Front End com_finder com_tags External Library Composer Change Installation SQL Administration com_admin Postgresql Language & Strings Libraries
avatar LadySolveig
LadySolveig - comment - 12 Feb 2025

Thank you @Bodge-IT for your interest 💚
After the very helpful hint from @dgrammatiko , I thought the PR would probably be better off in 6.0.
I have tested myself whether there is still a problem with sorting or filtering in the frontend. I have not been able to detect any more problems, but I have also included it in the testing instructions.
After the rebase I just waited for your upmerge to solve the remaining conflicts.

The note on the change in manual.joomla.org is still missing - I will deliver it soon and then gladly change the status to ready to review. 😊

Add a Comment

Login with GitHub to post a comment