Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Return cards and atoms from high-level insert methods #428

Merged
merged 1 commit into from
Jul 5, 2016

Conversation

joshfrench
Copy link
Contributor

It can be useful to get a handle on newly inserted cards/atoms. Currently the only way to do that is to drop to the lower-level postEditor methods. This keeps the convenience of high-level insert behavior (range detection, nested/blank sections, &c) while enabling additional operations on the inserted element.

Useful for getting a handle on inserted elements without having to drop
to lower level postEditor methods.
@mixonic
Copy link
Contributor

mixonic commented Jul 1, 2016

Seems reasonable, FF failed on Sauce and I've kicked it

@bantic
Copy link
Collaborator

bantic commented Jul 2, 2016

Sauce still finicky, I kicked it again

@bantic
Copy link
Collaborator

bantic commented Jul 5, 2016

investigating why tests in FF @ Sauce failed. Runs fine locally.

@bantic
Copy link
Collaborator

bantic commented Jul 5, 2016

There's an issue with FF at Sauce where it doesn't properly focus the editor element in some situations. I can't reproduce it locally, but adding a line to force a selection in the insertAtom test fixes this problem at Sauce. I'll add that manually after a merge.

thanks @joshfrench !

@bantic bantic merged commit 4f789d8 into bustle:master Jul 5, 2016
bantic added a commit that referenced this pull request Jul 5, 2016
@bantic
Copy link
Collaborator

bantic commented Jul 8, 2016

released in v0.9.8

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