Skip to content

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.

3 participants