Skip to content

Update integration.md #1261

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

patacra
Copy link

@patacra patacra commented May 8, 2025

Correct the CKEditor integration documentation

Summary of the change:

  • In "Sample 2 - With JQuery Selector", the JQuery selector is wrong as the dot is for selecting a class,
    and this class is missing and set on the id instead.
  • To help people understand, I propose to modify the id to show that it has to be unique if you use this way of
    setting up the editor. Personally, I prefer the class version, which will handle multiple editors in the page.

Correct the CKEditor integration: JQuery selector was wrong, so added a class like in the other examples and made the id look more unique, for clarity.
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.

1 participant