dididy commented on code in PR #5079:
URL: https://github.com/apache/zeppelin/pull/5079#discussion_r2382885465


##########
zeppelin-web-angular/src/app/pages/workspace/notebook/action-bar/action-bar.component.less:
##########
@@ -113,3 +113,22 @@
     }
   }
 });
+
+.search-code {
+  padding: 0;
+
+  &-row {
+    display: flex;
+
+    .ant-input-group,
+    .ant-input {
+      height: 32px;
+    }
+
+    .search-row-btn-group {
+    display: flex;
+    align-items: center;
+    flex-wrap: nowrap;
+    }
+  }
+}

Review Comment:
   I considered switching to a table structure and made several attempts, but 
it turned out to be difficult in many ways. Since the code you wrote works 
well, I think it’s fine to keep it as is. However, in your code, the borders 
overlap where elements are nested, which makes some parts appear as 2px 
borders. To resolve this, it would be good to add the following code under 
`&-row`.
   
   ```less
       nz-input-group + nz-button-group {
         margin-left: -1px;
       }
   
       & ~ & {
         margin-top: -1px;
       }
   ```



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscr...@zeppelin.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to