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

применение тэга b к заголовку display и пр. пожелания #38

Open
Dutik opened this issue Jan 17, 2020 · 3 comments

Comments

@Dutik
Copy link

Dutik commented Jan 17, 2020

  1. При назначении полужирности (b) к <h1 class="display-2">, его невозможно "отжать". Т.е. заголовок остаётся полужирным.

  2. Было бы удобно выделять активные инлайновые пункты. При установке курсора на жирный текст - применять активный класс к данному пункту меню.

  3. Для блоков типа Html ввести параметр inlineToolbar: ['link','bold','italic' etc.] где указывать какие инлайновые элементы разрешены. Скажем, в заголовки H нелепо добавлять списки.

@yakovlevga
Copy link
Owner

Привет! Спасибо за комментарии, постараюсь пофиксить в новой версии.
Про третий пункт согласен, добавлю в туду.

@Dutik
Copy link
Author

Dutik commented Feb 3, 2020

Погуглил. С первым пунктом ничего сделать не получится. Косяк в том, что у display-2 жирность отлична от normal (400) и у разработчиков дальше рвёт крышу в сторону нелогичности, но их дело. Так что execCommand тут не сработает.

Со вторым пунктом, если делать через queryCommandState будет косяк со ссылками. Типа, если ссылка подчёркнута, значит к ней применён стиль underline.

И ещё не хватает возможности убрать ссылку :) Сделать текст ссылкой можно, а разлинковать - нет :)

@yakovlevga
Copy link
Owner

По первому пункту можно подумать как анализировать selection range и какие-то простые случаи обрабатывать. Попробую реализовать это в новой версии - пока главная цель зарелизить версию без зависимости от jquery, в функциональном стиле и нормально покрытую тестами. Большое спасибо за идеи )

@yakovlevga yakovlevga added this to the ver-2 milestone Feb 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants