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

Add ckeditor tabletools plugin #3983

Merged

Conversation

Senen
Copy link
Member

@Senen Senen commented Apr 23, 2020

References

Bug discovered working on #3979.

Objectives

Fix error detected during manual testing of ckeditor table plugin: When a user clicks on table "Cell properties" feature from contextual menu browser throws a javascript error when running in production environment.

Tabletools plugin provides the dialog window to allow users to edit "Cell properties".

Documentation: https://ckeditor.com/cke4/addon/tabletools

How to reproduce

Create a table within any application ckeditor, right click on any cell and the click on "Cell properties" from contextual menu. You will see a javascript error within browser console.

This is happening with all browsers.

Error location:
Captura de pantalla 2020-04-23 a las 18 33 46

Visual Changes

None

Notes

This change was successfully tested on production environment!

@Senen Senen added the Bug label Apr 23, 2020
@Senen Senen self-assigned this Apr 23, 2020
@Senen Senen added this to Reviewing in Consul Democracy via automation Apr 23, 2020
@javierm javierm self-assigned this Apr 23, 2020
This plugin provides more control over tables and solves a JS error thrown
when user clicks on "Cell properties" ckeditor feature.

https://ckeditor.com/cke4/addon/tabletools
@Senen Senen force-pushed the ckeditor-table-plugin-addons branch from 82c3f61 to 103726f Compare April 23, 2020 16:18
@Senen Senen changed the title Add ckeditor table plugin addons Add ckeditor tabletools plugin Apr 23, 2020
@Senen Senen marked this pull request as ready for review April 23, 2020 16:36
@Senen Senen requested a review from javierm April 23, 2020 16:36
Consul Democracy automation moved this from Reviewing to Testing Apr 23, 2020
Copy link
Member

@javierm javierm left a comment

Choose a reason for hiding this comment

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

Great 😄.

@javierm javierm merged commit f56306f into consuldemocracy:master Apr 23, 2020
Consul Democracy automation moved this from Testing to Release 1.2.0 Apr 23, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants