Support for multiple class names on InlineBlot about parchment HOT 6 CLOSED

hey @chrisronline I did this to register multiple class names to a blot

class HeaderBlot extends Block {
  static create() {
    const node = super.create();
    node.classList.add('subheading');
    node.classList.add('secondary-color');
    return node;
  }
}

Hope this helps!

from parchment.

It’s just how our CSS heirarchy is setup - we could possibly change that but I don’t see why multiple class names isn’t a valid use case and something that is seemingly easy to support

from parchment.

On a separate note, the reason I'm using spann is because of this issue in quill and I don't know a better way around it. Feedback/thoughts are more than welcome!

from parchment.

Why do you need multiple classes as opposed to one unique one?

from parchment.

It's not easy to support. All features require ongoing maintenance and tests and all future features would have to consider interaction with this feature. Second it is not correct that just turning that line into a loop is all it would take.

from parchment.

Thanks for clarifying! I’m sorry if I assumed it was a simple request but I appreciate your time and patience. I’ve been using Quill/Parchment for just a short time and it’s been great so far!

from parchment.