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

Prevent registration button from moving #5105

Merged
merged 2 commits into from
Oct 28, 2024
Merged

Prevent registration button from moving #5105

merged 2 commits into from
Oct 28, 2024

Conversation

ivarnakken
Copy link
Member

@ivarnakken ivarnakken commented Oct 26, 2024

Description

Also fix and improve skeleton components on frontpage and event detail sidebar

Result

  • Changes look good on both light and dark theme.
  • Changes look good with different viewports (mobile, tablet, etc.).
  • Changes look good with slower Internet connections.
Screen.Recording.2024-10-26.at.16.50.51.mov

Testing

  • I have thoroughly tested my changes.

It's difficult to reproduce the button move on dev fro some reason - might have something to do with the websocket


Resolves ABA-1148

The list will be filled with skeletons if there's some events in the
redux state from earlier. Also better empty state
@ivarnakken ivarnakken added enhancement Pull requests that make enhancements, instead of just purely new features review-needed Pull requests that need review bug-fix Pull requests that fix a bug labels Oct 26, 2024
@ivarnakken ivarnakken requested a review from a team October 26, 2024 14:56
@ivarnakken ivarnakken self-assigned this Oct 26, 2024
Copy link

vercel bot commented Oct 26, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
lego-bricks-storybook ⬜️ Ignored (Inspect) Visit Preview Oct 26, 2024 6:41pm

Also improve skeleton components on sidebar. There's still issues with
the user grid skeleton, but that can be saved for later :)
Copy link

linear bot commented Oct 26, 2024

@ivarnakken ivarnakken merged commit d4fddd7 into master Oct 28, 2024
6 checks passed
@ivarnakken ivarnakken deleted the events-skeletons branch October 28, 2024 09:22
@ivarnakken ivarnakken added the approved Pull requests that have been approved label Oct 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Pull requests that have been approved bug-fix Pull requests that fix a bug enhancement Pull requests that make enhancements, instead of just purely new features review-needed Pull requests that need review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants