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

Hover widget: Markdown renderer - combine MarkdownRendererOptions wit… #4

Conversation

Ulop
Copy link
Collaborator

@Ulop Ulop commented Oct 19, 2021

Расширил MarkdownRendererOptions следующими полями:

  • headingBlockRenderer - для возможности переопределить, как будет рисоваться тег h[1-6]. Будет использоваться для "дорисовки" к имени функции, к которой выведен тултип, кнопки открытия справки
  • onClickHandler?: (content: string, event?: IMouseEvent) => boolean - для обработки нажатия на вышеуказанную кнопку, при возвращении false, будет выполняться логика стандартного обработчика
  • sanitizerConfig

SanitizerConfig и MarkdownRendererOptions для предотвращения дублирования кода. Если бы оба объекта передавались отдельными методами, то во всех затронутых ревизией файлах было бы дублирование методов setMarkdownSanitizerConfig/setMarkdownRendererOptions

me_custom.mp4

@Ulop Ulop merged commit 4bb81c0 into standalone/0.29.x Oct 20, 2021
@Ulop Ulop deleted the combined_markdown_renderer_options_with_sanitizer_config branch October 20, 2021 07:13
@github-actions github-actions bot locked and limited conversation to collaborators Jan 12, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants