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

feat!: rewrite to lit #367

Merged
merged 154 commits into from
Jan 23, 2024
Merged

feat!: rewrite to lit #367

merged 154 commits into from
Jan 23, 2024

Conversation

favna
Copy link
Member

@favna favna commented Jul 23, 2023

TODO taken from #337


Something weird is happening with ephemeral replies. I can't quite find out what and why:
image

Also in non ephemeral with compact mode the text doesn't center out correctly with the username:
image


Some things I am noticing that I'd just like to drop here:

  1. Kinda mentioned it before but there are some wacky things with compact mode. Outlining, font sizes, etc.

  2. The icon for channel mentions isn't properly aligned.
    image. I can't tell for sure myself but I also think the other mention SVGs are slightly misaligned.

  3. DiscordLink doesn't use the consumer for lightTheme yet

  4. Update all codesandbox links (post releasing alpha version)

  5. Add Svelte and Sveltekit codesandbox links

Will probably edit this message as I progress further writing unit tests

@favna favna mentioned this pull request Jul 23, 2023
@Lulalaby
Copy link

Lulalaby commented Sep 2, 2023

Members is not a mentionable and won't become one in near future, because it's an tab only moderators+ can see.
So might not be worth adding.

@favna
Copy link
Member Author

favna commented Sep 2, 2023

Members is not a mentionable and won't become one in near future, because it's an tab only moderators+ can see. So might not be worth adding.

Good point. We can skip it for now.

@Lulalaby
Copy link

Lulalaby commented Sep 2, 2023

Imma lyk if it changes

@favna
Copy link
Member Author

favna commented Sep 2, 2023

Finished rebasing the branch so it's clean to work in again

@favna favna marked this pull request as ready for review January 22, 2024 23:35
@favna favna enabled auto-merge (squash) January 23, 2024 00:05
@favna favna merged commit 8b31f45 into main Jan 23, 2024
7 checks passed
@favna favna deleted the feat/rewrite-to-lit branch January 23, 2024 00:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants