search

bastihilger / nova-tiptap

Laravel Nova Tiptap Field
MIT License
172 stars 61 forks source link

Allow to specify headings level as range #18

Closed alies-dev closed 4 years ago

alies-dev commented 4 years ago

By SEO and accessibility [and standards], it's better to have only one H1 per page. For this reason it will be nice to specify available headers as a range or a list, e.g. [3,4,5,6]. Is it possible right now? If now, can we have this feature in the package?

PS: thanks for your great package!

bastihilger commented 4 years ago

Ah - ok, that is an interesting point. Not possible at the moment, but should be relatively easy to integrate. I would probably make it so that the headingLevels method can accept either an integer (current functionality) or an array, which would be the new functionality you propose. I will keep you posted here.

alies-dev commented 4 years ago

@manogi that will be amazing! I'll wait for this feature 👍

desaintflorent commented 4 years ago

I'm interested in this functionality too !

bastihilger commented 4 years ago

This is now done in Release 0.4.0 (see Readme part about heading levels).

desaintflorent commented 4 years ago

That was fast 👍