michael-s-molina commented on a change in pull request #15390:
URL: https://github.com/apache/superset/pull/15390#discussion_r659684471
##########
File path:
superset-frontend/src/dashboard/components/nativeFilters/FilterBar/CascadeFilters/CascadeFilterControl/index.tsx
##########
@@ -53,28 +49,26 @@ const CascadeFilterControl:
React.FC<CascadeFilterControlProps> = ({
onFilterSelectionChange,
}) => (
<>
- <StyledFilterControlBox>
- <StyledCaretIcon name="caret-down" />
- <FilterControl
- dataMaskSelected={dataMaskSelected}
- filter={filter}
- directPathToChild={directPathToChild}
- onFilterSelectionChange={onFilterSelectionChange}
- />
- </StyledFilterControlBox>
-
- <StyledCascadeChildrenList>
+ <FilterControl
+ dataMaskSelected={dataMaskSelected}
+ filter={filter}
+ directPathToChild={directPathToChild}
+ onFilterSelectionChange={onFilterSelectionChange}
+ />
+ <StyledDiv>
{filter.cascadeChildren?.map(childFilter => (
- <li key={childFilter.id}>
+ <>
+ <Icons.CaretDownOutlined iconSize="s" />
Review comment:
I'm not against removing the query, it was my first design choice. The
order can be enough to show hierarchy. About the indentation though, I really
think that is not a good solution to keep inputs with different widths. Also,
it will break if the user adds a bunch o nesting.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
