? ? Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
25 Apr 2021

The cancel button should have the text
Cancel when its a new item
Close when its an existing item

With this PR I have fixed the few instances that incorrectly did not follow this pattern which is used in all the other 33 views.

Summary of changes
Create a New Article - Cancel
Edit an Article - Close
Create a Workflow - Cancel
Edit a Workflow - Close
Create a Workflow Stage - Cancel
Edit a Workflow Stage - Close
Create a Workflow Transition - Cancel
Edit a Workflow Transition - Close
Edit an image - Close
Create a private message - Cancel
Read a private message - Close

avatar brianteeman brianteeman - open - 25 Apr 2021
avatar brianteeman brianteeman - change - 25 Apr 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 25 Apr 2021
Category Administration com_content com_media com_messages com_workflow
a8d305f 25 Apr 2021 avatar brianteeman CS
avatar brianteeman brianteeman - change - 25 Apr 2021
Labels Added: ?
avatar richard67
richard67 - comment - 25 Apr 2021

@brianteeman I've just tested:

2 of the things in your list don't work for me:

  • "Create a New Article - Cancel" - I still have "Close" also after having applied the PR, even after having made a new installation after that.
  • "Create a private message - Close" - I have "Cancel" in this case. I guess it's a typo in the description, because the logic would say "Cancel" for the new item.

The rest is as described.

aee79a2 25 Apr 2021 avatar brianteeman space
avatar brianteeman
brianteeman - comment - 25 Apr 2021

Article should be fixed now. Its slightly different in that component for reasons that are beyond me.

fixed typo in description

avatar brianteeman brianteeman - change - 25 Apr 2021
The description was changed
avatar brianteeman brianteeman - edited - 25 Apr 2021
avatar richard67 richard67 - test_item - 25 Apr 2021 - Tested successfully
avatar richard67
richard67 - comment - 25 Apr 2021

I have tested this item successfully on 013a5e6


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

avatar richard67 richard67 - test_item - 25 Apr 2021 - Not tested
avatar richard67
richard67 - comment - 25 Apr 2021

I have not tested this item.


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

avatar richard67
richard67 - comment - 25 Apr 2021

Please fix PHP CS as suggested above.

d8397ab 25 Apr 2021 avatar brianteeman space
avatar richard67 richard67 - test_item - 25 Apr 2021 - Tested successfully
avatar richard67
richard67 - comment - 25 Apr 2021

I have tested this item successfully on d8397ab


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

avatar sandramay0905 sandramay0905 - test_item - 26 Apr 2021 - Tested successfully
avatar sandramay0905
sandramay0905 - comment - 26 Apr 2021

I have tested this item successfully on d8397ab


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

avatar richard67 richard67 - change - 26 Apr 2021
Status Pending Ready to Commit
avatar richard67
richard67 - comment - 26 Apr 2021

RTC


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

avatar Quy Quy - close - 26 Apr 2021
avatar Quy Quy - merge - 26 Apr 2021
avatar Quy Quy - change - 26 Apr 2021
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2021-04-26 13:03:15
Closed_By Quy
Labels Added: ?
avatar Quy
Quy - comment - 26 Apr 2021

Thanks

avatar brianteeman
brianteeman - comment - 26 Apr 2021

thx

Add a Comment

Login with GitHub to post a comment