-
Notifications
You must be signed in to change notification settings - Fork 819
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
Road to v1 #347
Comments
todo: define which browser are considered supported and mention them in the README |
I would hate to see framework specific parts leaking backwards into the core. I think its better to keep the core package agnostic, and then create framework specific packages. |
@fuzzzerd there's no plan to change to the core library. I'm thinking I'll just completely drop the jquery plugin and angular module altogether |
I'm somewhat confused on whether it's possible to refactor the core library to such that framework integrations wouldn't have to copy-paste the whole scroll detection business logic anymore? |
@lkraav that would be the ideal situation, yes. Then people could just create thin wrappers around the core logic. I've made a load of perf and big fixes recently that won't make it into react etc ports because the business logic is copy and pasted. This is a shame. I think by the time v1 is done we will be in a better position to explore this route. Though if you'd like to try it now, please go ahead and report back where it falls short :) |
@WickyNilliams Any further progress on this? |
Thought I'd gather a rough plan, and list of possible changes, for v1:
new
up an instance and then callinit()
.headroom(elem, options)
should doelem
?Headroom.cutsTheMustard
toHeadroom.isSupported
(rename cutsTheMustard -> isSupported #351)I've created a
v1
branch which is where i'll gather all these changes. This allows me to work on breaking changes in isolation, and still be able to publish any bugfixes onmaster
.If anybody else has any other suggestions, please let me know.
The text was updated successfully, but these errors were encountered: