Skip to content

Pass through the "item" parameter to registerAceCommand callbacks.#2929

Merged
lpagliari merged 1 commit intoether:developfrom
xavidotron:develop
Dec 13, 2016
Merged

Pass through the "item" parameter to registerAceCommand callbacks.#2929
lpagliari merged 1 commit intoether:developfrom
xavidotron:develop

Conversation

@xavidotron
Copy link
Copy Markdown
Contributor

Passing through the item parameter makes it easier to have one registerAceCommand callback that handles multiple toolbar items in a plugin and makes the interface more similar between registerCommand() and registerAceCommand().

@xavidotron xavidotron force-pushed the develop branch 2 times, most recently from 6f6be69 to 56b851a Compare May 7, 2016 01:55
@xavidotron
Copy link
Copy Markdown
Contributor Author

Any chance of getting this pull request looked at?

@Gared
Copy link
Copy Markdown
Member

Gared commented Dec 12, 2016

I think this should be finde. What do you think @lpagliari

@lpagliari lpagliari merged commit 01d23b1 into ether:develop Dec 13, 2016
@lpagliari
Copy link
Copy Markdown
Contributor

Yep, looks good to me. Sorry, @xavidotron , for taking so long.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants