django-rest-framework icon indicating copy to clipboard operation
django-rest-framework copied to clipboard

Update to Bootstrap 4.5.0

Open tomchristie opened this issue 4 years ago • 20 comments

https://blog.getbootstrap.com/2020/05/12/bootstrap-4-5-0/ Related: https://github.com/encode/django-rest-framework/pull/7313

tomchristie avatar May 13 '20 10:05 tomchristie

mainly the browsable API/ admin stuffs?

auvipy avatar May 13 '20 11:05 auvipy

Issue here is that we're still on Bootstrap v3.x, and the upgrade to v4.x isn't just a swap it in. There's HTML changes throughout, which make it a good mini-project for a willing contributor.

carltongibson avatar May 14 '20 12:05 carltongibson

@carltongibson Something that’s not 100% clear to me. Are we okay with the jQuery update we’ve made?

tomchristie avatar May 14 '20 13:05 tomchristie

I guess so, since it oughta be backwards compatible

tomchristie avatar May 14 '20 13:05 tomchristie

Yeah... I believe so. Bar some dropping of browsers we don't support jQuery hasn't broken anything for a long while.

carltongibson avatar May 14 '20 13:05 carltongibson

@carltongibson -- I'd be willing to have a go at this if you think I have the skills (It is 'just' template layouts and bootstrap right...).

smithdc1 avatar Jul 08 '20 20:07 smithdc1

@carltongibson -- I'd be willing to have a go at this if you think I have the skills (It is 'just' template layouts and bootstrap right...).

kind of yes.

auvipy avatar Jul 08 '20 23:07 auvipy

@smithdc1: yes, it’s exactly that. Updating the Browsable API templates to use the newer version. (I didn’t look into it but I see v5 is on the way — don’t know if it’s worth skipping straight to that?)

carltongibson avatar Jul 09 '20 06:07 carltongibson

Yes v5 in alpha and a few alpha releases expected over the coming months weeks/months, my guess is v5 will be out in Q4. The main thing with v5 is it no longer depends upon jQuery. The upgrade from v4-v5 should be much easier than v3-v4.

I'll go investigate.

smithdc1 avatar Jul 09 '20 07:07 smithdc1

Yes v5 in alpha and a few alpha releases expected over the coming months weeks/months, my guess is v5 will be out in Q4. The main thing with v5 is it no longer depends upon jQuery. The upgrade from v4-v5 should be much easier than v3-v4.

I'll go investigate.

agree

auvipy avatar Jul 09 '20 07:07 auvipy

Super. Thanks. 🥇

carltongibson avatar Jul 09 '20 08:07 carltongibson

On the topic of updating themes... What about the docs, they seem to be on bootstrap 2?

smithdc1 avatar Jul 15 '20 20:07 smithdc1

@smithdc1 The docs are a mkdocs theme. In docs_theme. Maybe if you have energy, but one thing at a time I'd say... 😬

carltongibson avatar Jul 15 '20 20:07 carltongibson

Any updates on this? Our team needs this for a security audit for medical industry. I'm not sure why Bootstrap 3.x is considered a security hole, but the hospitals are going to get the security audit report and they probably will not be able to be convinced that Bootstrap 3 javascript for dropdowns and form elements is not a security hole. We are kindof stuck :(

joshjung avatar Sep 10 '20 15:09 joshjung

Suggestion: bypass this and go straight for Bootstrap v5, see #7411.

jscaux avatar May 31 '21 04:05 jscaux

@carltongibson If possible, I would like to work on updating MkDocs theme in docs_theme to BS5. I checked #7411, and it's about the version for the templates. Although it does have some file changes inside docs_theme.

jeffersonataide avatar Oct 04 '21 03:10 jeffersonataide

@jefferson2z — I'm sure @tomchristie would be happy to see input on updating the docs theme. #7411 is about bringing the Browseable API up to the latest bootstrap. I'm sure input would be welcome there too! 😀

carltongibson avatar Oct 07 '21 09:10 carltongibson

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] avatar Mar 27 '22 19:03 stale[bot]

As others have mentioned, the old version of bootstrap comes up in security audits. Moving to v5 would be great.

https://github.com/encode/django-rest-framework/discussions/9028

sww314 avatar Jan 03 '24 15:01 sww314