fix deep dependency graph not show in uiEmbed=v0 mode (jaegertracing#768

) * fix issue jaegertracing#719 Signed-off-by: leroy-chen <leroy.chen.179@gmail.com> * add an new classname when use uiEmbed mode add an new classname when use uiEmbed mode Signed-off-by: leroy-chen <leroy.chen.179@gmail.com> * 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. 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 <leroy.chen.179@gmail.com> * fix ci lint error fix ci lint error Signed-off-by: leroy-chen <leroy.chen.179@gmail.com> Co-authored-by: Ruben Vargas <ruben.vp8510@gmail.com>
th3M1ke · Oct 14, 2021 · 56c2911 · 56c2911
1 parent e531c04
commit 56c2911
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/packages/jaeger-ui/src/components/App/Page.css b/packages/jaeger-ui/src/components/App/Page.css
@@ -29,5 +29,8 @@ limitations under the License.
   min-height: calc(100% - var(--nav-height));
   position: absolute;
   right: 0;
+}
+
+.Page--content--no-embedded {
   top: var(--nav-height);
 }
diff --git a/packages/jaeger-ui/src/components/App/Page.tsx b/packages/jaeger-ui/src/components/App/Page.tsx
@@ -52,7 +52,7 @@ export class PageImpl extends React.Component<TProps> {
 
   render() {
     const { embedded } = this.props;
-    const contentCls = cx({ 'Page--content': !embedded });
+    const contentCls = cx({ 'Page--content': true, 'Page--content--no-embedded': !embedded });
     return (
       <div>
         <Helmet title="Jaeger UI" />