-
Notifications
You must be signed in to change notification settings - Fork 485
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
fix deep dependency graph not show in uiEmbed=v0 mode #768
Conversation
b9f2d52
to
8a56bc3
Compare
Signed-off-by: leroy-chen <[email protected]>
Codecov Report
@@ Coverage Diff @@
## master #768 +/- ##
==========================================
+ Coverage 95.10% 95.12% +0.02%
==========================================
Files 231 231
Lines 7103 7103
Branches 1709 1752 +43
==========================================
+ Hits 6755 6757 +2
+ Misses 342 340 -2
Partials 6 6
Continue to review full report at Codecov.
|
022919c
to
d359d5e
Compare
add an new classname when use uiEmbed mode Signed-off-by: leroy-chen <[email protected]>
@rubenvp8510 could you review? |
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.
The CI is failing, run yarn prettier
to fix it :)
…the `top: var(--nav-height);` rule.and remove this from class .Page--content. change new added class name to .Page--content--no-embedded,just keep the `top: var(--nav-height);` rule.and remove this from class .Page--content. Signed-off-by: leroy-chen <[email protected]>
bf557c5
to
46737f8
Compare
fix ci lint error Signed-off-by: leroy-chen <[email protected]>
fix deep dependency graph not show in uiEmbed=v0 mode
Which problem is this PR solving?
Short description of the changes