@jobber/components-native 0.105.1-MIKEfix-c-fe51c3e.2 → 0.105.1

package/dist/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@jobber/components-native",
-    "version": "0.105.1-MIKEfix-c-fe51c3e.2+fe51c3e5",
+    "version": "0.105.1",
     "license": "MIT",
     "description": "React Native implementation of Atlantis",
     "repository": {
@@ -122,5 +122,5 @@
         "react-native-screens": ">=4.18.0",
         "react-native-svg": ">=12.0.0"
     },
-    "gitHead": "fe51c3e523604cb28481aa3e483371721dac684b"
+    "gitHead": "fff8aa8019d89e4a0d517f1eb963c8442935a16d"
 }

package/dist/src/ActivityIndicator/ActivityIndicator.js

@@ -1,5 +1,5 @@
 import React, { useEffect } from "react";
-import { AccessibilityInfo, StyleSheet, View } from "react-native";
+import { AccessibilityInfo } from "react-native";
 import Animated, { Easing, ReduceMotion, useAnimatedProps, useAnimatedStyle, useReducedMotion, useSharedValue, withRepeat, withSequence, withTiming, } from "react-native-reanimated";
 import Svg, { Circle } from "react-native-svg";
 import { tokens } from "../utils/design";
@@ -19,9 +19,6 @@ const RADIUS = 20;
 // "200" used as the gap value exceeds the circumference, ensuring only
 // one stroke segment is visible at a time.
 const GAP = 200;
-const ContainerStyle = StyleSheet.create({
-    container: { justifyContent: "center", alignItems: "center" },
-});
 function resolveLegacySize(size) {
     if (size === "large") {
         if (__DEV__) {
@@ -126,18 +123,12 @@ function FullMotionRing({ pixelSize, strokeWidth, arcColor, trackColor, style, t
             strokeDashoffset: offset,
         };
     });
-    return (React.createElement(View, { style: [ContainerStyle.container, style] },
-        React.createElement(Animated.View, Object.assign({ style: [
-                {
-                    width: pixelSize,
-                    height: pixelSize,
-                },
-            ], testID: testID }, a11yProps),
-            React.createElement(Animated.View, { style: [{ width: pixelSize, height: pixelSize }, outerStyle] },
-                React.createElement(Animated.View, { style: [{ width: pixelSize, height: pixelSize }, innerStyle] },
-                    React.createElement(Svg, { width: pixelSize, height: pixelSize, viewBox: `0 0 ${VIEWBOX} ${VIEWBOX}` },
-                        React.createElement(Circle, { cx: CENTER, cy: CENTER, r: RADIUS, fill: "none", stroke: trackColor, strokeWidth: strokeWidth }),
-                        React.createElement(AnimatedCircle, { cx: CENTER, cy: CENTER, r: RADIUS, fill: "none", stroke: arcColor, strokeWidth: strokeWidth, strokeLinecap: "round", animatedProps: arcAnimatedProps })))))));
+    return (React.createElement(Animated.View, Object.assign({ style: [{ width: pixelSize, height: pixelSize }, style], testID: testID }, a11yProps),
+        React.createElement(Animated.View, { style: [{ width: pixelSize, height: pixelSize }, outerStyle] },
+            React.createElement(Animated.View, { style: [{ width: pixelSize, height: pixelSize }, innerStyle] },
+                React.createElement(Svg, { width: pixelSize, height: pixelSize, viewBox: `0 0 ${VIEWBOX} ${VIEWBOX}` },
+                    React.createElement(Circle, { cx: CENTER, cy: CENTER, r: RADIUS, fill: "none", stroke: trackColor, strokeWidth: strokeWidth }),
+                    React.createElement(AnimatedCircle, { cx: CENTER, cy: CENTER, r: RADIUS, fill: "none", stroke: arcColor, strokeWidth: strokeWidth, strokeLinecap: "round", animatedProps: arcAnimatedProps }))))));
 }
 function ReducedMotionRing({ pixelSize, strokeWidth, ringColor, style, testID, a11yProps, }) {
     // Reduced motion: hide rotation entirely, render one static ring in the
@@ -173,8 +164,7 @@ function ReducedMotionRing({ pixelSize, strokeWidth, ringColor, style, testID, a
     const animatedStyle = useAnimatedStyle(() => ({
         opacity: pulse.value,
     }));
-    return (React.createElement(View, { style: [ContainerStyle.container, style] },
-        React.createElement(Animated.View, Object.assign({ style: [{ width: pixelSize, height: pixelSize }, animatedStyle], testID: testID }, a11yProps),
-            React.createElement(Svg, { width: pixelSize, height: pixelSize, viewBox: `0 0 ${VIEWBOX} ${VIEWBOX}` },
-                React.createElement(Circle, { cx: CENTER, cy: CENTER, r: RADIUS, fill: "none", stroke: ringColor, strokeWidth: strokeWidth })))));
+    return (React.createElement(Animated.View, Object.assign({ style: [{ width: pixelSize, height: pixelSize }, animatedStyle, style], testID: testID }, a11yProps),
+        React.createElement(Svg, { width: pixelSize, height: pixelSize, viewBox: `0 0 ${VIEWBOX} ${VIEWBOX}` },
+            React.createElement(Circle, { cx: CENTER, cy: CENTER, r: RADIUS, fill: "none", stroke: ringColor, strokeWidth: strokeWidth }))));
 }

package/dist/src/Icon/Icon.js

@@ -12,7 +12,8 @@ export function Icon({ name, color, size = "base", customColor, testID, }) {
         format: "js",
         tokens,
     });
-    const icon = paths.map((path) => {
+    // getIcon returns undefined paths for an unknown icon name; render nothing rather than crash.
+    const icon = (paths !== null && paths !== void 0 ? paths : []).map((path) => {
         return React.createElement(Path, { key: path, d: path, fill: customColor || svgStyle.fill });
     });
     return (React.createElement(Svg, { style: Object.assign(Object.assign({}, svgStyle), { display: "flex" }), testID: testID || name, viewBox: viewBox }, icon));

package/dist/src/Icon/Icon.test.js

@@ -41,6 +41,10 @@ it("applies name prop as testID when testID prop is not provided", () => {
     const { getByTestId } = render(React.createElement(Icon, { name: "home" }));
     expect(getByTestId("home")).toBeDefined();
 });
+it("renders an empty svg without throwing for an unknown icon name", () => {
+    const svg = render(React.createElement(Icon, { name: "someNonexistentIcon" }));
+    expect(svg.UNSAFE_root.findAllByType(Path)).toHaveLength(0);
+});
 describe("theme-aware fill", () => {
     function getPathFills(svg) {
         return svg.UNSAFE_root.findAllByType(Path).map(p => p.props.fill);