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

Add CSS variables for background colors #126

Merged
merged 1 commit into from
Dec 19, 2024

Conversation

joaopalmeiro
Copy link
Contributor

@joaopalmeiro joaopalmeiro commented Jul 15, 2024

Closes #124


Hi! 👋

As per the discussion on #124, here is a proposal to include two CSS variables to style the (animated) background colors for this extension. The highlight color fallback has also been updated. Let me know what you think, please!

Screen.Recording.2024-07-15.at.18.23.42.mov
Screenshot 2024-07-15 at 18 23 58

@krassowski krassowski added the enhancement New feature or request label Jul 15, 2024
@joaopalmeiro
Copy link
Contributor Author

Hi! 👋

Sorry to bother you, @krassowski, but any chance you could review this MR, please?

Thanks!

Copy link
Collaborator

@krassowski krassowski left a comment

Choose a reason for hiding this comment

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

Looks good to me, as per discussion in #124

@joaopalmeiro
Copy link
Contributor Author

Thanks, @krassowski!

@krassowski
Copy link
Collaborator

@joaopalmeiro if you have not signed it before you may need to sign CLA for this to be merged as per https://github.com/deshaw/jupyterlab-execute-time/blob/master/CONTRIBUTING.md

@joaopalmeiro
Copy link
Contributor Author

Thanks for the reminder! I signed at the time I opened MR. Is there any way to confirm?

@mlucool
Copy link
Member

mlucool commented Dec 19, 2024

Confirmed @joaopalmeiro signed. Merging

@mlucool mlucool merged commit d9011e9 into deshaw:master Dec 19, 2024
3 checks passed
@joaopalmeiro joaopalmeiro deleted the add-css-vars-bg branch December 19, 2024 17:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Use a theme CSS variable instead of a Material Design color for the highlight animation
3 participants