codex-team/editor.js

`inlineToolbar` always show convertTo, inconsist behavior with config value, and cannot be removed

Open

#2,833 opened on 2024年9月29日

GitHub で見る
 (4 comments) (1 reaction) (0 assignees)TypeScript (26,602 stars) (1,985 forks)batch import
buggood first issue

説明

Describe a bug.

const editor = new EditorJS({
  holder: 'editor',
  tools: {
    paragraph: {
      // built-in inline tools: link, bold, italic
      // disable inline tools: false,
      inlineToolbar: ['bold', 'link'],
    },
  },
})

Expected behavior:

convertTo should also follow inlineToolbar config, which control it show or hide.

Screenshots:

image

As you see, it even show when no convertor available.

Device, Browser, OS:

Editor.js version:

Plugins you use with their versions:

コントリビューターガイド

技術スタック
typescriptjavascript
領域
frontend
Issue 種別
bug
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
3
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
1-3 hours
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
fresh
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
clear
前提条件
Basic knowledge of JavaScriptFamiliarity with Editor.js configuration
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
75
調査方針
Investigate the inline toolbar component to understand how the convertTo button visibility is controlled. Look at the configuration merging logic to ensure that when `inlineToolbar` is explicitly set, the convertTo should respect that configuration. Check if there is a missing check for the availability of convertors. The relevant files are likely in `src/components/inline toolbar.ts` and `src/components/tools.ts`.
`inlineToolbar` always show convertTo, inconsist behavior with config value, and cannot be removed · codex-team/editor.js#2833 | Good First Issue