Skip to content

Teemu/enhancement/637 feedbackcard update and interactive tooltips#639

Open
temez26 wants to merge 9 commits into
devfrom
teemu/enhancement/637-feedbackcard-update-and-interactive-tooltips
Open

Teemu/enhancement/637 feedbackcard update and interactive tooltips#639
temez26 wants to merge 9 commits into
devfrom
teemu/enhancement/637-feedbackcard-update-and-interactive-tooltips

Conversation

@temez26

@temez26 temez26 commented Apr 18, 2026

Copy link
Copy Markdown
Contributor

📄 Pull Request Overview

Closes #637

🔧 Changes Made

  1. FeedbackCard overhaul: Refactored FeedbackCard to use the new FeedbackCardSuccess component for the post-submission view, which displays the selected emoji rating and links to full feedback forms (web & game). Added dynamic, rating-aware input placeholders that change based on the selected emoji.

  2. Interactive emoji tooltips: Updated FeedbackEmoji to render a speech-bubble style rating label (e.g. "Bad", "Meh..", "OK", "Good", "Great") below the selected emoji, with dynamic horizontal positioning. Emoji can now be deselected by clicking again.

  3. New FeedbackCardSuccess component: Created new FeedbackCardSuccess component and its SCSS module to handle the success state UI after feedback submission.

  4. CustomForm.TextareaField added: Extended the CustomForm compound component with a new TextareaField sub-component for more flexible, accessible multi-line text input in forms.

  5. i18n updates: Updated feedbackCard.json localization files with shorter, cleaner sentences and added new keys: rating-label-1 through rating-label-5, input-placeholder-1 through input-placeholder-5, error-submit-failed, href-to-webform, and href-to-gameform.


Checklist Before Submission

  • Functionality: I have tested my code, and it works as expected.
  • JSDoc: I have added or updated JSDoc comments for all relevant code.
  • Debugging: No console.log() or other debugging statements are left.
  • Clean Code: Removed commented-out or unnecessary code.
  • Tests: Added new tests or updated existing ones for the changes made.
  • Documentation: Documentation has been updated (if applicable).

📝 Additional Information

  • Screenshots: Include screenshots of the interactive emoji tooltip (speech bubble label) and the new FeedbackCardSuccess view.
  • Dependencies: No new dependencies added.
  • Known Issues: English text for some i18n keys may still need final confirmation/review.

@Skoivumaki Skoivumaki left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Good work! Unfortunately the original issue might have been a bit misleading, since the given Figma Design example is meant to be toggled via feedback button. (unless im mistaken @Rutjake ?) However the only differences between toggle vs footer applied feedback component really is the border😄

Todo:

  • Remove borders from component if its implemented inside footer.
  • Adjust components layout so it looks more clean. (see example)
  • ratingLabels go off-center on small devices, fix by detaching the arrow and center it to emoji... or figure something else. (see example)
  • Remove inline styling unless its setting CSS via variables.

Examples:
Feedback layout adjusted by fit-content and div filling the extra space, since the ratingLabels are absolute and don't contribute to flex:
https://github.com/user-attachments/assets/add71c85-c841-436b-90cc-35e1857d3131

ratingLabel off-center:

Image

@Rutjake

Rutjake commented Apr 20, 2026

Copy link
Copy Markdown
Contributor

Good work! Unfortunately the original issue might have been a bit misleading, since the given Figma Design example is meant to be toggled via feedback button. (unless im mistaken @Rutjake ?)

You’re absolutely right. The original issue was a bit misleading, and I apologize for the lack of clarity in the initial description.
The goal is to include both elements for now. This is because the new footer design is still being finalized by the design team.

@temez26

temez26 commented Apr 21, 2026

Copy link
Copy Markdown
Contributor Author

Alright, I will look into it at some point.

@temez26

temez26 commented Jun 26, 2026

Copy link
Copy Markdown
Contributor Author

So now I’ve added the fixes to the form, but the new footer doesn’t even have the form imported. Is this how it’s supposed to work, that the form is only accessible from the side button?

@temez26
temez26 requested a review from Skoivumaki June 26, 2026 09:27
@Skoivumaki

Copy link
Copy Markdown
Member

So now I’ve added the fixes to the form, but the new footer doesn’t even have the form imported. Is this how it’s supposed to work, that the form is only accessible from the side button?

According to latest Figma design, the feedback form is removed from footer and only accessible via side button. However your branch is quite behind dev branch and still has feedback form in footer. I would suggest you rebase from dev and then commit your changes related ti feedback component. (this also resolves the merge conflicts)

@temez26
temez26 force-pushed the teemu/enhancement/637-feedbackcard-update-and-interactive-tooltips branch from 0a6f039 to 51cf6c3 Compare July 6, 2026 11:10
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