Add username to reaction hovers

PlaceAVote / pav-web

The placeavote web front end.

0 stars 0 forks source link

Add username to reaction hovers #269

Open dave-hawkins opened 8 years ago

dave-hawkins commented 8 years ago

See video:

https://vid.me/qqRq

How easy is this to implement?

johnboy14 commented 8 years ago

We just need to add extra meta data to the payload on retrieval. Seems like a nice feature. Theres not alot of insight into it without it.

johnboy14 commented 8 years ago

Tool tips only?. No intention to navigate to a users profile from it?

dave-hawkins commented 8 years ago

I think that's probably going to be the next step. I'm wondering about usability issues with a hover and click event though. I also think the first couple of people should be linked to, then once it goes over that number then we have & 8 more.

Eg '(a)Luke Davis(/a), (a)John Ervine(/a) & 8 more'

johnboy14 commented 8 years ago

I think this makes the most sense. I don't see the value in returning them all. I'm wondering @atostudios if we can query this on hover or include it in the payload?. I'm worried about performance for all of this stuff. Baking alot of niche UI requirements into our backend APIs