Kunena / Kunena-Forum

Kunena Forum - Forum / Bulletin Board / Discussions component for Joomla - This is the 6.x/5.x main development branch. Please do not open issues regarding earlier versions of Kunena
https://www.kunena.org
GNU General Public License v3.0
1.75k stars 812 forks source link

Remove X button from CKEditor toolbar no longer works #9698

Closed rich20 closed 3 months ago

rich20 commented 4 months ago

Describe the bug https://www.kunena.org/forum/general-questions-and-how-tos/168492-remove-x-button-from-ckeditor-toolbar Confirmed

To Reproduce Select in the Aurelia configuration (tab BBcode) Show X symbol = No

Actual result The symbol is still displayed in the editor

Additional context It is possible that it no longer works due to the switch from Twitter to X.

Gindi50 commented 4 months ago

I am somewhat puzzled by this error message. In my installations, Ebay symbol in the editor, X symbol in the editor, Instagram symbol in the editor and Soundcloud symbol in the editor are listed in /Aurelia/BBCode, but in the frontend in the editor menu bar only "Tweet", (invisible without button) "Ebay", "soundcloud" and "instagram" are displayed in this group. I can also switch these visible buttons on and off in /Aurelia/BBCode. An "X" or "Twitter" button is not available, so "X symbol in editor" has no function. I also could not find this button in /libraries/kunena/src/BBCode/KunenaBBCode.php, /aurelia_editor.xml and editor.xml. "Tweet", "Ebay", "soundcloud" and "instagram" are mentioned there and have functions there.

Pinkeltje commented 4 months ago

Don't know if it helps, but in 6.3.1 at least hiding the button worked. Showing button no image displayed. 2024-06-28 Screenshot 475 2024-06-28 Screenshot 476

More findings; When renaming \core\js\plugins\twitter\icons**x_logo.png to twitter.png** and changing line 11 in plugin.js to CKEDITOR.plugins.add( 'twitter', { icons: 'twitter', x logo is shown.

In 6.3.3. dev when putiing back \core\js\plugins\twitter from 6.3.1 and deleting \core\js\plugins\x and changing params.ini Line 57 back to Twitter="0" and line 65 back to editorButtons="Smiley,Ebay,Twitter,Instagram,Soundcloud" It works in 6.3.3. dev as well.

Gindi50 commented 4 months ago

@pinkeltje: Unfortunately the suggestion doesn't work for me. I have two versions of the files in /core/js/plugins/, twitter/icons/twitter.png and x_social/icons/x_logo.png. The two plugin.js are adapted accordingly. I have set the params.ini according to the settings from K6.2.4 (everything still worked there). For testing I changed the group and the user for twitter and x_social with Chown but the X button was still displayed in the menu bar. In the text field I clicked right mouse button - Show page source and a file with the header "<!DOCTYPE html>" was displayed. From row 1519 the following was displayed:

<td width="40%"
                                                                    class="paramlist_key"><label id="jform_X_Social-lbl" for="jform_X_Social">
    Show <strong>X</strong> button</label>
</td>
                                                                <td class="paramlist_value"><select id="jform_X_Social" name="jform[X_Social]" class="form-select" aria-describedby="jform_X_Social-desc">
    <option value="0" selected="selected">No</option>
    <option value="1">Yes</option>
</select>
</td>
                                                                <td class="paramlist_key">Show <strong>X</strong> button on editor</td>
                                                            </tr>

The settings for ebay are above and instagram and soundcloud are below. The difference is that x_social has the text "Show x button" and "Show x button on editor". It is possible that the "Show" prevents the button from being activated and deactivated.