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(affix) #339

Closed
wants to merge 8 commits into from
Closed

feat(affix) #339

wants to merge 8 commits into from

Conversation

Nolanus
Copy link
Contributor

@Nolanus Nolanus commented Mar 29, 2016

The bootstrap affix component as angular2 directive in typescript. Appreciating your feedback!

@valorkin
Copy link
Member

This is Fckn huge! Looks really good, some things could be simplified a bit, let me get to PC.
I will check it locally

One note, now you could have difficulties with squashing to one commit because of merge commit at the end, so try to use git pull rebase when possible, it will make life easier for you :)

@Nolanus
Copy link
Contributor Author

Nolanus commented Apr 7, 2016

Did you find time to investigate it? Any feedback? I tried staying pretty close to the bootstrap javascript implementation, as I think they figured out the best approach to get it working in as many configurations as possible.
Let me know when I shall create a new Pull Request with everything squashed into a single commit.

@koodikindral
Copy link

What's the status of this PR? :) Would be great feature

@Flood
Copy link

Flood commented Nov 14, 2016

+1, would be an awesome improvement!

@pratheekhegde
Copy link

Waiting for this. +1

@rg1220
Copy link

rg1220 commented Dec 16, 2016

+1 Would love to see this added.

@yotixify
Copy link

Any update on this? Would be nice to have!

}

private checkPosition():void {
let elemPos = positionService.position(this.el.nativeElement);

This comment was marked as off-topic.

let scrollTop:number = this.body.scrollTop; // current scroll position in pixels from top
let targetHeight:number = this.window.innerHeight; // Height of the window / viewport area

if (offsetTop !== null && this.status === AffixStatus.AFFIX_TOP) {

This comment was marked as off-topic.

This comment was marked as off-topic.

This comment was marked as off-topic.

@kush-varma
Copy link

Any update on this? Do we planning to add affix to ngx-bootstrap anytime soon?

@Nexeuz
Copy link

Nexeuz commented Sep 5, 2017

this already implemented or still not?

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.

None yet