Skip to content

add support for native scroll and ion-side-menu - #32

Open
mixersoft wants to merge 1 commit into
djett41:masterfrom
mixersoft:master
Open

add support for native scroll and ion-side-menu#32
mixersoft wants to merge 1 commit into
djett41:masterfrom
mixersoft:master

Conversation

@mixersoft

Copy link
Copy Markdown

headerHeight was getting a value from the wrong element when using ion-side-menu

headerHeight = activeHeader.offsetHeight;

@MHabashy10

Copy link
Copy Markdown

Hi Everyone,

Thank you so much for this awesome scroll plugin, i'm having this issue with sidemenu, after scrolling back the subheader overlap the content, although my ion-content has the class 'has-subheader'
and i think this fix #26 solves the problem.

any plans to merge this pr soon?

Thank you again for this awesome feature
Regards

@MHabashy10

Copy link
Copy Markdown

anyone?
@djett41 @mixersoft

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