Skip to content
This repository has been archived by the owner on Jan 10, 2025. It is now read-only.

feat: add loader in baseapp #492

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

feat: add loader in baseapp #492

wants to merge 38 commits into from

Conversation

vasudev-io
Copy link
Contributor

@vasudev-io vasudev-io commented Nov 21, 2024

still requires some testing but i think it looks cool

probably need to polish some of the animations and stuff as well

Screen.Recording.2024-11-21.at.14.44.40.mov

Copy link

vercel bot commented Nov 21, 2024

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

Name Status Preview Comments Updated (UTC)
highlight-chat-frontend ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 27, 2024 5:04pm

@vasudev-io vasudev-io requested a review from vaunblu November 21, 2024 01:27
@vasudev-io vasudev-io added enhancement New feature or request help wanted Extra attention is needed labels Nov 23, 2024
@vasudev-io vasudev-io removed the help wanted Extra attention is needed label Nov 23, 2024
@vasudev-io vasudev-io changed the title feat: add loader in baseapp (DO NOT MERGE YET) feat: add loader in baseapp Nov 27, 2024
@vaunblu
Copy link
Contributor

vaunblu commented Dec 12, 2024

Hey so far looks good as it matches floaty. Will have to put this merge on pause for a little just to make sure it's necessary to add. We're making some big changes, and one of those is moving most of the chat stuff to floaty. There's a chance we end up using the implementation Josh pushes out (which is what you did on floaty)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
DO NOT MERGE enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants