codex-team / codex-team/editor.js

Looks like tool sanitize config merged with inline tools should be used for processHTML

Open
#2,382 0 comments 0 reactions 0 assignees View on GitHub
bug
Dominant language
TypeScript
Stars
31.9k
Forks
2.2k
Avg merge
1d 1h
Merged PRs (30d)
1

Description

https://github.com/codex-team/editor.js/blob/39018e07e4a9657ed1b3fe57a9c509e85fb1bffd/src/components/modules/paste.ts#LL663C20-L663C20

here used only combined inline and tunes config, and tool config is ignored. For ex. if my tool support br and I specified it - during paste event br will be sanitized. Looks like here should this.sanitizeConfig should be used instead of this.baseSanitizeConfig

Steps to reproduce:
1. paste

Expected behavior:
tool sanitize config should be applied too

Screenshots:
-

Device, Browser, OS:
any

Editor.js version:
2.0

Plugins you use with their versions:
-

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.