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

Feature/sticker finalization #133

Open
wants to merge 11 commits into
base: main
Choose a base branch
from
Open

Conversation

tyleryy
Copy link
Contributor

@tyleryy tyleryy commented Nov 4, 2023

Overview

Stickers are positioned by anchoring to an existing component. Using API created by @alexanderl19 to place the stickers on the corners of components with offsets.

Details

Refactored components for compatibility. The "responsiveness" of this PR is that the positioning will be off the screen on smaller screens. The stickers are still going to be oddly positioned on window resize. However, on reload, they will be correctly positioned.

The issue before was the initial positioning was done by passing props and putting them into Framer Motion's initial prop. This has issues with SSR, which was the source of the "jumping" on window resize. Moved to inline styles solved this.

Copy link

github-actions bot commented Nov 4, 2023

Deploy preview for zothacks-site-2023 ready!

Name Hack at UCI Site
Preview Visit Preview
Commit 061ca23

Copy link

github-actions bot commented Nov 4, 2023

Deploy preview for zothacks-site-2023-sanity ready!

Name Sanity Studio
Preview Visit Preview
Commit 061ca23

@samderanova
Copy link
Contributor

samderanova commented Nov 4, 2023

Personally don't want to include any more stickers besides the heart one because of how much trouble we've had with them, even if we do have a "fix".

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.

None yet

2 participants