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

Timeseries panel update #2927

Open
wants to merge 28 commits into
base: next
Choose a base branch
from
Open

Timeseries panel update #2927

wants to merge 28 commits into from

Conversation

kwongz
Copy link
Contributor

@kwongz kwongz commented Dec 17, 2024

Description

Checklist

  • For UI or styling changes, I have added a screenshot or gif showing before & after
  • I have added a changeset
  • I have added to the docs where applicable
  • I have added to the VS Code extension where applicable

@kwongz
Copy link
Contributor Author

kwongz commented Dec 31, 2024

@mcrascal

  • Fixed responsiveness for smaller screens
  • added Theme styling for darkmode

Lightmode:

image

Dark Mode:

image

Improvements:

  • I think we can look for a different text-color for the selected metric and the dates, currently using base-content-muted
  • Some of the font colors have changed, such as the bg for the positive and neg nums since we were using green and red 400 and 200, but I could only find the one positive and negative color. So the background red and green are a bit different then the original
    image
    -Not too familiar with bg-[radial-gradient(#f9fafb_1px,transparent_1px)] and adding out base vars instead of a hex code (#f9fafb), so I still need to fix the radial gradient in dark mode.

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