Grid

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

You must login to post a comment. Don't have an account? Register to get one!

  • 7 comments
  • Avatar of Phanx Phanx Aug 25, 2016 at 09:00 UTC - 0 likes

    It 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.

  • Avatar of Malisent Malisent Aug 24, 2016 at 11:34 UTC - 0 likes

    Ahh 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)?

  • Avatar of stassart stassart Jul 29, 2016 at 06:57 UTC - 1 like

    I 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.

    Last edited Jul 29, 2016 by stassart
  • Avatar of Phanx Phanx Jul 29, 2016 at 02:33 UTC - 1 like

    There 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).

  • Avatar of shandito shandito Jul 27, 2016 at 21:11 UTC - 0 likes

    Hi 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!

  • Avatar of Malisent Malisent Jul 19, 2016 at 19:53 UTC - 0 likes

    I really hope this makes it into the Legion version - have been really looking forward to this addition!

  • Avatar of Phanx Phanx Nov 03, 2015 at 12:40 UTC - 1 like

    Yes, 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).

  • 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

Reported by

Possible assignees