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

MDCMigration: Style run table and right pane checkboxes. #6615

Merged
merged 1 commit into from
Oct 3, 2023

Conversation

bmd3k
Copy link
Contributor

@bmd3k bmd3k commented Oct 2, 2023

Restyle the checkboxes in both the runs table and right pane.

For runs table this is mostly just about removing and adjusting padding.

For right pane this:

  • adds negative margin to account for checkbox padding - aligning the checkboxes with other elements in the pane.
  • adjusting the text of checkbox labels to look more like the current style.

Here are some sample screenshots but please patch it in locally to play with it:

image

image

@bmd3k bmd3k force-pushed the mdc-migration-checkboxes-02 branch from 9f49dd1 to ab906b2 Compare October 2, 2023 22:01
@bmd3k bmd3k requested a review from rileyajones October 2, 2023 22:02
// Add additional margin between most control elements. The exception is when
// the second control row is a checkbox, in which case we just rely on the
// padding that is already included in the checkbox.
section .control-row:not(:has(+ .control-row > mat-checkbox)):not(:last-child) {
Copy link
Contributor

Choose a reason for hiding this comment

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

😆 this is a wild selector. I can't figure out how to write this as a single sentence without being a runon.
A .control-row that is inside of a section and comes immediately after another .control-row which contains a mat-checkbox and is not the :last-child.

@bmd3k bmd3k merged commit 9cf30c5 into tensorflow:MDCMigration Oct 3, 2023
13 checks passed
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.

2 participants