Skip to content

Fix(importer): changes to handling of data watch #1826

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

Merged
merged 1 commit into from
Oct 15, 2014

Conversation

PaulL1
Copy link
Contributor

@PaulL1 PaulL1 commented Oct 15, 2014

Add a dataChangeCallback capability to grid, which is called
by the data watch function. Make use of this for importer to
know when the data change has occurred. Refer #1819 for the
concept here.

Fix rowEdit so that error rows aren't duplicate entered into
rowEditRowsDirty if they error twice.

Update tutorial numbering - newer tutorials have conflicted with
older.

Change importer to require an explicit callback to be provided
by the user - change the data from userspace rather than in the
grid itself.

Fix grid menu so that it displays correctly on Chrome, it had
started only partially rendering. Fixed by changing position from
absolute to fixed.

Add a dataChangeCallback capability to grid, which is called
by the data watch function.  Make use of this for importer to
know when the data change has occurred.  Refer angular-ui#1819 for the
concept here.

Fix rowEdit so that error rows aren't duplicate entered into
rowEditRowsDirty if they error twice.

Update tutorial numbering - newer tutorials have conflicted with
older.

Change importer to require an explicit callback to be provided
by the user - change the data from userspace rather than in the
grid itself.

Fix grid menu so that it displays correctly on Chrome, it had
started only partially rendering.  Fixed by changing position from
absolute to fixed.
PaulL1 added a commit that referenced this pull request Oct 15, 2014
Fix(importer): changes to handling of data watch
@PaulL1 PaulL1 merged commit 91de254 into angular-ui:master Oct 15, 2014
@c0bra c0bra removed the in progress label Oct 15, 2014
@PaulL1 PaulL1 deleted the importer_updates branch October 15, 2014 09:32
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