Skip to content

Sharadha taking over for Manvitha phase 3 - feedback form#4586

Open
myeeli wants to merge 17 commits intodevelopmentfrom
Manvitha-Phase-3-Feedback-Form
Open

Sharadha taking over for Manvitha phase 3 - feedback form#4586
myeeli wants to merge 17 commits intodevelopmentfrom
Manvitha-Phase-3-Feedback-Form

Conversation

@myeeli
Copy link
Copy Markdown
Contributor

@myeeli myeeli commented Dec 19, 2025

Description

image image image image

Related PRS (if any):

This frontend PR is related to the development backend PR.

Main changes explained:

How to test:

  1. check into current branch
  2. do npm install and ... to run this PR locally
  3. Clear site data/cache
  4. log as admin user
  5. go to localhost:5173/communityportal/ActivitiesPage click give feedback

Screenshots or videos of changes:

1.mp4

Note:

Include the information the reviewers need to know.

@netlify
Copy link
Copy Markdown

netlify Bot commented Dec 19, 2025

Deploy Preview for highestgoodnetwork-dev ready!

Name Link
🔨 Latest commit 4a6d1a6
🔍 Latest deploy log https://app.netlify.com/projects/highestgoodnetwork-dev/deploys/6945df238440b100081ae7f2
😎 Deploy Preview https://deploy-preview-4586--highestgoodnetwork-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@sonarqubecloud
Copy link
Copy Markdown

Copy link
Copy Markdown

@VijayAnirudh VijayAnirudh left a comment

Choose a reason for hiding this comment

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

2 test cases failed!
Skipped test cases; changes are as expected. I can submit feedback and rate with stars.
image

image

@one-community one-community added the High Priority - Please Review First This is an important PR we'd like to get merged as soon as possible label Jan 12, 2026
Copy link
Copy Markdown

@rohanrastogi311 rohanrastogi311 left a comment

Choose a reason for hiding this comment

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

Hi Manvitha,

Feedback is working as expected. Well done on this implementation (I know it's out of scope but dark mode is not functional at all)

PR 4586 Screenshot PR 4586 Screenshot 2 PR 4586 Screenshot 3

Copy link
Copy Markdown

@Vikas-8055 Vikas-8055 left a comment

Choose a reason for hiding this comment

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

hi manvitha i have reviewed and tested your PR everything works as expected

Screenshot 2026-01-17 at 9 02 13 AM Screenshot 2026-01-17 at 9 02 39 AM Screenshot 2026-01-17 at 9 02 59 AM Screenshot 2026-01-17 at 9 03 16 AM

Copy link
Copy Markdown
Contributor

@Shravan-neelamsetty Shravan-neelamsetty left a comment

Choose a reason for hiding this comment

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

Hi Manvitha, I have tested this PR locally by checking out the Manvitha-Phase-3-Feedback-Form branch and navigating to /communityportal/ActivitiesPage. The feedback functionality works as expected - I clicked the "Give Feedback" button in the top right corner which opened the feedback modal displaying the title "Activity Feedback" with a 5-star rating system, a text area for feedback input, and a character counter showing usage out of 300 characters. I successfully selected 3 stars for the rating, entered feedback text, and clicked "Submit Feedback" which triggered a green success banner displaying "Feedback submitted!" at the top of the modal. The star rating interaction is smooth, the text area accepts input correctly, and the character counter updates in real-time. The modal UI is clean and user-friendly with proper spacing and a close button.

Screenshot 2026-01-17 at 1 20 14 PM Screenshot 2026-01-17 at 1 20 18 PM Screenshot 2026-01-17 at 1 20 23 PM Screenshot 2026-01-17 at 1 23 29 PM Screenshot 2026-01-17 at 1 23 31 PM Screenshot 2026-01-17 at 1 26 40 PM

Copy link
Copy Markdown

@Ganesh112001 Ganesh112001 left a comment

Choose a reason for hiding this comment

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

I tested PR #4586 and verified that the Phase 3 Feedback Form works well with one critical issue. After checking out the Manvitha-Phase-3-Feedback-Form branch and navigating to /communityportal/ActivitiesPage, I clicked "Give Feedback" on an activity and the feedback modal opened correctly. In light mode, the modal displays a well-designed form with a 5-star rating system that provides clear visual feedback when clicking stars, an optional comment field with a 300-character limit and live character counter, and functional Cancel and Submit buttons. Rating validation works correctly by requiring a star selection before submission. However, when I enabled dark mode, the feedback modal has severe visibility issues—text, input fields, stars, and buttons are not visible or have poor contrast against the dark background, making the form unusable in dark mode. The modal needs proper dark mode styling to ensure all elements are visible and readable. In light mode, everything works perfectly including submission and cancel functionality with no console errors. Dark mode support must be added before merge.
Screenshot 2026-01-24 at 1 49 34 PM

Screenshot 2026-01-24 at 1 49 45 PM

Copy link
Copy Markdown

@Anusha-Gali Anusha-Gali left a comment

Choose a reason for hiding this comment

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

Hi Manvitha,

I have reviewed your PR locally and though it works as expected, the title is pretty lite in dark mode.

Image Image Image Image

const modalRef = useRef(null);
const errorRef = useRef(null);

useEffect(() => {
Copy link
Copy Markdown

@naznin07 naznin07 Mar 23, 2026

Choose a reason for hiding this comment

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

Focus trap works though modal doesn't auto-focus on open.

PR # 4586

@SharadhaKasiviswanathan SharadhaKasiviswanathan changed the title Manvitha phase 3 - feedback form Sharadha taking over for Manvitha phase 3 - feedback form Apr 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

High Priority - Please Review First This is an important PR we'd like to get merged as soon as possible

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants