Skip to content
This repository has been archived by the owner on Sep 11, 2022. It is now read-only.

Will only work as intended on elements with an ID #21

Closed
gingerbeardman opened this issue Aug 30, 2012 · 2 comments · May be fixed by #283
Closed

Will only work as intended on elements with an ID #21

gingerbeardman opened this issue Aug 30, 2012 · 2 comments · May be fixed by #283

Comments

@gingerbeardman
Copy link

Will only work as intended on elements with an ID, so:

$("h1, h2").sticky('destroy');

will result in all matching elements receiving a wrapper #undefined-sticky-wrapper which will not be unique.

not sure of the best way around this. maybe throw a warning? or just say in the docs that you must use ID'd elements?
@garand
Copy link
Owner

garand commented Nov 16, 2012

Ah yes, good point. I shall make note of that, or just support any element in the future.

@garand garand closed this as completed Nov 16, 2012
@aastudio
Copy link

aastudio commented Jun 8, 2015

#undefined-sticky-wrapper still appear

@bmalets bmalets mentioned this issue Jan 9, 2019
Open
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Made sticky wrapper Id unique on page bmalets/sticky
3 participants
@gingerbeardman @garand @aastudio