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

Added roles to player tables #2290

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Added roles to player tables #2290

wants to merge 3 commits into from

Conversation

ahi-el
Copy link
Contributor

@ahi-el ahi-el commented Nov 15, 2019

Noticed that we're getting each player's role in the data, so I've added it to the table

image

here's how it looks like, might want to add some padding/margin

image

solves(?) this issue odota/core#1844

note that this is also shown in all the other match tabs (laning/casts/etc)

@Argn0
Copy link
Contributor

Argn0 commented Nov 15, 2019

I don't like how this makes the tables taller. Maybe we should use the lane icons instead and put them in their own column?

Also we would need support/core icons.

@ahi-el
Copy link
Contributor Author

ahi-el commented Nov 16, 2019

I don't like how this makes the tables taller. Maybe we should use the lane icons instead and put them in their own column?

Also we would need support/core icons.

some of the rows are actually the same length.

image

Any hero with a backpack item is the same height as before

though this is only for the 'overview' tab, a lot of other tables are indeed too tall now

image

Maybe we should use the lane icons instead and put them in their own column?

I kind of like how you can see a hero's role in each tab, that way you can have more information for wards/tower damage/etc depending on role.
We can use the lane icons and have a tooltip when hovering on them with the role description, what do you think?

@Argn0
Copy link
Contributor

Argn0 commented Nov 16, 2019

I think there is enough space in the overview table to have both icons next to each other in their own separate columns.

If we made the icons a part of the hero-image component and displayed them in all tables then it would look really awkward in some of the tables.

@howardchung
Copy link
Member

Agreed that it looks a bit cramped/busy on its own line. Maybe we can try icons, or possibly inlining it with the rank line, but that might be confusing.

@ahi-el
Copy link
Contributor Author

ahi-el commented Jan 19, 2020

@howardchung do you have any icons in mind?

We can do a simple number (safe core 1, mid 2, off core 3, 4 and 5 for off/safe supports, but that won't account for trilanes / junglers

how about a simple icon for core/supp (or text?) and having that be a tooltip for the more descriptive role?

and of course - do you think this feature is worth it?

@howardchung
Copy link
Member

We have role icons here, e.g. https://github.com/odota/web/blob/master/public/assets/images/dota2/lane_1.svg

That includes off/mid/safe/jungle

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

Successfully merging this pull request may close these issues.

3 participants