From b510032246b8bc1ba3859370998e449572c82e55 Mon Sep 17 00:00:00 2001 From: Mert Akinc <7282195+m-akinc@users.noreply.github.com> Date: Thu, 14 Mar 2024 11:42:00 -0500 Subject: [PATCH] Improve comments --- .../src/number-field/tests/number-field-matrix.stories.ts | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/packages/nimble-components/src/number-field/tests/number-field-matrix.stories.ts b/packages/nimble-components/src/number-field/tests/number-field-matrix.stories.ts index 8077450a3a..fa41777306 100644 --- a/packages/nimble-components/src/number-field/tests/number-field-matrix.stories.ts +++ b/packages/nimble-components/src/number-field/tests/number-field-matrix.stories.ts @@ -88,9 +88,9 @@ export const numberFieldInteractionsThemeMatrix: StoryFn = createMatrixThemeStor [ interactionStates, disabledStates, - hideStepStates.filter(x => x[0] !== 'Hide Step'), // always show inc/dec buttons - valueStates.filter(x => x[0] !== 'Placeholder'), // don't test placeholder text - errorStates.filter(x => x[0] !== 'Error No Message'), // don't test error state w/o error text + hideStepStates.filter(x => x[0] !== 'Hide Step'), // always show inc/dec buttons (no need to test without) + valueStates.filter(x => x[0] !== 'Placeholder'), // value states shouldn't affect styling, so just test one (non-placeholder) + errorStates.filter(x => x[0] !== 'Error No Message'), // with or without message shouldn't matter, so just test with message appearanceStates ], disabledInteractionsFilter