Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix(clockface): remove same color used by text from the background in IndexList body #489

Merged
merged 3 commits into from
Apr 15, 2020

Conversation

TCL735
Copy link
Contributor

@TCL735 TCL735 commented Apr 15, 2020

Closes influxdata/influxdb#17635
Closes #488

Changes

Remove the background color which is the same value as the text color. This applies to IndexList body when it is empty.

Screenshots

BEFORE: With text and background color the same
Screen Shot 2020-04-15 at 2 49 13 PM

AFTER: Correct background color and text color
Screen Shot 2020-04-15 at 3 08 38 PM

Checklist

Check all that apply

  • Updated documentation to reflect changes
  • Added entry to top of Changelog with link to PR (not issue)
  • Tests pass
  • Peer reviewed and approved
  • Signed CLA (if not already signed)

@TCL735 TCL735 requested review from alexpaxton and mavarius April 15, 2020 21:53
Copy link
Collaborator

@mavarius mavarius left a comment

Choose a reason for hiding this comment

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

I think the text color was used where the background color was supposed to be.

@@ -154,7 +154,6 @@ $cf-index-list--sort-text: $g11-sidewalk;

// Empty state
.cf-index-list--empty-cell {
background-color: $cf-card-text--disabled;
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think this was supposed to be $cf-card-background--disabled

Copy link
Contributor

Choose a reason for hiding this comment

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

@mavarius good catch

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks guys! I have fixed it. Screenshot updated as well.

@alexpaxton
Copy link
Contributor

MASSIVE

Copy link
Collaborator

@mavarius mavarius left a comment

Choose a reason for hiding this comment

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

Good to go!

@TCL735 TCL735 merged commit 408c87c into master Apr 15, 2020
@TCL735 TCL735 deleted the fix_17635 branch April 15, 2020 22:15
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

IndexListBody is bright grey when empty Empty Run Logs looks gray
3 participants