Skip to content

UI Improvements, implements task 3 in issue #279 #340

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.

Sign up for GitHub

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

Jump to bottom
Merged
dartpain merged 2 commits into from
Sep 24, 2023
Merged

UI Improvements, implements task 3 in issue #279 #340

dartpain merged 2 commits into from
Sep 24, 2023

Conversation

@ivinjabraham
Copy link
Contributor

@ivinjabraham ivinjabraham commented Sep 23, 2023 •
edited
Loading

Following changes were implemented:

  • Button border and onHover border consistency changes as directed in issue UI/UX Updates  #279
  • Arrow icon positioning in source docs dropdown
  • APIKeyModal, SelectDocsModal and Modal border roundedness adjusted to make combined elements fit together better.
@vercel
Copy link

vercel bot commented Sep 23, 2023 •
edited
Loading

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

Name Status Preview Comments Updated (UTC)
docs-gpt ✅ Ready (Inspect) Visit Preview 💬 Add feedback Sep 23, 2023 4:23pm
nextra-docsgpt ✅ Ready (Inspect) Visit Preview 💬 Add feedback Sep 23, 2023 4:23pm
@ivinjabraham ivinjabraham changed the title UI Improvements and fixes Sep 23, 2023
dartpain
dartpain approved these changes Sep 24, 2023
View reviewed changes
Copy link
Contributor

@dartpain dartpain left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good PR, Thank you!
@codecov
Copy link

codecov bot commented Sep 24, 2023

Codecov Report

Merging #340 (d983a51) into main (9303f3b) will not change coverage.
Report is 1 commits behind head on main.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #340   +/-   ##
=======================================
  Coverage   13.41%   13.41%           
=======================================
  Files          44       44           
  Lines        2416     2416           
=======================================
  Hits          324      324           
  Misses       2092     2092

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more
@dartpain dartpain merged commit f7244dd into arc53:main Sep 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants

@ivinjabraham @dartpain