Skip to content
This repository has been archived by the owner on Mar 21, 2018. It is now read-only.

Need a ratings half-star glyph #864

Closed
jsantell opened this issue Jul 22, 2016 · 7 comments
Closed

Need a ratings half-star glyph #864

jsantell opened this issue Jul 22, 2016 · 7 comments
Assignees

Comments

@jsantell
Copy link
Contributor

In the PR #863, I used the bookmark glyphs (but blue) for full and empty stars for ratings -- we need a half star for half ratings, currently stored in app/ui/shared/assets/glyph-ratings-half-16.svg (right now it's just a copy of the hollow star).

@victorporof
Copy link
Contributor

Can't we just use a css mask or clip the asset we already have?

@jsantell
Copy link
Contributor Author

I imagined still having the outline of the star, which not sure we can do with CSS

@victorporof
Copy link
Contributor

You're right.

@phlsa
Copy link

phlsa commented Aug 1, 2016

All the stars!

Stars.zip

@jsantell
Copy link
Contributor Author

jsantell commented Aug 1, 2016

@phlsa thanks phil! So I was using the bookmarks star before (blue), and these are yellow, but then we're missing an empty (outline) star -- should we not show empty stars in this case, and just have the two yellow variants of full and half?
screen shot 2016-08-01 at 12 20 40 pm

@phlsa
Copy link

phlsa commented Aug 2, 2016

My bad, sorry. I simply forgot to upload the empty star.
Here it is!

Empty-Star.zip

jsantell added a commit to jsantell/tofino that referenced this issue Aug 2, 2016
@jsantell
Copy link
Contributor Author

jsantell commented Aug 2, 2016

Thanks!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants