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 and react-bootstrap to latest, update site components #313

Merged
merged 4 commits into from
Feb 28, 2023

Conversation

jimmcgaw
Copy link
Collaborator

No description provided.

@elimisteve
Copy link
Member

Nice! There have been a couple remaining old dependencies and this should help a bunch with that.

@elimisteve
Copy link
Member

These larger fonts look fantastic btw

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@elimisteve
Copy link
Member

elimisteve commented Feb 28, 2023

@jimmcgaw Looks like use of bsStyle was accidentally re-added to this branch in the last force-push?

EDIT: No I just checked out the wrong branch locally without running npm i; nevermind.

* LeapChat: Info button/icon: size={25} -> 19 (to match Username Edit button/icon)

* react-bootstrap upgrade: Make all modals size="lg", not the smaller default

* CSS: Make .icon-button global so it applies to modals, too

* FaCog: size={30} -> 27, plus `margin-top: 3px` to keep alignment
Copy link
Member

@elimisteve elimisteve left a comment

Choose a reason for hiding this comment

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

LGTM 👌

Mentioned a trivial thing, but let's merge and we can fuck with the README command formatting later; no need to block.

README.md Show resolved Hide resolved
README.md Show resolved Hide resolved
@elimisteve
Copy link
Member

@jimmcgaw Ready to merge? LGTM

@elimisteve
Copy link
Member

Dooo iiit

@elimisteve
Copy link
Member

🐎 🐎 🐎 🐎 🐎

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

Successfully merging this pull request may close these issues.

None yet

2 participants