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: add wide-mode #325

Merged
merged 5 commits into from
Feb 10, 2025
Merged

feat: add wide-mode #325

merged 5 commits into from
Feb 10, 2025

Conversation

yz778
Copy link
Contributor

@yz778 yz778 commented Feb 1, 2025

No description provided.

@yz778
Copy link
Contributor Author

yz778 commented Feb 1, 2025

Adding initial wide-screen support. Not ready to merge as not everything has been "wide-screen" enabled.

@yz778 yz778 changed the title feat: add wide-mode [WIP] feat: add wide-mode Feb 1, 2025
@yz778
Copy link
Contributor Author

yz778 commented Feb 1, 2025

Ok this should be ready for review and merging.

Also, I found what might be a bug @n4ze3m, in Message.tsx around line 138 you have:

${ props.message_type && "italic text-gray-500 dark:text-gray-400 text-sm" }

Which returns "undefined" into classNames. Not sure if that was intended, but I left it as is.

@n4ze3m
Copy link
Owner

n4ze3m commented Feb 1, 2025

Actually, it's used in the side panel since Message.tsx is a common component. :)

image

@yz778
Copy link
Contributor Author

yz778 commented Feb 9, 2025

@n4ze3m updated to work with the latest UI changes

@n4ze3m n4ze3m changed the base branch from main to next February 10, 2025 14:35
@n4ze3m n4ze3m merged commit 17c0b1c into n4ze3m:next Feb 10, 2025
3 checks passed
@yz778 yz778 deleted the wide-mode branch February 16, 2025 18:27
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