Update Bootswatch and Bootstrap dependencies

symfony / demo

Symfony Demo Application

https://symfony.com/

MIT License

2.46k stars 1.61k forks source link

Update Bootswatch and Bootstrap dependencies #1029

Closed javiereguiluz closed 1 year ago

javiereguiluz commented 5 years ago

We use bootswatch 3. and Bootstrap 3..

We could upgrade bootswatch to 4., which in turns requires upgrading Bootstrap to 4.. After the upgrade, the current theme (flatly) and the current styles must remain the same.

Any volunteer to help us with this? Thanks!

abdounikarim commented 5 years ago

Hello @javiereguiluz , i want to help you but i don't know where to start. I think i need to do this but i'm not sure :

Run a yarn install locally
Update the dependencies on the assets directory
Run a yarn build
Make a commit and create a pull request.

Am I wrong ? Thanks you 😉

stof commented 5 years ago

you are missing "update all templates to migrate Bootstrap from 3 to 4"

abdounikarim commented 5 years ago

thanks you for your answer @stof . i'm going to try to solve this issue.

javiereguiluz commented 5 years ago

@abdounikarim thanks for taking a look at this issue!

You first need to update the bootswatch and bootstrap-sass dependencies in package.json to use the latest ones (4..) and then you can run yarn install.

Then you can run yarn encore dev --watch and that's where you'll face some issues because some files have changed their path or name, etc.

When all issues are solved and assets build correctly, you need to check the app to see if it still looks OK or there are some minor html/css issues to fix. Thanks!

javiereguiluz commented 5 years ago

As @abdounikarim told me via Slack, there's a minor problem: the datepicker we use is only for Bootstrap 3. Apparently, the same creator maintains a new project at https://github.com/tempusdominus/bootstrap-4 which is compatible with Bootstrap 4. So, we're going to try that datepicker, but we'd like to listen to the community feedback about this. Thanks.

disjfa commented 4 years ago

I'd like to take a look when this is merged. I think more things come of this. Also i see that the variables that are in bootstrap are misused. Like to check that!

Also, should the build folder not be in the git repro? Or am i asking questions that are answered many times.

skmedix commented 4 years ago

Hi, can I take off this issue? Looks like #1030 is not going anywhere, and already have a lot of conflicts. If so, I can post a PR next week with upgraded all front dependencies and templates. I already have done some work on it, just let me know if I should continue.

King regards,

javiereguiluz commented 4 years ago

@skmedix No, please. It's true that the existing PR has some conflicts, but I'll try to fix those myself. That PR has been around for long time, so it's time to merge it (but I need to find some time for it first). Thanks for your patience!

gabiudrescu commented 4 years ago

@javiereguiluz can we help in some way?

javiereguiluz commented 4 years ago

@gabiudrescu yes! What we need to do is to compare every page in the old design and the new design. They don't have to be perfectly equal ... but they must be as similar as possible. When I checked this PR, I notice lots of significant differences and we'd like to minimize those.

So, the job to do is:

1) Compare every page before/after 2) Made the needed CSS changes/additions/removals 3) Submit those changes to this PR

Thanks!

phpfour commented 4 years ago

Hi @javiereguiluz

Is someone working with the correction of PR #1030 ? If not, I can help out.

Thanks!

javiereguiluz commented 1 year ago

Closing as fixed in #1412.