Skip to content

Conversation

@JamesHollyer
Copy link
Contributor

@JamesHollyer JamesHollyer commented Oct 4, 2023

Motivation for features / changes

Make the readme link mat-icon look like all the other buttons by forcing the color to be white instead of the color of the anchor tag. Note: the header keeps all the same colors in dark mode.

Screenshots of UI changes (or N/A)

Before this change:
Screenshot 2023-10-04 at 3 54 40 PM

After this change:
Screenshot 2023-10-04 at 3 54 18 PM

}

.readme mat-icon {
color: #fff;
Copy link
Contributor

Choose a reason for hiding this comment

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

Could you read this from the palette instead?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I am trying to stay consistent with the other buttons.

Copy link
Contributor

Choose a reason for hiding this comment

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

I believe I took care of this in #6620.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oh cool. Thanks for catching this!

@JamesHollyer
Copy link
Contributor Author

This was done in #6620.

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