User tests: Successful: Unsuccessful:
Pull Request for Issue # .
Editing the access level is split in two tabs: One tab for the name, another for the groups having access.
This means - when you are on the groups tab, you cannot be sure if you are editing the right access level.
This PR integrates access level name and groups with access in one area.
Edit access levels before and after patch. The result of editing must be the same
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
Status | New | ⇒ | Pending |
Category | ⇒ | Administration com_users |
Labels |
Added:
UI/UX
PR-6.0-dev
|
Category | Administration com_users | ⇒ | Administration com_users Language & Strings |
The system tests are failing at the "1) can open the access level form" step, so they need to be adjusted to the changes in this PR.
sorry, I have no idea what to change here https://github.com/joomla/joomla-cms/blob/6.0-dev/tests/System/integration/administrator/components/com_users/Level.cy.js
There are two failing tests.
They are failing because it is looking for
Maybe change Level Details to User Groups With Viewing Access
Labels |
Added:
Language Change
|
Category | Administration com_users Language & Strings | ⇒ | Administration com_users Language & Strings JavaScript Unit Tests |
Hope this helps. Thank you @brianteeman and @richard67
I have tested this item ✅ successfully on fe6100d
@brianteeman looks good - will change it.
I have tested this item ✅ successfully on fe6100d
As far as I understood there will be made some change to this PR to implement @brianteeman 's suggestion, so I'm waiting with RTC.
Labels |
Added:
Unit/System Tests
|
Following @brianteeman 's suggetion, the tab around the content is restored.
Could you re-test?
New appearance:
I have tested this item ✅ successfully on 24fac7a
I have tested this item ✅ successfully on 24fac7a
Status | Pending | ⇒ | Ready to Commit |
RTC
Labels |
Added:
RTC
|
Status | Ready to Commit | ⇒ | Fixed in Code Base |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2025-06-11 18:48:32 |
Closed_By | ⇒ | Bodge-IT |
Thanks @chmst, good catch and thanks to @ceford and @brianteeman for testing
Wow - I never spotted this before. Definitely an improvement