User tests: Successful: Unsuccessful:
Inspired by discussion in #31382
Currently, the component specific dashboards of com_menus, com_content, com_users and com_privacy take their dashboard title from a string in com_cpanel.
This makes no sense and is a special threatment of core components which shouldn't happen.
Also current code requires that 3rd party dashboard are named as "com_foo" unlike core dashboards which are just named "foo". This PR will work with both variants for 3rd party.
Moving the strings to the correct component language file
Adjusting the title lookup code so it first tries to load the string from a component language file. It prepends the dashboard name with "com_" (eg "content" becomes "com_content) if not already present to guess the related component.
If no string is found it will try with a com_cpanel string.
I removed the string COM_CPANEL_DASHBOARD_CONTENT_WORKFLOW_TITLE="Content Workflow Dashboard" as I couldn't find a workflow related dashboard.
Test the various dashboards if the title is still the same.
Titles are shown, but taken from com_cpanel
In core all looks fine, titles come from the component language files
Maybe if dashboard documentation exists somewhere.
Status | New | ⇒ | Pending |
Category | ⇒ | Administration com_cpanel Language & Strings |
I have tested this item
Could the icon name be in the manifest xml file? Or in config.xml as a hidden field?
Could the icon name be in the manifest xml file? Or in config.xml as a hidden field?
Could be an idea, I'll have a look at it. Maybe even the title could be taken from there, would make things much more intuitive for sure.
I'm closing this one, as the other basically became an extended version.
Status | Pending | ⇒ | Closed |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2020-11-14 08:43:18 |
Closed_By | ⇒ | Bakual | |
Labels |
Added:
?
?
|
On a sidenote, the icon code is very strange as well, using a language string to define the icon used. However language strings are for translating purposes, not for design.
For core this was already changed to a hardcoded list. Which is better but still not good and not useable for 3rd parties.
If someone has a good idea how to solve this, I'm all ears.