rusackas commented on a change in pull request #18988:
URL: https://github.com/apache/superset/pull/18988#discussion_r829409939
##########
File path: superset-frontend/src/components/FilterableTable/FilterableTable.tsx
##########
@@ -95,6 +100,7 @@ const StyledFilterableTable = styled.div`
// when more than MAX_COLUMNS_FOR_TABLE are returned, switch from table to
grid view
export const MAX_COLUMNS_FOR_TABLE = 50;
+export const MAX_COLUMN_WIDTH = 200;
Review comment:
The only downside to the AntD approach is a little more vendor lock-in,
should we ever decide to change component libraries again... but it probably
won't be our biggest battle then :)
The `getTextDimension` approach seems like it'd work, but is slightly
fragile-looking to me as a technique to start reusing in more places.
--
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]
