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

ui: overview page email signup missing background color #105740

Closed
j82w opened this issue Jun 28, 2023 · 0 comments · Fixed by #114538
Closed

ui: overview page email signup missing background color #105740

j82w opened this issue Jun 28, 2023 · 0 comments · Fixed by #114538
Labels
A-cluster-observability Related to cluster observability C-bug Code not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior. E-quick-win Likely to be a quick win for someone experienced.

Comments

@j82w
Copy link
Contributor

j82w commented Jun 28, 2023

There is text above the capacity usage Keep up-to-date with CockroachDB software releases and best practices. that is very hard to read. It seems like the background color is missing for the overview page.
Screenshot 2023-06-28 at 11 35 04 AM

Jira issue: CRDB-29170

@j82w j82w added C-bug Code not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior. T-cluster-observability A-cluster-observability Related to cluster observability labels Jun 28, 2023
@blathers-crl blathers-crl bot added this to Triage in Cluster Observability Jun 28, 2023
@j82w j82w moved this from Triage to Quick Win in Cluster Observability Jun 28, 2023
@maryliag maryliag added the E-quick-win Likely to be a quick win for someone experienced. label Sep 27, 2023
sudomateo added a commit to sudomateo/cockroachdb-cockroach that referenced this issue Nov 16, 2023
Previously, the styling for the `crl-email-subscription` CSS class would
render white text on a white background when the background image failed
to load. This made it difficult to read what the call to action was.

This patch removes the `background-image` property from the
`crl-email-subscription` CSS class and instead configures the class to
render black text on a white background.

Fixes: cockroachdb#105740
Part of: CRDB-29170

Release note (ui change): The overview page now correctly renders the
background color for the email signup, fixing an issue where it was
difficult to read the text within.
sudomateo added a commit to sudomateo/cockroachdb-cockroach that referenced this issue Nov 16, 2023
Previously, when the background image failed to load for the
`crl-email-subscription` CSS class it would render render white text on
a white background, making it difficult to read the call to action.

This patch removes the background image from the
`crl-email-subscription` CSS class and instead configures the class to
render black text on a white background.

Fixes: cockroachdb#105740
Part of: CRDB-29170

Release note (ui change): The overview page now correctly renders the
background color for the email signup, fixing an issue where it was
difficult to read the text.
craig bot pushed a commit that referenced this issue Nov 16, 2023
114538: ui: update `crl-email-subscription` styling r=sudomateo a=sudomateo

Previously, when the background image failed to load for the
`crl-email-subscription` CSS class it would render render white text on
a white background, making it difficult to read the call to action.

This patch adds the `background-color` property to the
`crl-email-subscription` CSS class so that it is used when the
background image fails to load.

Fixes: #105740
Part of: CRDB-29170

Release note (ui change): The overview page now correctly renders the
background color for the email signup, fixing an issue where it was
difficult to read the text.

Co-authored-by: Matthew Sanabria <24284972+sudomateo@users.noreply.github.com>
@craig craig bot closed this as completed in ca0a4c4 Nov 16, 2023
Cluster Observability automation moved this from Quick Win to Done Nov 16, 2023
blathers-crl bot pushed a commit that referenced this issue Nov 16, 2023
Previously, when the background image failed to load for the
`crl-email-subscription` CSS class it would render render white text on
a white background, making it difficult to read the call to action.

This patch adds the `background-color` property to the
`crl-email-subscription` CSS class so that it is used when the
background image fails to load.

Fixes: #105740
Part of: CRDB-29170

Release note (ui change): The overview page now correctly renders the
background color for the email signup, fixing an issue where it was
difficult to read the text.
blathers-crl bot pushed a commit that referenced this issue Nov 16, 2023
Previously, when the background image failed to load for the
`crl-email-subscription` CSS class it would render render white text on
a white background, making it difficult to read the call to action.

This patch adds the `background-color` property to the
`crl-email-subscription` CSS class so that it is used when the
background image fails to load.

Fixes: #105740
Part of: CRDB-29170

Release note (ui change): The overview page now correctly renders the
background color for the email signup, fixing an issue where it was
difficult to read the text.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-cluster-observability Related to cluster observability C-bug Code not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior. E-quick-win Likely to be a quick win for someone experienced.
Projects
No open projects
Development

Successfully merging a pull request may close this issue.

2 participants