-
-
Notifications
You must be signed in to change notification settings - Fork 387
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
Comment content styles #1704
Comment content styles #1704
Conversation
@@ -1,129 +0,0 @@ | |||
.raw-content { | |||
& .root_theme_dark { |
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.
Wrong nesting, should've been .root_theme_dark &
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #1704 +/- ##
==========================================
- Coverage 65.20% 61.47% -3.74%
==========================================
Files 141 132 -9
Lines 3512 2998 -514
Branches 824 757 -67
==========================================
- Hits 2290 1843 -447
Misses 1038 1038
+ Partials 184 117 -67 ☔ View full report in Codecov by Sentry. |
size-limit report 📦
|
className={b('comment-form__preview', { | ||
mix: b('raw-content', {}, { theme }), | ||
})} | ||
className="comment-form__preview raw-content" |
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.
theme set to root tag as dark
so we don't need additional theming tag
…ve styles to correct place
64055b6
to
f24d044
Compare
Pull Request Test Coverage Report for Build 7512286614
💛 - Coveralls |
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.
👍
code
block causes content to be hidden #1629pre > code
(the way chroma generates html)pre
had size 85% andcode
had 85% from it.Before
After