? ? ? Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
18 Nov 2019

The labels for some of these fields was missing or not being used due to the layout. In addition the labels have been standardised to "Select Client" which is more in keeping with other similar labels.

To test you will need to apply the patch and then view the source code to see that all these filters have an sr-only label of Select Client

avatar brianteeman brianteeman - open - 18 Nov 2019
avatar brianteeman brianteeman - change - 18 Nov 2019
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 18 Nov 2019
Category Administration com_menus com_modules com_templates Language & Strings
avatar Quy Quy - test_item - 19 Nov 2019 - Tested successfully
avatar Quy
Quy - comment - 19 Nov 2019

I have tested this item successfully on 57562c9


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

avatar wilsonge
wilsonge - comment - 19 Nov 2019

If we're standardising everything to Select Client doesn't it make more sense to normalise them to a global language string rather than setting them per component here

avatar brianteeman
brianteeman - comment - 19 Nov 2019

It can be if you want

avatar wilsonge
wilsonge - comment - 19 Nov 2019

Should keep translators lives a bit easier than 3 dupes. It's not totally outlandish to think it might have uses in fields etc too for frontend/backend form editing etc. so yeah lets go with it

avatar brianteeman brianteeman - change - 19 Nov 2019
Labels Added: ? ?
avatar joomla-cms-bot joomla-cms-bot - change - 19 Nov 2019
Category Administration com_menus com_modules com_templates Language & Strings Administration com_languages com_menus com_modules Language & Strings
avatar brianteeman
brianteeman - comment - 19 Nov 2019

@wilsonge done

avatar Quy Quy - test_item - 19 Nov 2019 - Tested successfully
avatar Quy
Quy - comment - 19 Nov 2019

I have tested this item successfully on 5c5db7e


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

avatar infograf768 infograf768 - test_item - 23 Nov 2019 - Tested successfully
avatar infograf768
infograf768 - comment - 23 Nov 2019

I have tested this item successfully on 5c5db7e


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

avatar infograf768 infograf768 - change - 23 Nov 2019
Status Pending Ready to Commit
avatar infograf768
infograf768 - comment - 23 Nov 2019

rtc


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/27099.
avatar infograf768
infograf768 - comment - 23 Nov 2019

Apparently this branch is out-of-date but I can't update it.

avatar brianteeman brianteeman - change - 23 Nov 2019
Labels Added: ?
avatar wilsonge
wilsonge - comment - 24 Nov 2019

@brianteeman can you fix conflicts please

avatar brianteeman
brianteeman - comment - 24 Nov 2019

conflicts resolved

avatar infograf768
infograf768 - comment - 25 Nov 2019

This works but I still can't update the branch and therefore can't merge this PR.

avatar brianteeman
brianteeman - comment - 25 Nov 2019
  1. Unless there are conflicts there is no need to update a branch before you can merge it
  2. I deliberately set my branch so that it could not be updated by others
avatar infograf768
infograf768 - comment - 25 Nov 2019

Then "ordinary" maintainers are not able to merge your stuff as "Squash and Merge" is not available.
Screen Shot 2019-11-25 at 11 41 38

avatar brianteeman
brianteeman - comment - 25 Nov 2019

so be it

avatar wilsonge wilsonge - change - 25 Nov 2019
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2019-11-25 13:37:14
Closed_By wilsonge
avatar wilsonge wilsonge - close - 25 Nov 2019
avatar wilsonge wilsonge - merge - 25 Nov 2019
avatar wilsonge
wilsonge - comment - 25 Nov 2019

Thanks!

avatar brianteeman
brianteeman - comment - 25 Nov 2019

Thanks

Add a Comment

Login with GitHub to post a comment