Skip to content
This repository has been archived by the owner on Oct 1, 2024. It is now read-only.

[BUG] Star rating colours can be confusing #134

Closed
1 task done
adityaraute opened this issue Apr 23, 2024 · 3 comments · Fixed by #156
Closed
1 task done

[BUG] Star rating colours can be confusing #134

adityaraute opened this issue Apr 23, 2024 · 3 comments · Fixed by #156
Assignees
Labels
🛠 goal: fix 🏁 status: ready for dev You can asked for this issue to be assigned (if not already assigned)

Comments

@adityaraute
Copy link
Member

Has this bug been raised before?

  • I have checked "open" AND "closed" issues and this is not a duplicate

Where did you find this bug?

production

Description

Upon trying to rate a repo, I noticed that the stars were coloured green, and the last star was actually coloured blue. If I changed the rating to 4 stars, the first 3 stars appear green, 4th one appears blue, and the 5th one appears black.
This leads to some confusion.

I suggest we colour all selected stars as golden #FFD700, and unselected stars can remain black or grey.
Then there will also not be any need to add a number in parantheses after the stars. (See the screenshot for clarity)

Steps to Reproduce

If you know the steps, follow the below format and provide steps to reproduce

For example:

  1. Go to Add Rating page
  2. Play around with the star ratings
  3. Notice that the UX is not the most straightforward

Screenshots

image

Do you want to work on this issue?

No

If "yes" to above, please explain how you would technically implement this

NA

@adityaraute adityaraute added 🚦 status: awaiting triage Waiting for maintainers to verify (please do not start work on this yet) 🛠 goal: fix labels Apr 23, 2024
@Bashamega
Copy link
Member

@eddiejaoude Hello:)
I would like to work on this issue

@eddiejaoude
Copy link
Member

Thanks for raising @adityaraute and thank you @Bashamega, i will assign it to you 👍

@eddiejaoude eddiejaoude added 🏁 status: ready for dev You can asked for this issue to be assigned (if not already assigned) and removed 🚦 status: awaiting triage Waiting for maintainers to verify (please do not start work on this yet) labels Aug 11, 2024
@Bashamega Bashamega mentioned this issue Aug 11, 2024
6 tasks
Copy link
Contributor

github-actions bot commented Oct 1, 2024

Thank you all for contributing to RepoRater! Please take a moment to rate this repo's DX on EddieHub's RepoRater and give the repo a star ⭐

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
🛠 goal: fix 🏁 status: ready for dev You can asked for this issue to be assigned (if not already assigned)
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants