Conversation
✅ Deploy Preview for delicate-starship-6bdc01 ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
jbloom
approved these changes
Mar 1, 2026
Member
jbloom
left a comment
There was a problem hiding this comment.
Thanks so much, @wperlichek.
Honestly, this sort of webpage styling is above my knowledge level, but the preview looks good and your description of the fix makes sense.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
I was looking at the deployed site and noticed my card was a bit thinner when I decreased the window size. Notice in the screenshot below that my card is just a bit thinner because I hardcoded the width previously. I did a fair bit of UI work years ago and this was bugging me 🙂
I think overall, how this was set up before, we had different styling settings for sections with >= 3 members, 1 team member, and 2 members. It doesn't really make sense to me, it looks like it's sort of an artifact of the past. I think the styling rules should be applied to all containers the same no matter how many members are there.
I think this is consistent and cleaner and we won't have to worry about messing with these styling any longer for any sections or team members we add.
Before this fix (deployed now)
After this fix (locally):
Smoke test LGTM: