? ? Pending

User tests: Successful: Unsuccessful:

avatar SharkyKZ
SharkyKZ
16 May 2018

Pull Request for Issue # .

Summary of Changes

Added filter attribute to some XML fields.
Fixed 'Show Buttons' option in Editor field.
Some CS (hidden tabs and spaces).

Testing Instructions

  1. Code review.

  2. In Fields - Editor plugin set Show Buttons option to Yes. Create editor type field, set Show Buttons to Use settings from plugin. Edit an article, in Fields tab check the editor field.

Expected result

  1. Buttons are shown.

Actual result

  1. Buttons not shown.

Documentation Changes Required

No.

avatar SharkyKZ SharkyKZ - open - 16 May 2018
avatar SharkyKZ SharkyKZ - change - 16 May 2018
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 16 May 2018
Category Front End Plugins
avatar SharkyKZ SharkyKZ - change - 16 May 2018
The description was changed
avatar SharkyKZ SharkyKZ - edited - 16 May 2018
avatar Quy Quy - test_item - 16 May 2018 - Tested successfully
avatar Quy
Quy - comment - 16 May 2018

I have tested this item successfully on bb646a9


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

avatar carlitorweb carlitorweb - test_item - 16 May 2018 - Tested successfully
avatar carlitorweb
carlitorweb - comment - 16 May 2018

I have tested this item successfully on bb646a9


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

avatar carlitorweb
carlitorweb - comment - 16 May 2018

I have tested this item successfully on bb646a9


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

avatar Quy Quy - change - 16 May 2018
Status Pending Ready to Commit
avatar Quy
Quy - comment - 16 May 2018

RTC


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

avatar SharkyKZ SharkyKZ - change - 16 May 2018
Labels Added: ?
avatar SharkyKZ
SharkyKZ - comment - 16 May 2018

Restored empty values. Although it makes no difference in terms of functionality, I guess it makes sense to keep it for readability and consistency.

avatar carlitorweb
carlitorweb - comment - 16 May 2018

I prefer without, but it's more explicit by adding it to the XML.

avatar brianteeman
brianteeman - comment - 16 May 2018

Thanks

avatar mbabker mbabker - change - 24 May 2018
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2018-05-24 00:01:11
Closed_By mbabker
Labels Added: ?
avatar mbabker mbabker - close - 24 May 2018
avatar mbabker mbabker - merge - 24 May 2018

Add a Comment

Login with GitHub to post a comment