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

Bump bootstrap from 4.6.2 to 5.0.0 #352

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Sep 17, 2024

Resolves #362
Bumps bootstrap from 4.6.2 to 5.0.0.

Release notes

Sourced from bootstrap's releases.

v5.0.0

Highlights

#32155: Updated make-col() mixin to generate equal columns when no size is specified #32763: Added new color-scheme() mixin #33389: Dropdown menus now have option become clickable #33453: Added new docs footer #33548: Offcanvas header components are now vertically aligned #33549: Added offcanvas-top modifier #33634: Added support for .dropdown-items wrapped in <li>s #33626: Fix v5 regressions in tab dropdown functionality

🚀 Features

  • #32763: Add color-scheme mixin
  • #33389: Dropdown — Add option to make the dropdown menu clickable
  • #33549: Add offcanvas-top modifier

🎨 CSS

  • #32155: Add equal column mixin
  • #32763: Add color-scheme mixin
  • #33292: Make accordion icon rotation more natural
  • #33411: Fix validation feedback icon in select multiple
  • #33478: Make .nav-link color consistent when using buttons
  • #33482: Dropdown — Apply positioning only when Popper is not used
  • #33548: Vertically align offcanvas header components
  • #33549: Add offcanvas-top modifier
  • #33550: Spinner alignment changes
  • #33598: Hide validation icons from multiple selects
  • #33600: Have $form-check-input-border's default derive from $black
  • #33607: Reduce color-scheme complexity
  • #33642: use :read-only css selector instead [readonly] for consistency
  • #33658: fix: use list-group variable instead of alert
  • #33736: accordion: fix border-top on Firefox

☕️ JavaScript

  • #32439: Decouple BackDrop from modal
  • #33245: Decouple Modal's scrollbar functionality
  • #33249: Simplify Modal Config
  • #33250: Simplify ScrollSpy config
  • #33310: fix: make EventHandler better handle mouseenter/mouseleave events
  • #33389: Dropdown — Add option to make the dropdown menu clickable
  • #33429: Remove element event listeners through base component
  • #33451: Add missing things in hide method of dropdown
  • #33456: Use our isDisabled util on dropdown
  • #33466: Refactor dropdown's hide functionality
  • #33479: Fix dropdown escape propagation
  • #33496: Use cached noop function

... (truncated)

Commits

Dependabot compatibility score

You can trigger a rebase of this PR by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)
    You can disable automated security fix PRs for this repo from the Security Alerts page.

Note
Automatic rebases have been disabled on this pull request as it has been open for over 30 days.

@dependabot dependabot bot added the dependencies Pull requests that update a dependency file label Sep 17, 2024
Copy link

netlify bot commented Sep 17, 2024

Deploy Preview for s-forms-kbss ready!

Name Link
🔨 Latest commit 7d0cc2c
🔍 Latest deploy log https://app.netlify.com/sites/s-forms-kbss/deploys/67379ae469ac74000824eb1f
😎 Deploy Preview https://deploy-preview-352--s-forms-kbss.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@palagdan
Copy link
Collaborator

palagdan commented Oct 10, 2024

@blcham
I cannot simulate the issue you described here. You might want to try again in the Netlify preview, as everything looks good on my end after I fixed the bottom margin.

Could you please take a look at the result? Perhaps you might find something else.

Also, when I attempted to upgrade Bootstrap React from version 1.0.1 to 2.10.5, everything broke. The blue you see is a wizard step.

@blcham
Copy link
Collaborator

blcham commented Oct 10, 2024

Also, when I attempted to upgrade Bootstrap React from version 1.0.1 to 2.10.5, everything broke. The blue you see is a wizard step.

Yes, I can imagine, have a look at:
https://react-bootstrap.netlify.app/docs/migrating/

Could you please take a look at the result? Perhaps you might find something else.

It would be nice to fix the invalid time first, as I cannot go over the whole form.

Could you explain the reason of the wizard step?

I do not understand. Could you reformulate the question?

@palagdan palagdan force-pushed the dependabot/npm_and_yarn/bootstrap-5.0.0 branch from 29ffeea to 4ddca1a Compare October 16, 2024 08:56
@blcham blcham force-pushed the dependabot/npm_and_yarn/bootstrap-5.0.0 branch 2 times, most recently from d8a96c7 to af52278 Compare November 4, 2024 13:27
@blcham
Copy link
Collaborator

blcham commented Nov 4, 2024

@blcham should compare how it looks compared with previous version

@blcham
Copy link
Collaborator

blcham commented Nov 7, 2024

Current state:

  • no need to change
    image
  • mouse pointer not indicating that section is clickable
    image
  • button to submit comments is outside of its pop-up
    image
  • selected value of codelist is underlined
    image

@blcham
Copy link
Collaborator

blcham commented Nov 7, 2024

@blcham it is needed to add autocomplete with tree example into form1

@palagdan palagdan force-pushed the dependabot/npm_and_yarn/bootstrap-5.0.0 branch from af52278 to e17bf31 Compare November 15, 2024 13:56
@palagdan palagdan requested review from LaChope and blcham and removed request for LaChope November 15, 2024 13:57
@blcham
Copy link
Collaborator

blcham commented Nov 15, 2024

  • Align opened/closed section (might consider opened wizard step layout as well) -- here, either remove blue or add blue. I do not care that much, although I prefer having blue to show the user which of those sections are open, but I do not know how it will look after the alignment
    image
  • Align bold/not bold fonts (to be honest, I do not understand why it should be bold, so I would leave it as not bold unless it has some meaning ... does it? ... I looked at bootstrap 5 examples, but I did not see components in bold font like that, but maybe I did not look well.
    image

@blcham
Copy link
Collaborator

blcham commented Nov 15, 2024

@palagdan, see the new issues that I found. I went over everything, so there should not be any more issues after this is done.

@blcham
Copy link
Collaborator

blcham commented Nov 15, 2024

@palagdan

image

Copy link
Collaborator

@blcham blcham left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See my comments.

@@ -227,12 +227,10 @@ export default class Question extends React.Component {
onMouseEnter={this._onMouseEnterHandler}
onMouseLeave={this._onMouseLeaveHandler}
>
<h6 className="d-inline" id={question["@id"]}>
Copy link
Collaborator

@blcham blcham Nov 15, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am a little afraid that removing id can ruin navigation to the questions; what do you think?

image

This is how we jump to concrete question:

...getP("startingQuestionId", "layout-options-65"),

Copy link
Collaborator

@palagdan palagdan Nov 15, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@blcham
In the older version, the red text color for the wizard was applied only on the first render. It has the same behavior now.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I returned the id for the question

@palagdan palagdan requested a review from blcham November 15, 2024 18:58
@palagdan palagdan removed the request for review from blcham November 15, 2024 18:58
@palagdan palagdan force-pushed the dependabot/npm_and_yarn/bootstrap-5.0.0 branch from 3ec324e to 7d0cc2c Compare November 15, 2024 19:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Migrate to bootstrap 5
2 participants