Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

V1 Charts Page #19

Closed
benparsons opened this issue Feb 16, 2017 · 19 comments
Closed

V1 Charts Page #19

benparsons opened this issue Feb 16, 2017 · 19 comments
Assignees
Milestone

Comments

@benparsons
Copy link
Contributor

No description provided.

@benparsons benparsons added this to the v1 milestone Feb 16, 2017
@pixlwave
Copy link
Contributor

Yeah I think it would be nice to have the vendors ranked. Either most liked and most hated. Or just likes minus hates sorted.

@benparsons
Copy link
Contributor Author

OK, I'll make a page which just shows that. Can you have a thought on layout

@benparsons benparsons self-assigned this Feb 16, 2017
@pixlwave
Copy link
Contributor

Can we do a bar chart using an emoji?

@benparsons
Copy link
Contributor Author

as in, count (like/hate/like-hate) on the y-axis and the emojis on the x axis? could be fun

@pixlwave
Copy link
Contributor

pixlwave commented Feb 16, 2017

Something like this:
chart
(F*ck knows how)

@benparsons
Copy link
Contributor Author

benparsons commented Feb 22, 2017

I love the chart! Can be implemented by varying the width of a container
(eg "<div with='xx'>"), then using background-image and background-repeat: repeat-x CSS. Glyph score is now available, can use that?

@benparsons
Copy link
Contributor Author

So, this is done... somewhat. The basic output is there but it will take more styling to make a really good output. @pixlwave

screen shot 2017-02-23 at 19 43 21

@benparsons
Copy link
Contributor Author

Improvements made. The use of negative glyph scores means we can have something more like this
screen shot 2017-03-07 at 23 30 36

@benparsons benparsons changed the title Do we need a "charts" page for a v1 release? V1 Charts Page Mar 8, 2017
@benparsons
Copy link
Contributor Author

Idea: this page should redirect to the same emoji voting page if there is no data. That way it matters less if #16 is still high.

@pixlwave
Copy link
Contributor

pixlwave commented Mar 9, 2017

Redirect automatically, or display a message saying "Currently there's no voting data for this Emoji, click here to cast the first vote!"

@pixlwave
Copy link
Contributor

pixlwave commented Mar 9, 2017

Also, this graph doesn't work in Firefox, just shows vendor names with no images either side.

@benparsons
Copy link
Contributor Author

benparsons commented Mar 9, 2017 via email

@benparsons
Copy link
Contributor Author

And good question on the the redirect, any idea?

@pixlwave
Copy link
Contributor

pixlwave commented Mar 9, 2017

I'd show the message, otherwise it just appears broken. Could redirect automatically after 10 seconds too.

@pixlwave
Copy link
Contributor

pixlwave commented Mar 9, 2017

Ok, i refreshed the page that wasn't working (2 hours ago) and it's fine now. Just tried another one which I know had no results before I voted, and it didn't show anything. Presume this is just the data being cached (or updating on a schedule).

@benparsons
Copy link
Contributor Author

benparsons commented Mar 9, 2017 via email

@benparsons
Copy link
Contributor Author

Made a little cleanup and added a message when there are no votes.
Made a link from the results page to the vote page.

When we are happy with this page, let's link to it from the vote page, and that can be v1.0!

@pixlwave
Copy link
Contributor

pixlwave commented Mar 10, 2017

Yep agree, only thing we need is for the table to size down when on mobile, then I'm game for v1.0.

Edit: How'd we do that??

@benparsons
Copy link
Contributor Author

Yeah it's more complex. https://github.com/emojirank/web/blob/master/results.js#L76 this is where the width of the bars is set. What we need is that regardless of the width of the page, a score of 100 is exactly the max width of one of the columns in that table.

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

No branches or pull requests

2 participants