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.

Sign up for GitHub

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

Jump to bottom

feat: add board list responsiveness #1439

Merged
merged 6 commits into from
Apr 28, 2023
Merged

feat: add board list responsiveness #1439

merged 6 commits into from
Apr 28, 2023

Conversation

juniorboos
Copy link
Contributor

@juniorboos juniorboos commented Apr 27, 2023 •
edited
Loading

Relates to:

Screenshots (if visual changes)

Old New
image image
image image
image image

Proposed Changes

  • Removed unnecessary inline styles and hardcoded pixels.
  • Removed unnecessary Flex wrapping TeamHeader component.
  • Replaced some margins with gap.
  • Added wrap so that components become responsive.
  • Added mx to CardBody to compensate for its box-shadow and doesn't overflow.

@juniorboos juniorboos marked this pull request as ready for review April 27, 2023 14:11
@juniorboos juniorboos self-assigned this Apr 27, 2023
@juniorboos juniorboos added enhancement New feature or request frontend labels Apr 27, 2023
@juniorboos juniorboos requested a review from a team April 27, 2023 14:11
@juniorboos juniorboos changed the title Feat/add board list responsiveness feat: add board list responsiveness Apr 27, 2023
@juniorboos juniorboos merged commit f2f7bcb into main Apr 28, 2023
@juniorboos juniorboos deleted the feat/add-board-list-responsiveness branch April 28, 2023 10:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nunocaseiro nunocaseiro nunocaseiro approved these changes

Assignees

@juniorboos juniorboos

Labels
enhancement New feature or request frontend
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@juniorboos @nunocaseiro