Mirrors/unleash.unleash
1
0
Fork 0
mirror of https://github.com/Unleash/unleash.git synced 2025-07-21 13:47:39 +02:00
Code Issues Projects Releases Wiki Activity

fix: remove separator from first option in list (#872)

Browse Source
This commit is contained in:
olav 2022-04-08 14:02:51 +02:00 committed by GitHub
parent 49b8e7329e
commit df60f2301f
1 changed files with 17 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

27
frontend/src/component/common/ConstraintAccordion/ConstraintOperatorSelect/ConstraintOperatorSelect.tsx
View File

@ -5,6 +5,7 @@ import {
semVerOperators,
dateOperators,
numOperators,
inOperators,
} from 'constants/operators';
import React, { useState, ChangeEvent } from 'react';
import { formatOperatorDescription } from 'component/common/ConstraintAccordion/ConstraintOperator/formatOperatorDescription';
@ -61,7 +62,8 @@ export const ConstraintOperatorSelect = ({
key={operator}
value={operator}
className={classNames(
needSeparatorAbove(operator) && styles.separator
needSeparatorAbove(options, operator) &&
styles.separator
)}
>
<div className={styles.optionContainer}>
@ -77,15 +79,20 @@ export const ConstraintOperatorSelect = ({
);
};
const needSeparatorAbove = (operator: Operator): boolean => {
const groups = [
stringOperators,
numOperators,
dateOperators,
semVerOperators,
];
const needSeparatorAbove = (options: Operator[], option: Operator): boolean => {
if (option === options[0]) {
return false;
}
return groups.some(group => {
return group[0] === operator;
return operatorGroups.some(group => {
return group[0] === option;
});
};
const operatorGroups = [
inOperators,
stringOperators,
numOperators,
dateOperators,
semVerOperators,
];