[CCUBE-895][RL] Update section headers, rename data test-id prop

LifeSG · Feb 2, 2024 · 8fa1d06 · 8fa1d06
1 parent 9dbd15d
commit 8fa1d06
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 5 deletions.
diff --git a/src/form/form-label-addon.tsx b/src/form/form-label-addon.tsx
@@ -19,9 +19,9 @@ export const PopoverAddon = ({
     rootNode,
 }: PopoverAddonProps): JSX.Element => {
     // =========================================================================
-    // CONST, STATE, RE
+    // CONST, STATE, REF
     // =========================================================================
-    const { content, type, icon, id, "data-testid": dataTestid } = addon;
+    const { content, type, icon, id, "data-testid": testId } = addon;
 
     // =========================================================================
     // RENDER FUNCTION
@@ -38,7 +38,7 @@ export const PopoverAddon = ({
         <PopoverTrigger
             trigger="click"
             id={id}
-            data-testid={dataTestid}
+            data-testid={testId}
             popoverContent={content}
             rootNode={rootNode}
         >

diff --git a/src/popover-v2/popover-trigger.tsx b/src/popover-v2/popover-trigger.tsx
@@ -22,6 +22,9 @@ export const PopoverTrigger = ({
     onPopoverDismiss,
     ...otherProps
 }: PopoverV2TriggerProps) => {
+    // =========================================================================
+    // CONST, STATE, REF
+    // =========================================================================
     const [visible, setVisible] = useState<boolean>(false);
     const nodeRef = useRef<HTMLDivElement>();
     const isMobile = useMediaQuery({
@@ -34,9 +37,9 @@ export const PopoverTrigger = ({
         middleware: [offset(16), flip(), shift()],
     });
 
-    // =============================================================================
+    // =========================================================================
     // EFFECTS
-    // =============================================================================
+    // =========================================================================
     useEffect(() => {
         // NOTE: Do not add mouse down event if it's mobile
         if (isMobile || !visible) {