use ordered list to show the boot order in expanded list

openshift · Nov 26, 2019 · 2c8629f · 2c8629f
1 parent 8aca2f3
commit 2c8629f
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/...end/packages/kubevirt-plugin/src/components/boot-order/boot-order-summary-empty-state.tsx b/...end/packages/kubevirt-plugin/src/components/boot-order/boot-order-summary-empty-state.tsx
@@ -14,7 +14,9 @@ export const BootOrderSummaryEmptyState = ({ devices }: BootOrderSummaryEmptySta
     setIsExpanded(!isExpanded);
   };
 
-  // Note(Yaacov): className='text-secondary' is a hack to fix TextVariants being overriden.
+  // Note(Yaacov):
+  // className='text-secondary' is a hack to fix TextVariants being overriden.
+  // Using <ol> because '@patternfly/react-core' <List> currently miss isOrder parameter.
   return (
     <>
       <Text component={TextVariants.p} className="kubevirt-boot-order-summary__empty-text">
@@ -30,11 +32,11 @@ export const BootOrderSummaryEmptyState = ({ devices }: BootOrderSummaryEmptySta
           isExpanded={isExpanded}
           className="kubevirt-boot-order-summary__expandable"
         >
-          <List>
+          <ol>
             {options.map((option) => (
               <ListItem key={deviceKey(option)}>{deviceLabel(option)}</ListItem>
             ))}
-          </List>
+          </ol>
         </Expandable>
       )}
     </>