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

alternate background color for table rows #2476

Merged
merged 4 commits into from
Oct 11, 2018

Conversation

daiyam
Copy link
Contributor

@daiyam daiyam commented Oct 8, 2018

This change alternates the background color of the table rows.

screenshot

@ZeroX-DG ZeroX-DG added awaiting review ❇️ Pull request is awaiting a review. awaiting changes 🖊️ Pull request has been reviewed, but contributor needs to make changes. and removed awaiting review ❇️ Pull request is awaiting a review. labels Oct 9, 2018
@ZeroX-DG
Copy link
Member

ZeroX-DG commented Oct 9, 2018

Should we append css to all CodeMirror themes? Or can we just include another separate css file for that?

UPDATE

Sorry, my fingers moved faster than my brain haha 😄

Copy link
Member

@ZeroX-DG ZeroX-DG left a comment

Choose a reason for hiding this comment

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

The color is a bit hard to see on dark theme like 3024 night theme. Can you improve it?

@daiyam
Copy link
Contributor Author

daiyam commented Oct 9, 2018

@ZeroX-DG I've updated how to calculate the alternative color. Now, it's used a mix of the background color with a little bit of the text color.

@ZeroX-DG
Copy link
Member

ZeroX-DG commented Oct 9, 2018

LGTM 👍

@ZeroX-DG ZeroX-DG closed this Oct 9, 2018
@ZeroX-DG ZeroX-DG reopened this Oct 9, 2018
Copy link
Member

@ZeroX-DG ZeroX-DG left a comment

Choose a reason for hiding this comment

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

Wrong button 😄

@ZeroX-DG ZeroX-DG added approved 👍 Pull request has been approved by sufficient reviewers. and removed awaiting changes 🖊️ Pull request has been reviewed, but contributor needs to make changes. labels Oct 9, 2018
@Rokt33r Rokt33r added next release (v0.11.10) and removed approved 👍 Pull request has been approved by sufficient reviewers. labels Oct 11, 2018
@Rokt33r Rokt33r merged commit 851f57c into BoostIO:master Oct 11, 2018
@daiyam daiyam deleted the table-even-odd branch October 11, 2018 08:02
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.

None yet

3 participants