-
Notifications
You must be signed in to change notification settings - Fork 0
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
Comments
Yeah I think it would be nice to have the vendors ranked. Either most liked and most hated. Or just likes minus hates sorted. |
OK, I'll make a page which just shows that. Can you have a thought on layout |
Can we do a bar chart using an emoji? |
as in, count (like/hate/like-hate) on the y-axis and the emojis on the x axis? could be fun |
I love the chart! Can be implemented by varying the width of a container |
So, this is done... somewhat. The basic output is there but it will take more styling to make a really good output. @pixlwave |
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. |
Redirect automatically, or display a message saying "Currently there's no voting data for this Emoji, click here to cast the first vote!" |
Also, this graph doesn't work in Firefox, just shows vendor names with no images either side. |
Works in FF for me - did the images not load? It's just CSS background,
shouldn't be any compatibility probs
…On 9 March 2017 at 12:12, Doug ***@***.***> wrote:
Also, this graph doesn't work in Firefox, just shows vendor names with no
images either side.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
<#19 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/ABJHpylQozb4da84GNXydkpJ9Dty5Gk3ks5rj-zJgaJpZM4MC16->
.
--
M +447866867226 / S ben-parsons / T @benparsons / W www.bpulse.co.uk
|
And good question on the the redirect, any idea? |
I'd show the message, otherwise it just appears broken. Could redirect automatically after 10 seconds too. |
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). |
Could be. Should update on request but maybe there is some cache on
mongolab's side.
…On 9 March 2017 at 14:16, Doug ***@***.***> wrote:
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).
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#19 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/ABJHp-0WV-W_MCGHuFnX4v4XDTA8hHtPks5rkAnTgaJpZM4MC16->
.
--
M +447866867226 / S ben-parsons / T @benparsons / W www.bpulse.co.uk
|
Made a little cleanup and added a message when there are no votes. When we are happy with this page, let's link to it from the vote page, and that can be v1.0! |
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?? |
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. |
No description provided.
The text was updated successfully, but these errors were encountered: