User tests: Successful: Unsuccessful:
Pull Request resolves #33432
Based on the previous discussion in the issue and earlier pr attempts, I have kept this fix intentionally limited in scope.
This PR updates only the three action dropdowns where Disable appears together with Delete or Uninstall. In those dropdowns, Delete or Uninstall now uses the exclamation-triangle icon so it is clearly different from Disable(X).
Also this PR does not change the global delete icon mapping.
Navigate to:
System > Update Sites > now select an item , open Actions, and check/verify the icons.
also
System > Manage > Extensions > now select an item, open Actions, and check/verify the icons.
also
Users > manage > now Select a user, open Actions, and check/verify the icons.
In these dropdowns: Disable and Delete or Uninstall used same X icons.
In these dropdowns: Disable and Delete or Uninstall are clearly differentiated ie Disable keeps the existing icon and Delete or Uninstall uses exclamation-triangle.
Please select:
Documentation link for guide.joomla.org:
No documentation changes for guide.joomla.org needed
Pull Request link for manual.joomla.org:
No documentation changes for manual.joomla.org needed
| Status | New | ⇒ | Pending |
| Category | ⇒ | Administration com_installer com_users |
I have tested this item β
successfully on 9f3b06b
I have tested this successfully! Love this PR update!
Thanks @krishnagandhicode!
| Status | Pending | ⇒ | Ready to Commit |
| Labels |
Added:
RMDQ
PR-5.4-dev
|
||
RTC
@drmenzelit or @chmst Could you please take a look at this UX change? Are the icon change and the icon used okay?
@krishnagandhicode Thank you very much for your contribution. We have discussed this and do not see this UX change as a bug fix. Could you please move this PR to the β6.2-devβ branch?
This pull request has conflicts, please resolve those before we can evaluate the pull request.
| Title |
|
||||||
| Category | Administration com_installer com_users | ⇒ | Administration com_installer com_users Language & Strings Installation Libraries |
| Labels |
Added:
Feature
RTC
Language Change
Conflicting Files
Updates Requested
PR-6.2-dev
Removed: RMDQ |
||
| Category | Administration com_installer com_users Language & Strings Installation Libraries | ⇒ | Administration com_installer com_users |
From a UX point of view, the solution has been reviewed by @drmenzelit and @chmst, and both have given π.
I have tested this item β successfully on 9f3b06b
testing instructions verified!
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/47578.