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

Make iText fires object:modified on text change on exit editing #2927

Merged
merged 10 commits into from
May 2, 2016
Merged

Make iText fires object:modified on text change on exit editing #2927

merged 10 commits into from
May 2, 2016

Conversation

asturur
Copy link
Member

@asturur asturur commented May 2, 2016

inssein reading your issue about text not fire object:modified in during editing, i made this draft.

Is a simple change that may add the missing feature.
The point is that object:modified fires normally at the end of transformation, and i felt like that end of editing was a nicer place for it.
Do you have some suggestion

i would add:
deleting the _textBeforeEditingproperty at end of function.
Do not fire if there is no _textBeforeEditingset on exitEditing.

close #2308

@asturur asturur merged commit 17775fa into fabricjs:master May 2, 2016
@asturur asturur deleted the lostbranch branch May 2, 2016 06:33
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.

Modifying text does not dispatch object:modified event
1 participant