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

Made changes to improve code & style of "try our assistant" in dashboard #2695 #2709

Closed
wants to merge 1 commit into from

Conversation

Amarjit0511
Copy link

Made the following changes for the issue: #2695

  • Used Box instead of Flex as it is simpler and more appropriate here.
  • Added mb={6} to add some margin at the bottom of the box.
  • Used Heading with as="h2" instead of size="lg".
  • Added fontWeight="bold" and fontSize="xl" to make the button more prominent.
  • Changed the button text to "Start Chatting" as it is more action-oriented.
  • Added _hover to change the background color of the button on hover.
  • Used passHref in the Link component to pass the href to the child component.
  • Changed the Button component to an anchor a tag with as="a" to make it more accessible.

@github-actions
Copy link

pre-commit failed.
Please run pre-commit run --all-files locally and commit the changes.
Find more information in the repository's CONTRIBUTING.md

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.

1 participant