Skip to content

[4.0] Fix TinyMCE - simple preset#30583

Merged
laoneo merged 1 commit into4.0-devfrom
30014-tinymce
Sep 8, 2020
Merged

[4.0] Fix TinyMCE - simple preset#30583
laoneo merged 1 commit into4.0-devfrom
30014-tinymce

Conversation

@Quy
Copy link
Copy Markdown
Contributor

@Quy Quy commented Sep 6, 2020

Pull Request for Issue #30014 .

Summary of Changes

Fix incorrect key.

Steps to reproduce the issue

  1. Go to Plugin Manager and open TinyMCE
  2. Click the medium and advanced preset buttons

You'll notice below the TinyMCE toolbar changes based on the preset chosen.

  1. Click the simple preset button

Expected result

Toolbar with buttons.

Actual result

Toolbar is empty.

@ChristineWk
Copy link
Copy Markdown

I have tested this item ✅ successfully on 46cb99e


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

1 similar comment
@ghost
Copy link
Copy Markdown

ghost commented Sep 7, 2020

I have tested this item ✅ successfully on 46cb99e


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

@SharkyKZ
Copy link
Copy Markdown
Contributor

SharkyKZ commented Sep 7, 2020

RTC.


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Sep 7, 2020
@laoneo laoneo added this to the Joomla 4.0 milestone Sep 8, 2020
@laoneo laoneo merged commit 82a8521 into 4.0-dev Sep 8, 2020
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Sep 8, 2020
@laoneo laoneo deleted the 30014-tinymce branch September 8, 2020 14:10
@laoneo
Copy link
Copy Markdown
Member

laoneo commented Sep 8, 2020

Thanks!

sakiss pushed a commit to sakiss/joomla-cms that referenced this pull request Oct 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants