codex-team/editor.js

[Bug] Switching to readonly mode with an empty editor

Open

#1,702 opened on 2021年6月1日

GitHub で見る
 (2 comments) (2 reactions) (1 assignee)TypeScript (1,985 forks)batch import
buggood first issue

Repository metrics

Stars
 (26,602 stars)
PR merge metrics
 (30d に merged PR はありません)

説明

Describe a bug. If you switch the editor to read-only mode after initialization, an error appears: System.Exception: Cannot read property 'holder' of undefined

togglePreviewVisibility(visible: boolean) {
		this._editor.readOnly.toggle(visible); // this._editor.blocks.getBlocksCount() === 0
		this.previewVisible = visible;
	}

Editor.js version: 2.22.0

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

調査方針
このエラーは、空のエディター（ブロックなし）で読み取り専用モードを切り替える際に発生します。問題は togglePreviewVisibility メソッドまたは関連する readOnly ロジックにある可能性があります。'holder' プロパティがアクセスされる箇所を検索し、null チェックを追加してください。editor.js 内の読み取り専用モード切り替えに関連するコードを確認してください。
技術スタック
typescript
領域
frontend
Issue 種別
バグ
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
1
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
1時間未満
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
アクティブ
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
明確
前提条件
JavaScriptEditor.js basics
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
90