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

feat(representation): add segment slider and pull to refresh #14

Merged
merged 1 commit into from
Apr 19, 2016

Conversation

thielpa
Copy link
Collaborator

@thielpa thielpa commented Apr 15, 2016

add segment slider and pull to refresh

@thielpa thielpa force-pushed the feat/ion-slider-ion-refresher branch from 018e9bf to 7a26a13 Compare April 15, 2016 20:38
},
];
});
if (refresher !== undefined) refresher.complete();
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can be simplified to if (refresher)

@thielpa thielpa force-pushed the feat/ion-slider-ion-refresher branch from 7a26a13 to 1484b56 Compare April 16, 2016 17:47
@thielpa
Copy link
Collaborator Author

thielpa commented Apr 16, 2016

@devversion pr updated

@devversion
Copy link
Owner

devversion commented Apr 16, 2016

@Lordnoname Is pull to refresh still bugging?

@thielpa
Copy link
Collaborator Author

thielpa commented Apr 16, 2016

This is an issue from ionic 2. I can't fix that

@devversion
Copy link
Owner

Okay, but then we should remove that, if it's still bugging.

@thielpa thielpa force-pushed the feat/ion-slider-ion-refresher branch from 1484b56 to 7a5660e Compare April 16, 2016 19:09
@thielpa
Copy link
Collaborator Author

thielpa commented Apr 16, 2016

@devversion I removed it.^
BTW: its not the pull to refresh thats bugging, but the ion-spinner

@thielpa thielpa force-pushed the feat/ion-slider-ion-refresher branch from 7a5660e to 93182f0 Compare April 17, 2016 12:04
@thielpa
Copy link
Collaborator Author

thielpa commented Apr 17, 2016

@devversion resolved conflicts. Ready for merge?

@thielpa thielpa force-pushed the feat/ion-slider-ion-refresher branch from 93182f0 to 77e942b Compare April 19, 2016 11:00
update(): refresh each slide individually
update(): remove buggy ion-spinner
update(): improve slider and add loading window
@thielpa thielpa force-pushed the feat/ion-slider-ion-refresher branch from 77e942b to 9980611 Compare April 19, 2016 16:46
@thielpa
Copy link
Collaborator Author

thielpa commented Apr 19, 2016

@devversion Depends on #25 (angular-beta.4)

@pgschwendtner
Copy link

Thanks for your reply, we will analyze your comment as soon as possible.


[hidden] {
display: none;
}
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Attribute Selectors are slow IE11

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well.....good joke :P

@devversion devversion merged commit 6363968 into master Apr 19, 2016
@devversion devversion deleted the feat/ion-slider-ion-refresher branch April 19, 2016 17:45
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