Skip to content

TTAHUB-4779 - Updating CR ActSum Act Desc to use WYSIWYG#3263

Merged
Andrew565 merged 10 commits intomainfrom
4779-cr-act-desc-wysiwyg
Feb 13, 2026
Merged

TTAHUB-4779 - Updating CR ActSum Act Desc to use WYSIWYG#3263
Andrew565 merged 10 commits intomainfrom
4779-cr-act-desc-wysiwyg

Conversation

@Andrew565
Copy link
Collaborator

Description of change

Updating CR ActSum Act Desc to use a WYSIWYG editor.

How to test

  1. Create a new Collab Report.
  2. Notice on the activity summary page that the activity description field is now a rich text editor.
  3. Click into the field and out again, notice the presence of an error.
  4. Click into the field and type something and then click out, notice the error goes away.

Issue(s)

Checklists

Every PR

  • Meets issue criteria
  • JIRA ticket status updated
  • Code is meaningfully tested
  • Meets accessibility standards (WCAG 2.1 Levels A, AA)
  • API Documentation updated
  • Boundary diagram updated
  • Logical Data Model updated
  • Architectural Decision Records written for major infrastructure decisions
  • UI review complete
  • QA review complete

Before merge to main

  • OHS demo complete
  • Ready to create production PR

Production Deploy

  • PR created as Draft
  • Staging smoke test completed
  • PR transitioned to Open
  • Reviewer added (after transitioning to Open to ensure Slack notifications trigger)
    • Sequence: Draft PR → Smoke test → Open PR → Add reviewer
    • Confirm that Slack notification was sent after reviewer was added

After merge/deploy

  • Update JIRA ticket status

@thewatermethod
Copy link
Collaborator

thewatermethod commented Jan 26, 2026

Screenshot 2026-01-26 at 11 51 02 AM

Do we want a label and a legend?

Copilot AI review requested due to automatic review settings February 4, 2026 21:22
Copy link
Contributor

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 PR updates the Collaboration Report's Activity Summary page to use a WYSIWYG rich text editor for the activity description field, replacing the previous plain textarea implementation.

Changes:

  • Replaced textarea with HookFormRichEditor component for the activity description field
  • Added validation logic to HookFormRichEditor to support required fields with custom error messages
  • Updated the review section to properly display rich text content
  • Extended DOMPurify's allowed HTML tags to include 'del' and 'ins' for strikethrough and insertion formatting
  • Updated tests to interact with the new rich text editor structure

Reviewed changes

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

Show a summary per file
File Description
frontend/src/components/HookFormRichEditor.js Added id and errorMessage props, implemented validation for required fields, and added onBlur handling
frontend/src/pages/CollaborationReportForm/Pages/activitySummary.js Replaced textarea with HookFormRichEditor, removed manual validation function, and marked description as rich text in review section
frontend/src/components/renderReadOnlyContentData.js Added 'del' and 'ins' to allowed HTML tags for displaying strikethrough and insertion formatting
frontend/src/pages/CollaborationReportForm/index.js Added conditional form update logic using shouldUpdateFormData helper
frontend/src/pages/CollaborationReportForm/tests/index.js Updated test to query for rich text editor element using CSS class selector

@Andrew565 Andrew565 added this pull request to the merge queue Feb 13, 2026
Merged via the queue into main with commit 83599dd Feb 13, 2026
14 checks passed
@Andrew565 Andrew565 deleted the 4779-cr-act-desc-wysiwyg branch February 13, 2026 16:01
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