Skip to content
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

Upgrade bootstrap version #987

Closed
kerchner opened this issue Jul 22, 2019 · 5 comments
Closed

Upgrade bootstrap version #987

kerchner opened this issue Jul 22, 2019 · 5 comments
Assignees
Milestone

Comments

@kerchner
Copy link
Member

kerchner commented Jul 22, 2019

@kerchner kerchner added the UX label Jul 22, 2019
@kerchner kerchner added this to the Backlog milestone Jul 22, 2019
@lwrubel lwrubel modified the milestones: Backlog, 2.next Mar 11, 2020
This was referenced Mar 11, 2020
@dolsysmith
Copy link
Contributor

dolsysmith commented Mar 17, 2020

  • base.html
    • changes to navbar
  • change_log.html
    • pager class deprecated
  • collection_create.html
    • breadcrumb class now requires .breadcrumb-item on descendants
  • collection_detail.html
    • see above for breadcrumb class
    • .panel deprecated in favor of .card classes
    • .glyphicon no longer included (?)
    • changes to .dropdown-menu implementation
    • .btn-default --> .btn-secondary
    • changes to .nav implementation
  • collection_set_create.html
    • see above for .breadcrumb
  • collection_set_detail.html
    • see above: .breadcrumb, .panel, .dropdown-menu, '.glyphicon, .btn-default`
  • collection_set_list.html
    • see above: .nav
  • collection_set_update.html
    • see above: .breadcrumb, .panel
  • export_create.html
    • see above: .breadcrumb
  • export_detail.html
    • see above: .breadcrumb, .panel
  • harvest_detail.html
    • see above: .breadcrumb
  • harvest_list.html:
    • see above: .breadcrumb, .pager
  • home.html
    • see above: .panel
  • monitor.html
    • see above: .panel
  • profile_details.html
    • see above: .glyphicon
  • seed_create.html
    • see above: .breadcrumb
  • seed_detail.html
    • see above: .glpyhicon, .dropdown-menu, .panel
  • seed_update.html
    • see above: .breadcrumb
    • .page-header deprecated
  • terms_snippet.html
    • .well deprecated
  • data_tables.js
    • pager class now pagination

@lwrubel
Copy link
Collaborator

lwrubel commented Apr 3, 2020

Note that we use django-crispy-forms for forms creation and formatting. We're currently on 1.7.2 but should likely upgrade to 1.9.0 for bootstrap 4 support. However, upgrading to any version beyond the current one makes the seed choice fields in the export form not work. (Release notes).

This part will need attention: https://github.com/gwu-libraries/sfm-ui/blob/master/sfm/ui/forms.py#L950

@dolsysmith
Copy link
Contributor

Glyphicons is no longer part of Bootstrap as of V4. The migration guide recommends the upstream version, but that appears to have an associated cost (albeit low).

@dolsysmith
Copy link
Contributor

dolsysmith commented Apr 13, 2020

The dropdown menu for Add Collection is dropping up in Bootstrap 4. I can't quite figure out why, except that it seems to have something to do with the behavior of Popper.js (which Bootstrap 4 uses for dropdowns).

@lwrubel
Copy link
Collaborator

lwrubel commented Apr 14, 2020

Looks like it is popping up because the menu is so long there is not enough room on the screen below the button. I'm looking at making it a little more compact since it seems like there is too much spacing in the menu after the upgrade.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants