51 - broad spell groups (ground mount, flyable, HS)
What is the enhancement in mind? How should it look and feel?
It would be great if it was possible to use item classifications. e.g. Having quite some mounts (past albino drake) it would be a hell lot of work to classify all of these spells with this drop down menu. The same goes for vanity pets.
Another nice thing would be casting Hearth stone.
- 4 comments
- 4 comments
Facts
- Last updated on
- 17 Sep 2009
- Reported on
- 11 Sep 2009
- Status
- Duplicate - This report duplicates an existing ticket.
- Type
- Enhancement - A change which is intended to better the project in some way
- Priority
- Medium - Normal priority.
- #4
rismisner Thu, 17 Sep 2009 20:23:32Thanks for these suggestions.
The basic idea to have arbitrary spell groups is already supported under the Custom Macros feature set. Creating a group of mount messages is possible that way, though it has some limitations I will try to address.
You hinted at several other related ideas which are all great, but its too much to track under a single ticket, given the kind of time I get in tiny bursts to do one little enhancement at a time, so I have spread the ideas around into new tickets 53, 54, and 55.
- #3
rismisner Thu, 17 Sep 2009 20:19:30Capturing the vanity pet idea under the mount idea in ticket 53
- #2
rismisner Thu, 17 Sep 2009 20:18:49"/ss macro mount" is the appropriate solution to creating spell groups, so you're on the right track there. I have not been able to think of any more elegant design than this for creating broad spell groups that share speeches.
I see the limitation that a spell event triggering a /ss macro event doesn't pass down its substitution variable information like its <target> and <spellname> so an enhancement there would help. (I created ticket 54 for this issue: /ss macro should inherit data)
For mounts in particular, that is a very large group, and it's a pain to add all the mount spells to the group. Would be nice to detect mounting as a SpeakinSpell System Event, and add support for <mount>. (I created ticket 53 for this idea: System Event: Mounted, and <mount>)
Hearth stone should be detected as an event already. Since it's an item you use, it should cause a UNIT_SPELLCAST_SENT event, which should show up in the SpeakinSpell interface as "When I cast: Hearthstone". Is that not working?
I added ticket 55 to add <home> for hearthstone home town
- #1
rismisner Thu, 17 Sep 2009 20:03:29essmene posted a related comment on curse.com - capturing a copy-paste of it for reference here:
I am trying to make a mount emote. From RPHelper 2 and similar like:
So i setup a "/ss macro mount" with 'The instructions say "Just add water" and woha it's a <spellname>.'
and in the mount spell i added '/ss macro mount'.
However when executed the <spellname> is not the mount spell, but contains 'macro mount'.