feat: Remove QUESTION comments

Workday · Jan 3, 2024 · 0b26b11 · 0b26b11
1 parent dff6269
commit 0b26b11
Show file tree

Hide file tree

Showing 6 changed files with 3 additions and 11 deletions.
diff --git a/modules/react/loading-sparkles/README.md b/modules/react/loading-sparkles/README.md
@@ -3,5 +3,3 @@
 View the
 [documentation for Loading Sparkles](https://workday.github.io/canvas-kit/?path=/docs/components-indicators-loading-sparkles-react--basic)
 on Storybook.
-
-<!-- QUESTION: Do we want to add any reference to usage of AI-related components? -->
diff --git a/modules/react/loading-sparkles/lib/LoadingSparkles.tsx b/modules/react/loading-sparkles/lib/LoadingSparkles.tsx
@@ -145,11 +145,11 @@ const Sparkle = ({animationDelay = 0}: SparkleProps) => (
  */
 const Container = styled('div')({
   display: 'inline-flex',
-  gap: '1px', // QUESTION: is there an appropriate rem or space value for this?
+  gap: '1px',
 });
 
 /**
- * A simple component that displays three horizontal sparkles, to be used when an AI operation is loading.
+ * A simple component that displays three horizontal sparkles, to be used when an AI operation is in progress.
  */
 export const LoadingSparkles = (props: React.HTMLAttributes<HTMLDivElement>) => {
   return (

diff --git a/modules/react/loading-sparkles/lib/sparkleStage0Icon.tsx b/modules/react/loading-sparkles/lib/sparkleStage0Icon.tsx
@@ -1,4 +1,3 @@
-// QUESTION: Should these icons be included in @workday/canvas-system-icons-web instead?
 import {CanvasSystemIcon, CanvasIconTypes} from '@workday/design-assets-types';
 
 export const sparkleStage0Icon: CanvasSystemIcon = {

diff --git a/modules/react/loading-sparkles/lib/sparkleStage1Icon.tsx b/modules/react/loading-sparkles/lib/sparkleStage1Icon.tsx
@@ -1,4 +1,3 @@
-// QUESTION: Should these icons be included in @workday/canvas-system-icons-web instead?
 import {CanvasSystemIcon, CanvasIconTypes} from '@workday/design-assets-types';
 
 export const sparkleStage1Icon: CanvasSystemIcon = {

diff --git a/modules/react/loading-sparkles/lib/sparkleStage2Icon.tsx b/modules/react/loading-sparkles/lib/sparkleStage2Icon.tsx
@@ -1,4 +1,3 @@
-// QUESTION: Should these icons be included in @workday/canvas-system-icons-web instead?
 import {CanvasSystemIcon, CanvasIconTypes} from '@workday/design-assets-types';
 
 export const sparkleStage2Icon: CanvasSystemIcon = {

diff --git a/modules/react/loading-sparkles/stories/LoadingSparkles.stories.mdx b/modules/react/loading-sparkles/stories/LoadingSparkles.stories.mdx
@@ -8,10 +8,7 @@ import {Accessible} from './examples/Accessible';
 
 # Canvas Kit Loading Sparkles
 
-Loading Sparkles make users aware that content is currently being loaded, processing, or that change
-will occur on the page.
-
-<!-- QUESTION: Do we want to add any reference to usage of AI-related components? -->
+Loading Sparkles make users aware that an AI operation is in progress.
 
 ## Installation