codex-team/editor.js

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

Open

#2.833 geöffnet am 29. Sept. 2024

Auf GitHub ansehen
 (4 Kommentare) (1 Reaktion) (0 zugewiesene Personen)TypeScript (26.602 Stars) (1.985 Forks)batch import
buggood first issue

Beschreibung

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:

Contributor Guide

Tech Stack
typescriptjavascript
Domain
frontend
Issue Type
bug
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 hours
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
fresh
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
Basic knowledge of JavaScriptFamiliarity with Editor.js configuration
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
75
Research-Richtung
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`.