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

Zw/feature/tool tip summary #72

Merged
merged 10 commits into from
Sep 19, 2024
Merged

Zw/feature/tool tip summary #72

merged 10 commits into from
Sep 19, 2024

Conversation

zachwolek
Copy link
Collaborator

@zachwolek zachwolek commented Sep 19, 2024

Description of Changes

PR creates a Tooltip for the Dashboard page, completing the following:

  • links will have a "hover over" action will pop up a tooltip bubble, will pop up to the right of the link, and have the arrow pointing left at the middle of the link's height
  • Text under "Popular Links" header and above list of links will suggest "Hover over links for a summary of content"

What are the relevant tickets (if any)

Works on #69 but does not fully complete it, as waiting AI summary for display

Screenshots (if applicable)

Screenshot 2024-09-19 at 11 58 26 AM

Testing

Testing is not completed on Tooltip functionality yet

Checklist

  • The code follows the project's coding standards.
  • Unit tests have been added or updated to cover the changes.
  • Documentation has been updated to reflect the changes (if applicable).
  • The code compiles without errors.
  • The changes have been tested locally and pass all relevant tests.
  • All new and existing tests pass.
  • The pull request has been reviewed by at least one other contributor.

Reviewer Instructions

PR has already been merged with previous Dashboard.tsx on main

Deployment Notes

N/A

Additional Information

Tooltip functionality is ready for AI summary.

@Swatzell
Copy link
Collaborator

Great Job on the design of the pop up!

@Swatzell Swatzell merged commit 1dfd9c0 into main Sep 19, 2024
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