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

fix for the stylesheet toobar name changing. #578

Merged
merged 2 commits into from Jan 25, 2024

Conversation

KevinJump
Copy link
Owner

Sometime between v10 and v13 the TinyMCE names for the toobar styles button changed from "styleselect" to "styles"

if you import for v10 setup into v13 this isn't changed, so this pr adds a config serializer for the TimeMCE property that delves into the json and renames it. (we could have done a string rename, but less elegant more danger!)

Copy link

@lukehook lukehook left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM - Thanks so much for jumping on this so quickly Kevin H5YR!

@KevinJump KevinJump merged commit 1db5a03 into v13/main Jan 25, 2024
@KevinJump KevinJump deleted the v13/rte-styletoolbar-fix branch January 25, 2024 15:15
KevinJump added a commit that referenced this pull request Feb 6, 2024
* fix for the stylesheet toobar name changing.

* Add extra check incase 'toolbar' is missing
KevinJump added a commit that referenced this pull request Feb 16, 2024
* roots - part 1. wip.

* Sort the generic properties correctly on xml merge.

* merging properties.

* generic - image cropper.

* Add all the stops.

* Add settings to the settings box.

* make sure deletes when the key is a string work on merge.

* Make status checks - check all folders.

* fix missing null check on callbacks., causing firstboot to fail.

* some logging around first boot.

* moving controllers to .assets project (prepping core for multi-targeting)

* fix for the stylesheet toobar name changing. (#578)

* fix for the stylesheet toobar name changing.

* Add extra check incase 'toolbar' is missing

* Add RTE null check when editor config is missing.

* Make it all less breaking.

* experimental: background notification threading.

* Exeuction flow around background task (as per core)

* top and tail the ordering message, so it looks like things are indeed happening.

* Background threding, and state events.

* least best breaking changes.

* build action

* All branches

* Duplicate checking.

* chgore: schema file update

* #583 - removes `.substring(1)` from handler names (#584)

see #583

* Fix - compatability - mark field setters as obsolete

---------

Co-authored-by: Liam Laverty <liamlaverty@gmail.com>
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.

None yet

2 participants