Skip to content

Remove react-contenteditable dependency - #12761

Open
jna4k wants to merge 1 commit into
geosolutions-it:masterfrom
jna4k:remove-react-contenteditable
Open

Remove react-contenteditable dependency#12761
jna4k wants to merge 1 commit into
geosolutions-it:masterfrom
jna4k:remove-react-contenteditable

Conversation

@jna4k

@jna4k jna4k commented Aug 2, 2026

Copy link
Copy Markdown
Collaborator

Description

This PR removes the react-contenteditable dependency and replaces the functionality with standard html.

  • I removed the setText prop because it was not being used anywhere.
  • I removed the onChange handler from the element, because div doesn't support it, while the ContentEditable element did. This is not a problem because it was not being used, and even if it did, the onBlur handler would end up catching the change anyways.
  • I added suppressContentEditableWarning to avoid errors in the browser devtool

Please check if the PR fulfills these requirements

What kind of change does this PR introduce? (check one with "x", remove the others)

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Other... Please describe:

Issue

What is the current behavior?

#12760

What is the new behavior?

We will no longer download the react-contentable npm package and the component is replaced with a

Breaking change

Does this PR introduce a breaking change? (check one with "x", remove the other)

  • Yes, and I documented them in migration notes
  • No

Other useful information

@cla-bot cla-bot Bot added the CLA Ready label Aug 2, 2026
@tdipisa tdipisa linked an issue Aug 10, 2026 that may be closed by this pull request
6 tasks
@tdipisa
tdipisa requested a review from offtherailz August 10, 2026 08:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove react-contenteditable dependency

3 participants