-
Notifications
You must be signed in to change notification settings - Fork 394
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
base: master
Are you sure you want to change the base?
Conversation
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. 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
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. |
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. |
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. |
@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? |
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 |
Noticed that we're getting each player's role in the data, so I've added it to the table
here's how it looks like, might want to add some padding/margin
solves(?) this issue odota/core#1844
note that this is also shown in all the other match tabs (laning/casts/etc)