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

Reorganize styling #969

Closed
wants to merge 3 commits into from
Closed

Reorganize styling #969

wants to merge 3 commits into from

Conversation

victorlin
Copy link
Member

@victorlin victorlin commented Jul 30, 2024

Description of proposed changes

Small fixes around styling. I might incorporate more from #963 (comment) into this PR.

Related issue(s)

#963

Checklist

Reverts "Set container styles in one place" (32229ad) and moves the
additional styling to globals.css
This has no effect because there is only one child element.
Use the media queries from Bootstrap's various tiers for more
predictable responsive design behavior. Two changes were made:

1. Convert max-width queries to min-width queries
2. For queries on widths that were not at Bootstrap's breakpoints, round
   to the closest Bootstrap breakpoint.
@victorlin victorlin self-assigned this Jul 30, 2024
@nextstrain-bot nextstrain-bot temporarily deployed to nextstrain-s-victorlin--r1cpz0 July 30, 2024 23:50 Inactive
@victorlin
Copy link
Member Author

Closing this since we are moving away from styled-components: #963 (comment)

Thinking through if each commit should be considered in the future:

  1. e40cac8 @genehack you may be interested in this one – it replaces styled-components usage with plain CSS.
  2. f32948a removes one line of styled-components which may be beneficial as well.
  3. 8dab1d8 is an adjustment within styled-components which would still be good to apply to the styled-components replacement when that lands.

@victorlin victorlin closed this Oct 2, 2024
@victorlin victorlin deleted the victorlin/reorganize-styling branch October 2, 2024 19:27
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.

2 participants