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

[UI] fix home cards bug, sass dependency issue #88

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

Conversation

Nauxie
Copy link
Member

@Nauxie Nauxie commented Jul 4, 2022

No description provided.

taesungh and others added 30 commits August 17, 2021 22:25
The navigation bar was improperly built to produce nested `<a>` tags.
The component logic was also more complicated than necessary.
- use proper react-bootstrap components for the navigation bar
- replace complicated history logic with React Router's `NavLink`
- make the navigation logo to use a NavLink instead of a regular anchor
- specify dark variant for `Navbar` to get light collapsing icon
- Simplify component logic for customizing header content
- Convert 16-bit png to optimized image
- Fix positioning of background image
- Simplify footer styles
- Add more descriptive alt text to footer icons
- Create variable for colors of blue gradient
- Add blue gradient to body
…card, fixed image squishing into image cropping
… spacing, fixed name sizing issue on mobile on About page
More fixes to improve experience on mobile/tablet devices
Bumps [node-sass](https://github.com/sass/node-sass) from 4.14.1 to 7.0.0.
- [Release notes](https://github.com/sass/node-sass/releases)
- [Changelog](https://github.com/sass/node-sass/blob/master/CHANGELOG.md)
- [Commits](sass/node-sass@v4.14.1...v7.0.0)

---
updated-dependencies:
- dependency-name: node-sass
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <[email protected]>
…-7.0.0

Bump node-sass from 4.14.1 to 7.0.0
Nauxie and others added 2 commits July 4, 2022 16:34
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.

3 participants