Allow for Pre and Post Actions. about buttons

That's interesting - thanks. I haven't put this into Buttons already as I'm concerned as to what would happen if you need to have a pre or post action on that button that already has a pre or post action! That would then require a prePre / postPost action - gulp...

My inclination is currently to have the extended button define its own action and have that call the action function of the button that it wants to extend - a bit like calling super() in an OO language, but a bit more verbose.

[michaeldjeffrey]

I think if there were some pre/post events thrown around the execution of the action (Akin to Bootstraps events for modals, tooltips and the likes) that might help. The reason I didn't go with that solution is because you don't seem to have many events in the library already, and I didn't want to propose a code solution that is of a completely different style than what we're working with.

I can get behind the idea of making the extended buttons action explicitly call the action. I'm not sure of the best way to get at the actions from inside an action so it still goes through the _resolveExtends.

With the super() way, people who are trying to lazily load an action will still get a race condition where the postAction get called immediately after the action even though the action hasn't actually completed.

The problem with pre/post events is that an async action such as loading an external script would not be complete before the action function itself is then executed. These events might be useful in some cases - when synchronous actions are all that are required, but I'm not sure they would help in this case.

[michaeldjeffrey]

ahh, true. I think this could work with Promises, but that's a lot of work for one case. :/

You can now do post actions on a button with the buttons-action event - 696e161. I think that's probably as far as I want to take it at the moment. I'm not sure how useful a pre event would be, but will consider it in future if there is a solid use case for it. For now going to close.