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

Text Color and Background extensions, and additional supporting of the "type" attribute to OrderedList #880

Closed
wants to merge 3 commits into from

Conversation

Joost-dm
Copy link

@Joost-dm Joost-dm commented Nov 2, 2020

Text Color and Background extensions, and additional supporting of the "type" attribute of

    to OrderedList =)

Adding Text Color Extension
Adding Text BackgroundColor extension
Adding a supporting of the "type" attribute to OrderedList
@Joost-dm Joost-dm changed the title Широков Василий Text Color and Background extensions, and additional supporting of the "type" attribute of to OrderedList Nov 2, 2020
@Joost-dm Joost-dm changed the title Text Color and Background extensions, and additional supporting of the "type" attribute of to OrderedList Text Color and Background extensions, and additional supporting of the "type" attribute to OrderedList Nov 2, 2020
@hanspagel hanspagel self-assigned this Jan 4, 2021
@hanspagel hanspagel mentioned this pull request Jan 4, 2021
6 tasks
@hanspagel
Copy link
Contributor

hanspagel commented Jan 4, 2021

Thanks for your work on this @Joost-dm! We’ll add support for background-color and color to tiptap v2. 🙌 I’m not sure about the type option for OrderedLists, but it’ll definitely easier to add this with v2 for everyone who wants to do it. It’ll make a great example for the documentation. So thank you very much!

If you don’t mind: For maintainers it’s way easier to have multiple small PRs for features, than one PR with multiple features. It’s easier to look through and it’s easier to merge one by one. Have a nice day, and thanks again for your contribution! 💖

Oh, and I added it to the list of community extensions: #819

@hanspagel hanspagel closed this Jan 4, 2021
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.

None yet

2 participants