-
-
Notifications
You must be signed in to change notification settings - Fork 18.2k
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
[BUG] Move CSS outside of JS #2781
Comments
You mean with all your currently merged changes?
I would say to 80% 🤣
Where we dropped IE10 Support? :) |
Yeah, so that I know what kind of changes I can make.
Nowhere, but I thought you planned to drop it. :) |
At the moment I would create only a patch version and then I planned to start with a minor version (v3.1) with all the open issues in it.
I would drop IE10 with the next major Bootstrap version (v5) |
I'll test later all the changes and publish AdminLTE v3.0.6 (you only added ESLint & StyleLint for this I wouldn't create a minor version 😃 I fixed only small stuff and update the docs), then you can create your next changes for the next minor version and I will do all the open v3.1.0 issues 😄. |
I think we could land some more stuff. But I wouldn't make it a patch release personally. There have been too many changes |
Since we only fixed one small issue and only you have reported it we could directly move on to develop v3.1 and I'll start with the first changes. |
I mark it as a bug, because it reduces performance. It's faster to add/remove a CSS class than setting/getting CSS properties via jQuery.
Now, this should wait a bit after I'm done with my refactoring PRs, should be done in the next 2 days. After that, this issue should be tackled.
This, but most importantly #2719, along with any JS refactoring, should really make things much more performant. :)
PS. Is the next version going to be a minor release? Do you follow semver properly? Because dropping IE 10 support is a breaking change, for example. But on the other hand, we could narrow the supported browsers in .browserslistrc and thus reduce the CSS size too.
The text was updated successfully, but these errors were encountered: