Skip to content

feat: upgrade TinyMCE to v8.0.2 and update configurations for enhanced editor functionality#818

Merged
kevin-atnos merged 10 commits intomasterfrom
CU-86c4tvrvm_General-Mise-a-jour-de-TinyMCE-WYSIWYG
Oct 8, 2025
Merged

feat: upgrade TinyMCE to v8.0.2 and update configurations for enhanced editor functionality#818
kevin-atnos merged 10 commits intomasterfrom
CU-86c4tvrvm_General-Mise-a-jour-de-TinyMCE-WYSIWYG

Conversation

@brunto
Copy link
Copy Markdown
Collaborator

@brunto brunto commented Sep 8, 2025

This pull request upgrades TinyMCE from version 4 to version 8 and updates its integration throughout the Angular application. The changes include updating how TinyMCE is loaded and initialized, modifying editor configuration options, and refactoring related styles to support the new version. These updates ensure compatibility with the latest TinyMCE features and improve maintainability.

TinyMCE Upgrade and Integration

  • Upgraded the TinyMCE dependency from version 4.9.11 to 8.0.2 in package.json and updated asset handling in angular.json to copy the new TinyMCE files and use tinymce.min.js instead of older scripts/styles. [1] [2] [3]
  • Updated TinyMCE initialization in all editor-related components (evaluations.component.ts, measures.component.ts, questions.component.ts) to use new configuration options (license_key, base_url, suffix, and placeholder) and removed deprecated skin option. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10]

Editor Lifecycle and Cleanup

  • Refactored the editor cleanup logic in questions.component.ts by introducing a new private closeEditor() method to handle editor removal and placeholder reset, improving code clarity and reliability. [1] [2]

Styling Updates for TinyMCE 8

  • Replaced legacy TinyMCE 4 styles in _editor.scss with new selectors and styles compatible with TinyMCE 8, including updates for .tox-tinymce elements and toolbar appearance.

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This pull request upgrades TinyMCE from version 4.9.11 to 8.0.2, requiring significant configuration and integration updates throughout the Angular application. The upgrade modernizes the editor functionality and improves maintainability by aligning with the latest TinyMCE features and API changes.

  • Upgraded TinyMCE dependency to version 8.0.2 and updated build configuration
  • Updated TinyMCE initialization configuration across all editor components
  • Refactored editor cleanup logic and replaced deprecated styling selectors

Reviewed Changes

Copilot reviewed 8 out of 9 changed files in this pull request and generated 1 comment.

File Description
package.json Updates TinyMCE dependency from 4.9.11 to 8.0.2
angular.json Modifies asset copying and script loading for TinyMCE 8 compatibility
src/assets/stylesheets/_editor.scss Replaces legacy TinyMCE 4 CSS selectors with TinyMCE 8 equivalents
Multiple component files Updates TinyMCE initialization with new configuration options and cleanup methods
Comments suppressed due to low confidence (1)

src/app/modules/structure/content/questions/questions.component.ts:1

  • The editor cleanup may cause race conditions when called during the focusout event handler. Consider adding a check to ensure the editor is still valid before calling tinymce.remove(this.editor).
import {

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread src/app/modules/pia/content/questions/questions.component.ts Outdated
@kevin-atnos kevin-atnos force-pushed the CU-86c4tvrvm_General-Mise-a-jour-de-TinyMCE-WYSIWYG branch 2 times, most recently from 66ef04f to f744414 Compare October 3, 2025 14:50
@kevin-atnos
Copy link
Copy Markdown
Collaborator

kevin-atnos commented Oct 3, 2025

  • @brunto @kevin-atnos Supprimer les secret key sur Github (CYPRESS) une fois cette PR mergée.

@brunto brunto force-pushed the CU-86c4tvrvm_General-Mise-a-jour-de-TinyMCE-WYSIWYG branch from f744414 to a5092d1 Compare October 3, 2025 15:46
@brunto brunto force-pushed the CU-86c4tvrvm_General-Mise-a-jour-de-TinyMCE-WYSIWYG branch from cf09ed0 to 716c2f3 Compare October 7, 2025 09:13
@kevin-atnos kevin-atnos merged commit 5ffc30d into master Oct 8, 2025
3 checks passed
@kevin-atnos kevin-atnos deleted the CU-86c4tvrvm_General-Mise-a-jour-de-TinyMCE-WYSIWYG branch October 8, 2025 14:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants