TheWebShop / bootstrap-shortcodes

Wordpress plugin to add shortcodes for Twitter Bootstrap 3.0
36 stars 15 forks source link

wp 3.9 #25

Closed Firestorm-Graphics closed 10 years ago

Firestorm-Graphics commented 10 years ago

incompatible with wp 3.9 and tinymce 4, createControl is now deprecated

Sinetheta commented 10 years ago

Image

Danny159 commented 10 years ago

Any news on this? As its stopped working on all my websites now?

Sinetheta commented 10 years ago

Sorry @Danny159 I haven't had time to address this myself, but I'd be happy to merge in any fixes.

Alternatively, this would be a good place for discussion of the update path. If tinymce.createControl is gone, what's taking it's place? If someone could suggest a replacement or find code samples online, I'm sure it would greatly speed up this process.

No3x commented 10 years ago

Hi @Sinetheta & the others. I've taken a look into the changes and want to share some thoughts:

No3x commented 10 years ago

After hours of trying things I have to say that it seems that tinymce 4 is not meant to be backwards compatible.

Sinetheta commented 10 years ago

@No3x it seems that way, and thanks for doing some experimentation.

It kinda blows my mind how little info there is online about the differences between 3 and 4 or how plugin authors might tackle updates (besides that silly compat plugin targeted at users).

Firestorm-Graphics commented 10 years ago

ive created a premium plugin using the new tinymce 4, i dont do backwards compatibility with wp as it should be kept up to date, but maybe you could check wp version in php and include the right js dependant on wp version

Firestorm-Graphics commented 10 years ago

split button is in tinymce 4, ive used it in my plugin

No3x commented 10 years ago

@Sinetheta I agree with the little amount of information available online. @Firestorm-Graphics Yes split button is in 4 for sure but I've talked about the use of compatible code for 3 and 4 as well. I can't use split button in a new plugin version target wp 3.9 without breaking it for all users on versions prior 3.9. (we would need an abstraction/compbatibility layer [what the compat3x could be]).

By the way I really like the changes made in the new version of tinymce 4.

Sinetheta commented 10 years ago

Like @Firestorm-Graphics I tend to favour a simpler plugin over backwards compatibility, this thing's ugly enough as is. Besides, if your WP isn't up-to-date you've got bigger problems than some missing WYSIWYG helper buttons :wink:

No3x commented 10 years ago

All right! So that's a clear direction.

Danny159 commented 10 years ago

I agree, keep it simple and working with the latest.

No3x commented 10 years ago

Hey guys, I've ported the plugin to 3.9. (https://github.com/TheWebShop/bootstrap-shortcodes/pull/27) There is a lot of testing/improvement/refactoring required especially:

The mce_buttons_3 didn't work until all problems in the plugins had been resolved - but now everything is fine.

22, #15 should be solved by this.

Danny159 commented 10 years ago

Nice one @No3x looks good, I will test.

Sinetheta commented 10 years ago

:100: I'll close this issue once everything's live in the wordpress plugin repo, but huge props to @No3x for tackling this migration!