Skip to content
This repository was archived by the owner on May 12, 2026. It is now read-only.

alternate background color for table rows#2476

Merged
Rokt33r merged 4 commits into
BoostIO:masterfrom
daiyam:table-even-odd
Oct 11, 2018
Merged

alternate background color for table rows#2476
Rokt33r merged 4 commits into
BoostIO:masterfrom
daiyam:table-even-odd

Conversation

@daiyam

@daiyam daiyam commented Oct 8, 2018

Copy link
Copy Markdown
Contributor

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

ZeroX-DG commented Oct 9, 2018

Copy link
Copy Markdown
Member

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 😄

@ZeroX-DG ZeroX-DG left a comment

Copy link
Copy Markdown
Member

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

daiyam commented Oct 9, 2018

Copy link
Copy Markdown
Contributor Author

@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

ZeroX-DG commented Oct 9, 2018

Copy link
Copy Markdown
Member

LGTM 👍

@ZeroX-DG ZeroX-DG closed this Oct 9, 2018
@ZeroX-DG ZeroX-DG reopened this Oct 9, 2018

@ZeroX-DG ZeroX-DG left a comment

Copy link
Copy Markdown
Member

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 subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

3 participants