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

Refactor Material Theme #178

Merged
merged 1 commit into from
Aug 6, 2018
Merged

Refactor Material Theme #178

merged 1 commit into from
Aug 6, 2018

Conversation

sc1f
Copy link
Contributor

@sc1f sc1f commented Aug 1, 2018

  • split out common colors & fonts into variables.less
  • restructure and refactor material.less & material.dark.less for readability and reduction of duplicate rule declaration.

@timkpaine
Copy link
Member

Copy link
Member

@texodus texodus left a comment

Choose a reason for hiding this comment

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

Small tweak


#divider {
display: none;
}
}
Copy link
Member

Choose a reason for hiding this comment

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

These seem to revert a commit I made on responsive-sidebar before merging, to consolidate .columns_horizontal.

Copy link
Member

Choose a reason for hiding this comment

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

Nevermind - I misread, this works as intended.

Copy link
Member

Choose a reason for hiding this comment

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

I didnt know you could do this...

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Changes fixed.


perspective-row:hover {
Copy link
Member

Choose a reason for hiding this comment

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

Might be worth putting style rules for perspective-computed-column in a future commit rather than here.

@sc1f sc1f self-assigned this Aug 2, 2018
@sc1f
Copy link
Contributor Author

sc1f commented Aug 2, 2018

@timkpaine the Jlab css looks exactly like the compiled version of the .less files in perspective-viewer. The compilation of less > css allows me to do the refactoring because I can reduce bulk through the less syntax, but I can't necessarily do that in straight CSS. We could try to set up .less compilation for Jlab but that seems to be outside the scope of this PR.

@sc1f
Copy link
Contributor Author

sc1f commented Aug 2, 2018

@texodus looks like the travis/push build broke on the flaky perspective-hypergrid test. Is there a way to manually run that check again without pushing to the repo/updating the PR?

@timkpaine
Copy link
Member

@sc1f I reran the build, passed now.

@texodus texodus merged commit c8c1bf8 into master Aug 6, 2018
@texodus
Copy link
Member

texodus commented Aug 6, 2018

Looks good, thanks!

@texodus texodus deleted the material-refactor branch August 8, 2018 05:09
texodus pushed a commit that referenced this pull request May 13, 2019
Treat negative sunburst size values as zero
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