-
Notifications
You must be signed in to change notification settings - Fork 3
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
Remove or update Bootstrap and jQuery #23
Comments
@c-w would you mind providing more details about your expectations? I would be glad to work on the styling of the website. |
The lowest scope here is to update bootstrap and jquery to the most up to date versions and make all necessary compatibility fixes. |
are you okay with replacing bootstrap with tailwind for example ? |
For changes to the design, @nzola would have to review and sign off a proposal. Or if the end result is functionally equivalent, no concerns. |
@c-w @eltazy @magmukendi @mabuelhagag What ae the advantages between bootstrap and tailwind? Because I am a bit concern to change the design which can create more work.? |
Moving from bootstrap 3 to 5 is a major jump in migration. It won't be an easy one as many classes are not supported anymore or have been renamed. This change is as big as changing libraries completely, we're skipping version 4. The UI can remain the same but the structure of the page and the stylesheet will change a lot. |
I think choosing bootstrap over tailwind is a matter of preference. We can still reproduce the same design (to some extend). But I might take some time to implement the change. I can work on that if I am assigned to this task. |
@magmukendi @eltazy will you both work together to implement the change? Let me know so that I can assign the task to both of you @c-w |
@eltazy what is your email address? I would like to invite you to the repository. |
The page depends on outdated versions of Bootstrap and jQuery for fairly simple functionality. Ideally we'd remove the dependencies and implement the functionality in some simple CSS, or at least we should upgrade to the latest versions of the packages.
The text was updated successfully, but these errors were encountered: