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

Animated highlight text on hover #2036

Merged
merged 4 commits into from
Jun 25, 2024
Merged

Conversation

Zubair286
Copy link
Contributor

@Zubair286 Zubair286 commented Jun 14, 2024

Copy link
Contributor

@raineorshine raineorshine left a comment

Choose a reason for hiding this comment

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

The intention is to create a pulse animation that goes back and forth between light blue and dark blue. Sorry that wasn't clear in the issue description!

@@ -268,13 +268,14 @@ const ThoughtContainer = ({
// See: https://stackoverflow.com/a/46452396/480608
const styleThought = useMemo(
(): React.CSSProperties => ({
...(isChildHovering ? { color: colors.highlight, WebkitTextStrokeWidth: '0.05em' } : null),
...(isChildHovering
? { color: isChildHovering ? '#538392' : '#FFFFFF', transition: 'color 1s', WebkitTextStrokeWidth: '0.05em' }
Copy link
Contributor

Choose a reason for hiding this comment

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

Please use themeColors rather than hard-coding color values.

@Zubair286 Zubair286 changed the title Animated highlight text on hover Animated highlight text on hover #1954 Jun 20, 2024
@Zubair286 Zubair286 changed the title Animated highlight text on hover #1954 Animated highlight text on hover Jun 20, 2024
@raineorshine raineorshine force-pushed the feature/1954 branch 3 times, most recently from 06601c1 to 9b18ba3 Compare June 25, 2024 18:28
@raineorshine raineorshine merged commit 4724720 into cybersemics:main Jun 25, 2024
4 checks 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.

2 participants