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.

Sign up for GitHub

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

Jump to bottom

Customize CKEditor's toolbar #5130

Closed
alansemenov opened this issue Jun 19, 2017 · 3 comments
Closed

Customize CKEditor's toolbar #5130

alansemenov opened this issue Jun 19, 2017 · 3 comments
Assignees
@ashklianko

Comments

@alansemenov
Copy link
Member

alansemenov commented Jun 19, 2017 •
edited
Loading

Customize the default configuration of the toolbar so that we have the same set of tools as we do now. Implement support for custom configuration via HTML Area schema.

http://docs.ckeditor.com/#!/guide/dev_toolbar
@alansemenov
Copy link
Member Author

Also, don't forget that toolbar should only become visible when the HTML Area gets focused.

@ashklianko
Copy link
Member

Sure

ashklianko added a commit that referenced this issue Aug 10, 2017
@ashklianko
Customize CKEditor's toolbar #5130
c641fe9 
-Sticky toolbar, custom config, hiding toolbar on blur etc.
-'Pre' (wrapping with <pre> tag) is done via formatting dropdown menu option 'Fomatted'
-'Code' wrapping does not seem to work?
alansemenov added a commit that referenced this issue Aug 17, 2017
@alansemenov
Merge pull request #5461 from enonic/issue-5130
61dfb37 
Customize CKEditor's toolbar #5130
@alansemenov
Copy link
Member Author

@ashklianko
There were a few small things to fix, see if you can do this in one of the next PRs:

  • The footer of the area is now always visible (also in TinyMCE), we should hide it when the area is not active
  • Remove the "Code" (it doesn't work because <code> tag must be placed inside a container element, btw) and "Create Div container" buttons
  • Move "Block quote" inside the group of buttons to the left
  • Remove 20px margin inside the HTML Area's body element

@alansemenov alansemenov mentioned this issue Aug 21, 2017
Merged
@ashklianko ashklianko mentioned this issue Sep 25, 2017
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ashklianko ashklianko

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ashklianko @alansemenov