-
Notifications
You must be signed in to change notification settings - Fork 3
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
WE-6629: Update styles #14
Conversation
6e67dc0
to
a58536b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
pretty good. just a few small changes.
@@ -20,6 +20,10 @@ | |||
line-height: 32px; | |||
margin-bottom: 0; | |||
} | |||
|
|||
> .h4 { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this seems kinda random. what's the use case?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, good point. I'll update the markup in publisher instead (story template).
line-height: 1.6em; | ||
font-weight: bold; | ||
} | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
let's put this in the legacy folder since the story page is due for a revamp to strictly ember
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed!
@@ -48,6 +48,7 @@ body:not(.simplified_wrapper):not(.container_stripped) > :not( | |||
@import 'slideshow'; | |||
@import 'videoplayer'; | |||
@import 'pagination'; | |||
@import 'story'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
to @import 'legacy/story';
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed.
@@ -522,7 +522,7 @@ var HTML = `<html> | |||
|
|||
|
|||
<div class="media-body"> | |||
<div class="text--small dimmed">Published in</div> | |||
<div class="text--small dimmed">Published by</div> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is an unused file, no need to update it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sounds good.
bde909f
to
8f0d179
Compare
8f0d179
to
839c35b
Compare
839c35b
to
5233737
Compare
https://jira.wnyc.org/browse/WE-6629
The last commit relates to #13 on Publisher.