-
Notifications
You must be signed in to change notification settings - Fork 487
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
ClayLink styles #396
Comments
@carloslancha There is no |
@pat270 since lexicon defines those two link styles ClayLink component should be able to provide it (Imagine for example that Collaboration team wants to use the component in blogs, out of the rest of components we're implementing) |
This is exactly the use case I'm trying to prevent. From a CSS framework perspective having generic color classes scattered throughout an application lose meaning once you start deviating from whatever the base themes styles are. |
But it's not the same like, for example, |
Hey @pat270, I think the question is... how do we implement the different types of links right now? How do we produce: Primary Text Alone Link Secondary Text Alone Link |
@jbalsas, Right now, there is no generic class to style the links that way. I would be ok with adding generic link styles if they were named something like:
The ones below shouldn't be used in components that already have link styles (pretty much every Clay component, I missed some that need to be fixed)
If component teams want different font-size, margin, or styles in general for their title/subtitle, set it using a css selector (
In the screenshot, Juan Anton Torres user name should use markup:
Read more should use:
|
As Lexicon defines there are two link styles, primary and secondary
https://lexicondesign.io/docs/patterns/link.html
Do we have classes for that?
The text was updated successfully, but these errors were encountered: