? ? Pending

User tests: Successful: Unsuccessful:

avatar chmst
chmst
28 Apr 2020

Pull Request for Issue #28851 .

Summary of Changes

Change Button text so that it matches the link to template styles.

Testing Instructions

Code inspect.

Expected result

Actual result

Documentation Changes Required

avatar chmst chmst - open - 28 Apr 2020
avatar chmst chmst - change - 28 Apr 2020
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 28 Apr 2020
Category Administration Language & Strings
avatar infograf768
infograf768 - comment - 28 Apr 2020

We could avoid having multiple strings with the same value by using here a JGLOBAL string.

For example it would also be used here:
#28844 (comment)

There may be other quickicons strings concerned by such a modification.

avatar chmst chmst - change - 28 Apr 2020
Title
Quickicon text for Template styles
[4.0] Quickicon text for Template styles
avatar chmst chmst - change - 28 Apr 2020
Labels Added: ?
Easy No Yes
avatar chmst chmst - edited - 28 Apr 2020
avatar ChristineWk
ChristineWk - comment - 28 Apr 2020

I have tested this item successfully on d21b2a3


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

avatar ChristineWk ChristineWk - test_item - 28 Apr 2020 - Tested successfully
avatar adj9
adj9 - comment - 28 Apr 2020

I have not tested this item.

@Chraneco
Sorry but I didn't understand which text you want to change. Whether it is on the frontend or on the backend.

In the setting of the template (I only have CASSIOPEA) in Advanced I only have title and description.


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

avatar adj9 adj9 - test_item - 28 Apr 2020 - Not tested
avatar chmst
chmst - comment - 28 Apr 2020

@adj9 it is the backend dashboard with quick ions. See it on the dashboard.
The icon templates goes to view template styles (frontend), not to templatesoverview.

The aim of quickicons is to make it easier builing a site. In general, web designers want to do overrides or change params in a template style. (for the frontend).

So the text on the icon "Templates" is confusing, as @coolcat-creations said, as the icon goes to the view "template styles".
This PR makes this clear.

avatar jwaisner
jwaisner - comment - 28 Apr 2020

I have tested this item successfully on d21b2a3

Reviewed language string change and looks good. Also tested PR and makes proper changes with no issues.


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

avatar jwaisner jwaisner - test_item - 28 Apr 2020 - Tested successfully
avatar jwaisner jwaisner - change - 28 Apr 2020
Status Pending Ready to Commit
avatar jwaisner
jwaisner - comment - 28 Apr 2020

RTC


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

avatar infograf768
infograf768 - comment - 29 Apr 2020

Concerning #28854 (comment)
I got no reply...

When things get a bit more stable, I therefore may have to make a specific PR.

avatar jwaisner
jwaisner - comment - 29 Apr 2020

@chmst Concerning @infograf768 comment. Is that something you may be able to work with him on to minimize the need for multiple PRs?

avatar wilsonge wilsonge - close - 2 May 2020
avatar wilsonge wilsonge - merge - 2 May 2020
avatar wilsonge wilsonge - change - 2 May 2020
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2020-05-02 10:45:07
Closed_By wilsonge
Labels Added: ?
avatar wilsonge
wilsonge - comment - 2 May 2020

@infograf768 separate PR sounds good to me - seems logical.

Thanks @chmst !

Add a Comment

Login with GitHub to post a comment