Skip to content

Commit feeef67

Browse files
committed
Remove EuiDescritpionList IE comments
- Many consumers are overriding `width` instead of `flex-basis` at this point for the component, so changing it to flex-basis would cause unnecessary churn - `width` is also still valid and intuitive to use instead over flex-basis
1 parent 2ea3246 commit feeef67

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

src/components/description_list/description_list_description.styles.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ export const euiDescriptionListDescriptionStyles = (
2222
const { euiTheme } = euiThemeContext;
2323

2424
const columnDisplay = `
25-
${logicalCSS('width', '50%')} // Flex-basis doesn't work in IE with padding
25+
${logicalCSS('width', '50%')}
2626
${logicalCSS('padding-left', euiTheme.size.s)}
2727
&:not(:first-of-type) {
2828
${logicalCSS('margin-top', euiTheme.size.base)}

src/components/description_list/description_list_title.styles.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ export const euiDescriptionListTitleStyles = (euiThemeContext: UseEuiTheme) => {
2121
const { euiTheme, colorMode } = euiThemeContext;
2222

2323
const columnDisplay = `
24-
${logicalCSS('width', '50%')} // Flex-basis doesn't work in IE with padding
24+
${logicalCSS('width', '50%')}
2525
${logicalCSS('padding-right', euiTheme.size.s)}
2626
`;
2727
return {

0 commit comments

Comments
 (0)