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

Adding callbacks #344

Open
wants to merge 16 commits into
base: master
Choose a base branch
from
Open

Adding callbacks #344

wants to merge 16 commits into from

Conversation

jocluz
Copy link

@jocluz jocluz commented Oct 16, 2015

New callbacks in gridsterItem directive. This callbacks are added inside bindToController object of the directive with the &.

For example:

bindToController:{
......
gridsterItemDragStop: '&'
......
}

This is called inside dragStop function.
From the outside it can be used as:

<li gridster-item gridster-item-drag-stop:'ctrller.doSomethingOnDragStop()'></li>

jespinosa and others added 8 commits September 28, 2015 19:07
New events fired when:
- item moves down
- item floats up
- item resized stop
- item drag stop
Added resizing callback.
Changed resized stop callback call.
Added events for the tests
@TheBosZ
Copy link
Contributor

TheBosZ commented Dec 11, 2015

It looks like you accidentally committed your changes to package.json. Please fix and push.

Remove package.json wrong changes.
Updated package name, changes.
Remove updated version.
@jocluz
Copy link
Author

jocluz commented Dec 11, 2015

Done! Sorry for that. Do I need to create another pull request?
Thank you!

@TheBosZ
Copy link
Contributor

TheBosZ commented Dec 11, 2015

No, this PR is still valid.

angular-gridster has problems when the number of colums/rows are big. I merged coryasilva login from his pull-request and the performance now is great. I will be waiting for this and coryasilva pull request to be merged into the main repo.
Event thrown when the layout changes. This is added to save the boxes layout.
Added event on size changed to adapt the content of the gridster items
Added gridsterItemSizeChanged when gridster item position changed
@danomatic
Copy link
Member

Sorry for the delay on reviewing this. I can't read the diff because it shows the entire file as changed. Did you run some kind of formatter on the JS?

@danomatic
Copy link
Member

The whitespace issues need to be resolved before this can be merged.

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