? Success

User tests: Successful: Unsuccessful:

avatar nonumber
nonumber
4 Mar 2016

This creates the ability for an editor-xtd plugin to return an array of buttons, instead of just one button object.

Testing

To test, replace this code in
/plugins/editors-xtd/readmore/readmore.php

return $button;

with:

        $button2 = new JObject;
        $button2->modal = false;
        $button2->class = 'btn';
        $button2->onclick = 'alert("Hurray!");return false;';
        $button2->text = 'An extra button!';
        $button2->name = 'smiley-2';
        $button2->link = '#';

        return array($button, $button2);

Result after patch (using core TinyMCE editor):

Request to get this into J3.5

The reason I would like this to make it into J3.5 is because of the changes made to the way the new TinyMCE handles the editor buttons.
This makes it impossible to 'manipulate' the html output of the buttons and make an editor button plugin create multiple buttons.

avatar nonumber nonumber - open - 4 Mar 2016
avatar nonumber nonumber - change - 4 Mar 2016
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 4 Mar 2016
Labels Added: ?
avatar nonumber
nonumber - comment - 4 Mar 2016

@dgt41 Would like you to look at this too

avatar infograf768
infograf768 - comment - 4 Mar 2016

Is this a new feature?

avatar nonumber
nonumber - comment - 4 Mar 2016

I updated the description: Request to get this into J3.5

avatar nonumber
nonumber - comment - 4 Mar 2016

Yes, it is a new 'feature'. Or more an added ability.
If this does not make it into Joomla 3.5 I will need to drop support for the TinyMCE editor in J3.5 for one of my extensions.

avatar nonumber
nonumber - comment - 4 Mar 2016

PS: There is no change in behaviour for any existing editor plugin extensions. They will all just keep working.

avatar shubhamnba2009
shubhamnba2009 - comment - 4 Mar 2016

I have tried to test this. New button with smiley face is coming but readmore button functionality is not working properly.
When i clicked readmore button between two paragraph in the article, on the front end it is just introducing new line between the two paragraph. Either i am using readmore button wrong way or there is some problem.
Reply me what is it!

avatar joomdonation joomdonation - test_item - 4 Mar 2016 - Tested successfully
avatar joomdonation
joomdonation - comment - 4 Mar 2016

I have tested this item :white_check_mark: successfully on b50885b

Tested using two editor-xtd plugins from my custom extension. After patch, the two plugins still work as before. Plus, with this change, I can now merge two plugins into one (which is generating two buttons)


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

avatar nonumber
nonumber - comment - 4 Mar 2016

@shubhamnba2009 That's how the readmore button works.
You can test how it works before the patch. Should do the same.

avatar shubhamnba2009 shubhamnba2009 - test_item - 4 Mar 2016 - Tested successfully
avatar shubhamnba2009
shubhamnba2009 - comment - 4 Mar 2016

I have tested this item :white_check_mark: successfully on b50885b

The new button came on TinyMCE editor as expected.


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

avatar dgt41 dgt41 - change - 4 Mar 2016
Status Pending Ready to Commit
avatar dgt41
dgt41 - comment - 4 Mar 2016

RTC

avatar joomla-cms-bot joomla-cms-bot - change - 4 Mar 2016
Labels Added: ?
avatar wilsonge wilsonge - change - 5 Mar 2016
Status Ready to Commit Closed
Closed_Date 0000-00-00 00:00:00 2016-03-05 19:27:08
Closed_By wilsonge
avatar wilsonge wilsonge - close - 5 Mar 2016
avatar joomla-cms-bot joomla-cms-bot - close - 5 Mar 2016
avatar wilsonge wilsonge - reference | 74048f7 - 5 Mar 16
avatar wilsonge wilsonge - merge - 5 Mar 2016
avatar wilsonge wilsonge - close - 5 Mar 2016
avatar wilsonge
wilsonge - comment - 5 Mar 2016

Merged

avatar joomla-cms-bot joomla-cms-bot - change - 5 Mar 2016
Labels Removed: ?
avatar wilsonge wilsonge - change - 5 Mar 2016
Milestone Added:
avatar nonumber
nonumber - comment - 5 Mar 2016

Thanks :)

avatar nonumber nonumber - head_ref_deleted - 5 Mar 2016
avatar nonumber
nonumber - comment - 6 Mar 2016

Yes, now fixed in:
#9317

Add a Comment

Login with GitHub to post a comment