822 - Add "Show tanks" option
Hi there ! I'm using latest version 6.2.0.1754-beta (as recommended) and while you made this change :
"Version 6.0.3.1714-beta - There are no longer separate "w/ Pets" layouts; use the "Show Pets" option on the Layout tab to add pets to any of the built-in layouts."
there is no more "w/ Tanks layouts" nor "Show Tanks" option on the layout tab. Do you plan to bring it back ?
| User | When | Change |
|---|---|---|
| Phanx | Jul 29, 2016 at 11:02 UTC | Changed name from Add promised "Show tanks" option to Add "Show tanks" option |
| Phanx | Nov 03, 2015 at 12:41 UTC | Changed status from New to Accepted Changed name from No "Show tanks" option to Add promised "Show tanks" option |
| Daxterminator | Oct 27, 2015 at 10:00 UTC | Create |
- 7 comments
- 7 comments
Facts
- Last updated
- Jul 29, 2016
- Reported
- Oct 27, 2015
- Status
- Accepted - Problem reproduced / Need acknowledged.
- Type
- Enhancement - A change which is intended to better the project in some way
- Priority
- Medium - Normal priority.
- Votes
- 1
- Reply
- #7
Phanx Aug 25, 2016 at 09:00 UTC - 0 likesIt could be done as a plugin, but based on your given scenario, I think you may be misunderstanding some terminology and/or not actually using the layout you think you're using.
If you're using the "By Role" layout, then which subgroups the tanks are in doesn't matter. As long as nobody is joining or leaving the group, the "By Role" layout shouldn't change at all when players are moved between subgroups. The "By Role" layout only looks at the raid as a whole. It does not care about subgroups at all.
Also remember that the "By Role" layout looks at the Tank/Healer/Damager roles, not the Main Tank / Main Assist roles. Players who don't have a Tank/Healer/Damager role assigned are lumped in with the Damager (DPS) players.
If you are seeing players move around in the layout when they're moved around in the raid, then either:
a) you are not actually using the "By Role" layout (you are probably using the "By Group" layout), or
b) the players who moved had their assigned role changed, or
c) something is wrong.
If you are 100% sure you are using "By Role" and nobody's role is being changed, but you're still seeing tanks mixed in with non-tanks, please submit a separate bug ticket, because that is a bug, and has nothing to do with this feature request.
- Reply
- #6
Malisent Aug 24, 2016 at 11:34 UTC - 0 likesAhh thanks for the update! I don't find grouping by role particularly useful as a lot of times the RL will set-up groups based on where he wants people to stand in a room and having a separate group to quickly find tanks when they are getting moved around a lot is really helpful.
Could something like this be done as a plug-in to Grid or would it have to be done as part of the base add-on (I was just wondering if I could find someone who would take a little cash to write an add-on)?
- Reply
- #5
stassart Jul 29, 2016 at 06:57 UTC - 1 likeI am doing my best to support Grid and help Phanx out. I am still playing the game although my raid died until the next expansion so I am not actively raiding at the moment (other than LFR and pugs for testing). However, my current priority is fixing tickets for bugs with Legion. The adding back a manual layout option feature request has priority over this one, IMO, as there are many more people asking for it.
I spent a lot of hours last week fighting with the Curse packager for Grid also which could have been time spent fixing issues.
- Reply
- #4
Phanx Jul 29, 2016 at 02:33 UTC - 1 likeThere is no ETA on when this feature will be implemented. I haven't had an active game subscription in months, and while I'm willing to spent a bit of time on a trial account fixing patch issues, I don't have a lot (read: any) of motivation to spend time adding new features at this point, and unfortunately nothing I've read about Legion has generated any excitement for me, so I don't know when or even if I'll ever return to actively playing the game.
However, a "Show tanks" option wouldn't show targets, as Grid doesn't support that. It would just show the tanks as a separate row at the top. Realistically this would only be useful if you were using the "By Group" layout -- if you use the "By Role" layout, then tanks are already shown first.
If anyone has programming experience and wants to take a stab at submitting a patch, fork the repo on GitHub, make your changes, test them, and open a pull request. The existing code for the "Show pets" option should provide a decent guide for what's required to dynamically add a new group header independent of the layout's defined header(s).
- Reply
- #3
shandito Jul 27, 2016 at 21:11 UTC - 0 likesHi Phanx, I love your work, all of them. Are you talking about same behavior from default raid frame? I mean, tanks apart with their respectives target? if so, plz plz plz plz!
- Reply
- #2
Malisent Jul 19, 2016 at 19:53 UTC - 0 likesI really hope this makes it into the Legion version - have been really looking forward to this addition!
- Reply
- #1
Phanx Nov 03, 2015 at 12:40 UTC - 1 likeYes, it's still planned. I just haven't had time to work on it.
In the meantime, though, you can use one or both of these options:
use the 'By Role' layout to show tanks first, then healers, then DPS.
use the 'Role' status to mark tanks (and/or healers, and/or DPS; you can choose which roles are marked).