feat: UTRP-87: add feedback to more areas and update feedback form style#761
Conversation
IsaDavRod
left a comment
There was a problem hiding this comment.
Will leave pics and comments in the discord
There was a problem hiding this comment.
Pull request overview
Adds an in-page feedback dialog and updates the feedback entry points/styles across the extension UI to match the new design.
Changes:
- Introduces a reusable
useReportIssueDialoghook to open the feedback form in a centered dialog. - Replaces “open report window” links with in-app feedback buttons in the Calendar, popup header, and course catalog popup.
- Restyles
ReportIssueMainto the new form layout and adds close behavior for dialog embedding.
Reviewed changes
Copilot reviewed 7 out of 9 changed files in this pull request and generated 6 comments.
Show a summary per file
| File | Description |
|---|---|
| src/views/hooks/useReportIssueDialog.tsx | New hook to open the feedback form via the dialog system. |
| src/views/components/injected/CourseCatalogInjectedPopup/CourseCatalogInjectedPopup.tsx | Adds a “Send us Feedback!” CTA in the injected course popup. |
| src/views/components/calendar/TeamLinks.tsx | Routes the report link to the new in-page feedback dialog. |
| src/views/components/calendar/DiningAppPromo.tsx | Minor icon styling update (color handling). |
| src/views/components/calendar/Calendar.tsx | Adds new feedback/what’s-new buttons and improves UT Dining promo dismissal race handling. |
| src/views/components/ReportIssueMain.tsx | Redesigns the feedback form UI and supports closing when embedded in a dialog. |
| src/views/components/PopupMain.tsx | Adds a feedback icon button in the popup header and tweaks header button sizing. |
| src/assets/flag.svg | New flag icon asset for feedback affordances. |
| src/assets/exclamation-mark.svg | New exclamation icon asset for “What’s New” affordance. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
a8d88f6 to
968be27
Compare
9c6f7ab to
4d75fad
Compare
|
@DereC4 @Razboy20 in the future I encourage you to have fellows handle merge conflicts and quality issues on their own. Although, I understand the need for quick work on this due to the upcoming deadline. This is a discussion for after the release: To avoid this next semester, we should consider adopting a feature freeze a week or two before the submission deadline. With all these changes at once, it's increasing the likelihood of bugs around reg time. (I've broken an extension for 1M+ people before, it is not fun!) |
I 100% agree—I would much rather give constructive criticism and give a full and proper review; I don't like pushing commits on others' PRs personally! In this specific case however, we really need to push this update out ASAP, so as much as I hate to feel like I'm treading on the work and learning of @vincent-situ in this PR, I hope we can be happy at having a release to show before this registration period starts :) |
6649d9b to
7d82d10
Compare
7d82d10 to
82c315d
Compare
Razboy20
left a comment
There was a problem hiding this comment.
@Razboy20 partially reviewed 5 files.
Reviewable status: 1 of 17 files reviewed, 1 unresolved discussion (waiting on doprz and vincent-situ).
Razboy20
left a comment
There was a problem hiding this comment.
@Razboy20 reviewed 16 files and all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on doprz and vincent-situ).
Razboy20
left a comment
There was a problem hiding this comment.
@Razboy20 resolved 1 discussion and dismissed @doprz from a discussion.
Reviewable status:complete! all files reviewed, all discussions resolved (waiting on vincent-situ).






Old Design:
New Design:

This change is