Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[5.2] TinyMCE Accordion #43577

Merged
merged 4 commits into from
Jun 4, 2024
Merged

[5.2] TinyMCE Accordion #43577

merged 4 commits into from
Jun 4, 2024

Conversation

brianteeman
Copy link
Contributor

@brianteeman brianteeman commented May 30, 2024

Since Joomla 5.0 we have shipped a version of tinyMCE that includes a plugin called accordion. However we never exposed it to our plugin so it was not available to use.

This PR remedies that.
New installs will see the accordion icon on the toolbar with the list icons For existing installs you will need to EITHER drag the icon into the toolbar in the plugin configuration OR if you havent changed the toolbar you can use the clear and the the use advanced preset buttons.

Technically tinymce have misnamed the the plugin as it should not be confused with other accordion functionality such as the js based solution that can be enabled with bootstrap.

This plugin implements the native html details/summary disclosure element which requires no javascript and is accessible out of the box.

To test

As this requires a js change either apply the pr and run nmp ci or use one of the prebuilt packages

If it is a new install then the icon will now be able in the toolbar. If it is an upgrade then you will need to use the tinymce plugin editor to add it (as described above)

Demo Usage

accordion

Notes

  1. It is a known issue in with tinymce that currently the only way to exit the accordion is to use the arrow keys to navigate out of the details/summary Unable to create text below accordion tinymce/tinymce#8890
  2. You can quickly create an accordion by highlighting a line of text and pressing the accordion button. This will convert the text into the details part

Link to documentations

Please select:

  • Documentation link for docs.joomla.org:

  • No documentation changes for docs.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • No documentation changes for manual.joomla.org needed

Since Joomla 5.0 we have shipped a version of tinyMCE that includes a plugin called accordion. However we never exposed it to our plugin so it was not available to use.

This PR remedies that.
New installs will see the accordion icon on the toolbar with the list icons
For existing installs you will need to EITHER drag the icon into the toolbar in the plugin configuration OR if you havent changed the toolbar you can use the clear and the the use advanced preset buttons.

Technically tinymce have misnamed the the plugin as it should not be confused with other accordion functionality such as the js based solution that can be enabled with bootstrap.

This plugin implements the native html details/summary disclosure element which requires no javascript and is accessible out of the box.

To test
As this requires a js change either apply the pr and run `nmp ci` or use one of the prebuilt packages

If it is a new install then the icon will now be able in the toolbar. If it is an upgrade then you will need to use the tinymce plugin editor to add it (as described above)
@joomla-cms-bot joomla-cms-bot added NPM Resource Changed This Pull Request can't be tested by Patchtester PR-5.2-dev labels May 30, 2024
@brianteeman
Copy link
Contributor Author

could someone please tag this as feature

@alikon alikon added the Feature label May 31, 2024
@Fedik
Copy link
Member

Fedik commented May 31, 2024

I have tested this item ✅ successfully on ff21e79


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

1 similar comment
@viocassel
Copy link
Contributor

I have tested this item ✅ successfully on ff21e79


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

@Quy Quy removed the Feature label May 31, 2024
@Quy
Copy link
Contributor

Quy commented May 31, 2024

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label May 31, 2024
@pe7er pe7er self-assigned this Jun 3, 2024
@pe7er pe7er enabled auto-merge (squash) June 3, 2024 10:19
@pe7er pe7er disabled auto-merge June 3, 2024 10:20
@pe7er pe7er enabled auto-merge (squash) June 4, 2024 08:55
@pe7er pe7er merged commit cd7b78d into joomla:5.2-dev Jun 4, 2024
3 of 4 checks passed
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Jun 4, 2024
@brianteeman brianteeman deleted the accordion branch June 4, 2024 11:25
@pe7er
Copy link
Contributor

pe7er commented Jun 4, 2024

Thanks @brianteeman !

@Quy Quy added this to the Joomla! 5.2.0 milestone Jun 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
NPM Resource Changed This Pull Request can't be tested by Patchtester PR-5.2-dev
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants