search

editor-js / header

Header Tool for Editor.js 2.0
MIT License
96 stars 122 forks source link

Add config for available heading levels and default level #37

Closed JustTSK closed 4 years ago

JustTSK commented 4 years ago

Fixes #14

JustTSK commented 4 years ago

@neSpecc, @gohabereg forgot to tag someone in Reviewers, can you take a look please?

neSpecc commented 4 years ago

@neSpecc, @gohabereg forgot to tag someone in Reviewers, can you take a look please?

Ok, thanks. I will test it.

JustTSK commented 4 years ago

@neSpecc any luck testing this?

natterstefan commented 4 years ago

Hi @neSpecc, hi @gohabereg,

any chance you can take a look at this soon? Our customers need this as well. Thank you.

lih1989 commented 4 years ago

Это нужные настройки, почему так долго не применяются?

neSpecc commented 4 years ago

Thanks, @JustTSK

Farrien commented 4 years ago

Это нужные настройки, почему так долго не применяются?

Уже больше года жду этого изменения) Пришлось убрать Editor.js из проекта из-за отсутствия запрета отдельных заголовков.

lih1989 commented 4 years ago

Это нужные настройки, почему так долго не применяются?

Уже больше года жду этого изменения) Пришлось убрать Editor.js из проекта из-за отсутствия запрета отдельных заголовков.

По моему, как то через чур радикально :)

Можно было просто расширить класс:

import Header from '@editorjs/header';

class CustomHeader extends Header {
  get levels() {
    return super.levels.filter((l) => [2, 3, 4, 5, 6].includes(l.number));
  }

  get defaultLevel() {
    return this.levels[0];
  }
}