angular-sticky-things icon indicating copy to clipboard operation
angular-sticky-things copied to clipboard

Odd behavior inside the library.

Open EliezerB123 opened this issue 6 years ago • 0 comments

Hi! I was looking over the code as reference for writing a library for my private work, and I noticed something odd.

sticky-thing.directive.ts

It seems that this.sticky is always false if there's no scrollContainer.

Which means that if there's no container, then determineElementOffsets() won't remove sticky before doing its math.

Is this intended behavior?

Edit: Oh. I see. this.sticky controls whether it uses position:sticky or position:fixed. Okay. So it is working as intended....... Except why would you need to turn off position:sticky, but not position:fixed before running determineElementOffsets()?

EliezerB123 avatar Nov 18 '19 11:11 EliezerB123