-
Notifications
You must be signed in to change notification settings - Fork 109
Open
Description
Hi,
To what it’s worth this is not really crucial since the css file is pretty small but 🤷♂️
By looking around I found that in the kyt-universal-starter
package that the shared-styles
are being imported but not used.
You can see the file right here 👇 https://github.com/NYTimes/kyt/blob/4121828cf1c8061a90f1ed9f69979331bc4a02e1/packages/kyt-starter-universal/starter-src/src/components/Tools/styles.scss#L2-L6
I think that the $padding
variable got forgotten.
One of this things can be done.
- Use the variable
$padding
on thepadding-top
property (Values are differents) - In the case that the value must be
5px
, one can create another value on theshared-styles
likesmall-padding
- Remove the import and leave the property with its current value
Metadata
Metadata
Assignees
Labels
No labels