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

🪟 fix+feat: General UI Enhancements #2619

Merged
merged 46 commits into from May 9, 2024

Conversation

techwithanirudh
Copy link
Contributor

@techwithanirudh techwithanirudh commented May 5, 2024

Summary

A few UI changes to match closer to OpenAI's design and a few general UI Updates.

Before:
image

After:
image

Before:
image

After:
image

Before:
image

After:
image

Before:
image

After:
image

Before:
image

After:
image

Before:
image

After:
image

Before:
image

After:
image

Before:
image

After:
image

General Fixes:

fix: Change validation errors texts from black/white to red.
fix: Handle an edge case where focus is lost while a SocialButton is held, but not clicked
fix: Add focus border for ChatInput on light mode
feat: Add disabled property for NavLink
feat: Move export conversation top right, like ChatGPT's share button.
image
feat: Match Login/Register/ResetPassword Form input styles to OpenAI Input styles
fix: Optimize settings for mobile, and add a fixed width for the modal
fix: General colors fixes for dark mode
fix: Hover states for Prev/Next buttons

Change Type

  • Bug fix (non-breaking change which fixes an issue)

Testing

N/A

Test Configuration:

N/A

Checklist

  • My code adheres to this project's style guidelines
  • I have performed a self-review of my own code
  • I have commented in any complex areas of my code
  • My changes do not introduce new warnings
  • Local unit tests pass with my changes

techwithanirudh and others added 24 commits May 5, 2024 06:34
@techwithanirudh techwithanirudh marked this pull request as ready for review May 6, 2024 10:21
@techwithanirudh techwithanirudh marked this pull request as draft May 7, 2024 04:21
TODO: Make height dynamic, fix dark mode colors
fix: make endpoint icon smaller
- Updated the dropdown styles to match the aesthetic of ChatGPT.
- Decreased spacing within the conversation area for cleanliness.
- Replaced the current archive icon with the ChatGPT's icon.
@techwithanirudh techwithanirudh marked this pull request as ready for review May 8, 2024 12:18
@techwithanirudh
Copy link
Contributor Author

@danny-avila Could you please review this P.R?

@danny-avila
Copy link
Owner

There's 2 minor merge conflicts, I will see if I can resolve them later unless you beat me to it.

@techwithanirudh
Copy link
Contributor Author

@danny-avila I resolved the merge conflicts 😉

@techwithanirudh techwithanirudh changed the title feat: Minor UI Design Changes General UI Enhancements May 9, 2024
@techwithanirudh techwithanirudh changed the title General UI Enhancements feat: General UI Enhancements May 9, 2024
@techwithanirudh techwithanirudh changed the title feat: General UI Enhancements fix+feat: General UI Enhancements May 9, 2024
@danny-avila
Copy link
Owner

A lot of changes here that could've been individual PRs. Please reduce the scope of your PRs next time.

@danny-avila danny-avila changed the title fix+feat: General UI Enhancements 🪟 fix+feat: General UI Enhancements May 9, 2024
@danny-avila danny-avila merged commit 8f20fb2 into danny-avila:main May 9, 2024
1 check passed
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