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 tooltip and hover text color for Export icon #419

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

VikasShivakumar
Copy link

Fixes #409

Fix the blank tooltip for the Export icon and add hover text color changes for navigation icons.

  • Tooltip Fix:

    • Set the tooltip content for the Export icon to "Export Chat" in app/components/chat/ExportChatButton.tsx.
  • Hover Text Color Change:

    • Add CSS rule to change text color of .ExportIcon to red on hover in app/components/chat/BaseChat.module.scss.
    • Add CSS rule to change text color of other navigation icons to red on hover in app/components/chat/BaseChat.module.scss.
    • Add class ExportIcon to the IconButton component in app/components/chat/ExportChatButton.tsx.
    • Add class NavigationIcon to other navigation icons in app/components/sidebar/Menu.client.tsx.

Fixes coleam00#409

Fix the blank tooltip for the Export icon and add hover text color changes for navigation icons.

* **Tooltip Fix:**
  - Set the tooltip content for the Export icon to "Export Chat" in `app/components/chat/ExportChatButton.tsx`.

* **Hover Text Color Change:**
  - Add CSS rule to change text color of `.ExportIcon` to red on hover in `app/components/chat/BaseChat.module.scss`.
  - Add CSS rule to change text color of other navigation icons to red on hover in `app/components/chat/BaseChat.module.scss`.
  - Add class `ExportIcon` to the `IconButton` component in `app/components/chat/ExportChatButton.tsx`.
  - Add class `NavigationIcon` to other navigation icons in `app/components/sidebar/Menu.client.tsx`.
@wonderwhy-er
Copy link
Collaborator

There is conflict to resolve

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.

### Minor Bug: Blank tooltip for Export icon and request to add hover text colour changes for other icons
2 participants