Skip to content
GitLab
Projects Groups Topics Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
  • castellum castellum
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributor statistics
    • Graph
    • Compare revisions
  • Issues 3
    • Issues 3
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 1
    • Merge requests 1
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Artifacts
    • Schedules
  • Deployments
    • Deployments
    • Releases
  • Packages and registries
    • Packages and registries
    • Container Registry
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • castellumcastellum
  • castellumcastellum
  • Merge requests
  • !1406

replace django-bootstrap4 by django-bs4

  • Review changes

  • Download
  • Patches
  • Plain diff
Closed Bengfort requested to merge spike-django-bs4 into master Jun 17, 2020
  • Overview 0
  • Commits 1
  • Pipelines 3
  • Changes 2

A while ago I got annoyed with django-bootstrap4 because of issues like !1405 (merged), so I re-implemented the whole thing and called it django-bs4. Now I finally found the time to test how well this works in the context of castellum.

I designed it as a drop-in replacement, so no changes are required to the code. As far as I could see this works just fine.

For the most part, django-bootstrap4 is great. We already have workarounds for the file widget. So I would not merging using this MR. But it is nice to know we have the option.

Edited Jun 17, 2020 by Bengfort
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: spike-django-bs4