Skip to content

feat: ProfilePicture bounce, better tooltip#340

Open
ninetailedtori wants to merge 1 commit intocatppuccin:mainfrom
ninetailedtori:maintainer-state
Open

feat: ProfilePicture bounce, better tooltip#340
ninetailedtori wants to merge 1 commit intocatppuccin:mainfrom
ninetailedtori:maintainer-state

Conversation

@ninetailedtori
Copy link
Copy Markdown

@ninetailedtori ninetailedtori commented Apr 24, 2026

ProfilePicture optimisations, make it look pretty AND useful!!!

Direct changes:

  • Add a bounce state to the ProfilePicture element
  • Also, use a defined "placeholder" class that handles the svg for us, as well as the animations and colours.
    • Move the import from PortMaintainers.svelte to ProfilePicture.svelte to follow this new modular structure.
  • Add some optimisations as well
  • Add a bespoke tooltip, much more descriptive of the user as well.
  • Adds a click state to take you to the maintainer's GH URL. (makes it feel more reactive.)

Related changes:

  • Also fixes the fetch for maintainer icons, such that it has a proper retry logic and fail states, as well as updated to match the new profilepicture svelte object.

- Add a bounce state to the ProfilePicture element
- Also, use a defined "placeholder" class that handles the svg for us, as well as the animations and colours.
  - Move the import from PortMaintainers.svelte to ProfilePicture.svelte to follow this new modular structure.
- Add some optimisations as well
- Add a bespoke tooltip.

Signed-off-by: Toria <ninetailedtori@uwu.gal>
@ninetailedtori
Copy link
Copy Markdown
Author

Unfortunately, I cannot undo index.astro, as the commit hook that runs astro check and fmt actually caused it to end up empty commits every time! Very annoying :(

@ninetailedtori
Copy link
Copy Markdown
Author

recording_2026-04-26_16.29.12.mp4

@scarcekoi
Copy link
Copy Markdown
Member

scarcekoi commented Apr 26, 2026

Looks nice, seems good to merge?

@ninetailedtori
Copy link
Copy Markdown
Author

Looks nice, seems good to merge?

yea, good to merge imo! I triple checked, tested a ton, cleaned up stuff for performance too :]

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