Skip to content

fix: remove invalid bullet format from ReactQuill configs#6585

Open
SueJianjian wants to merge 2 commits into
keephq:mainfrom
SueJianjian:fix-reactquill-formats-bullet
Open

fix: remove invalid bullet format from ReactQuill configs#6585
SueJianjian wants to merge 2 commits into
keephq:mainfrom
SueJianjian:fix-reactquill-formats-bullet

Conversation

@SueJianjian

@SueJianjian SueJianjian commented Jun 16, 2026

Copy link
Copy Markdown

Summary

Closes #6581

  • removes the invalid bullet entry from the Quill formats arrays
  • keeps list in place, which already covers ordered and bullet lists
  • updates both affected components to avoid the browser console error consistently

Why

Quill expects list in the formats array, while bullet is only used as a toolbar value such as { list: "bullet" }.

Having bullet inside formats causes the console error:

Cannot register bullet specified in formats config

Change

  • update keep-ui/features/alerts/alert-note/ui/alert-note-modal.tsx
  • update keep-ui/features/incidents/create-or-update-incident/ui/create-or-update-incident-form.tsx

Validation

  • verified both affected files contain the invalid bullet entry in formats
  • removed only bullet and left the existing toolbar config unchanged
  • scope kept intentionally minimal because the issue is a direct config mismatch

Notes

  • I did not broaden this into a richer editor refactor or toolbar behavior change.

@dosubot dosubot Bot added size:XS This PR changes 0-9 lines, ignoring generated files. Bug Something isn't working javascript Pull requests that update Javascript code labels Jun 16, 2026
@shahargl

Copy link
Copy Markdown
Member

hey @SueJianjian , may you link the ticket?

@SueJianjian

Copy link
Copy Markdown
Author

Linked the PR to #6581 in the description now. Thanks for flagging it.

@SueJianjian

Copy link
Copy Markdown
Author

Hi @shahargl, this now links to #6581 and the checks are green. Could you please take another look when you have a moment?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bug Something isn't working javascript Pull requests that update Javascript code size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[🐛 Bug]: ReactQuill console error

2 participants