borderRadius is passed to image style (#3148)

adids1221 · web-flow · commit 5fa1a406ec91 · 2024-06-25T15:11:15.000+03:00
diff --git a/demo/src/screens/__tests__/__snapshots__/AvatarScreen.spec.js.snap b/demo/src/screens/__tests__/__snapshots__/AvatarScreen.spec.js.snap
@@ -944,6 +944,7 @@ exports[`AvatarScreen renders screen 1`] = `
               undefined,
               undefined,
               undefined,
+              undefined,
               false,
               [
                 {
@@ -1199,6 +1200,7 @@ exports[`AvatarScreen renders screen 1`] = `
               undefined,
               undefined,
               undefined,
+              undefined,
               false,
               [
                 {
@@ -1467,6 +1469,7 @@ exports[`AvatarScreen renders screen 1`] = `
                 undefined,
                 undefined,
                 undefined,
+                undefined,
                 false,
                 [
                   [
@@ -1659,6 +1662,7 @@ exports[`AvatarScreen renders screen 1`] = `
               undefined,
               undefined,
               undefined,
+              undefined,
               false,
               [
                 {
@@ -1914,6 +1918,7 @@ exports[`AvatarScreen renders screen 1`] = `
               undefined,
               undefined,
               undefined,
+              undefined,
               false,
               [
                 {
@@ -2039,6 +2044,7 @@ exports[`AvatarScreen renders screen 1`] = `
                   undefined,
                   undefined,
                   undefined,
+                  undefined,
                   false,
                   {
                     "backgroundColor": "#FFC50D",
@@ -2213,6 +2219,7 @@ exports[`AvatarScreen renders screen 1`] = `
               undefined,
               undefined,
               undefined,
+              undefined,
               false,
               [
                 {
@@ -2441,6 +2448,7 @@ exports[`AvatarScreen renders screen 1`] = `
               undefined,
               undefined,
               undefined,
+              undefined,
               false,
               [
                 {
@@ -2669,6 +2677,7 @@ exports[`AvatarScreen renders screen 1`] = `
               undefined,
               undefined,
               undefined,
+              undefined,
               false,
               [
                 {
@@ -2855,6 +2864,7 @@ exports[`AvatarScreen renders screen 1`] = `
               undefined,
               undefined,
               undefined,
+              undefined,
               false,
               [
                 {
diff --git a/demo/src/screens/__tests__/__snapshots__/TextFieldScreen.spec.js.snap b/demo/src/screens/__tests__/__snapshots__/TextFieldScreen.spec.js.snap
@@ -2770,6 +2770,7 @@ exports[`TextField Screen renders screen 1`] = `
                     undefined,
                     undefined,
                     undefined,
+                    undefined,
                     false,
                     [
                       {
@@ -3443,6 +3444,7 @@ exports[`TextField Screen renders screen 1`] = `
                   undefined,
                   undefined,
                   undefined,
+                  undefined,
                   false,
                   [
                     {
diff --git a/src/components/button/__tests__/__snapshots__/index.spec.js.snap b/src/components/button/__tests__/__snapshots__/index.spec.js.snap
@@ -1134,6 +1134,7 @@ exports[`Button container size should have no padding of button is an icon butto
         undefined,
         undefined,
         undefined,
+        undefined,
         false,
         [
           {
@@ -2266,6 +2267,7 @@ exports[`Button icon should apply color on icon 1`] = `
         undefined,
         undefined,
         undefined,
+        undefined,
         false,
         [
           {
@@ -2342,6 +2344,7 @@ exports[`Button icon should apply color on icon 2`] = `
         undefined,
         undefined,
         undefined,
+        undefined,
         false,
         [
           {
@@ -2418,6 +2421,7 @@ exports[`Button icon should include custom iconStyle provided as a prop 1`] = `
         undefined,
         undefined,
         undefined,
+        undefined,
         false,
         [
           {
@@ -2499,6 +2503,7 @@ exports[`Button icon should return icon style according to different variations
         undefined,
         undefined,
         undefined,
+        undefined,
         false,
         [
           {
@@ -2575,6 +2580,7 @@ exports[`Button icon should return icon style according to different variations
         undefined,
         undefined,
         undefined,
+        undefined,
         false,
         [
           {
@@ -2651,6 +2657,7 @@ exports[`Button icon should return icon style according to different variations
         undefined,
         undefined,
         undefined,
+        undefined,
         false,
         [
           {
@@ -3901,6 +3908,7 @@ exports[`Button labelColor should return undefined color if this is an icon butt
         undefined,
         undefined,
         undefined,
+        undefined,
         false,
         [
           {
diff --git a/src/components/image/index.tsx b/src/components/image/index.tsx
@@ -233,6 +233,7 @@ class Image extends PureComponent<Props, State> {
       customOverlayContent,
       modifiers,
       recorderTag,
+      borderRadius,
       ...others
     } = this.props;
     const shouldFlipRTL = supportRTL && Constants.isRTL;
@@ -247,6 +248,7 @@ class Image extends PureComponent<Props, State> {
           shouldFlipRTL && styles.rtlFlipped,
           width && {width},
           height && {height},
+          borderRadius && {borderRadius},
           cover && styles.coverImage,
           this.isGif() && styles.gifImage,
           aspectRatio && {aspectRatio},
@@ -270,7 +272,7 @@ class Image extends PureComponent<Props, State> {
             intensity={overlayIntensity}
             color={overlayColor}
             customContent={customOverlayContent}
-            borderRadius={others?.borderRadius}
+            borderRadius={borderRadius}
           />
         )}
       </ImageView>

Original file line number	Diff line number	Diff line change
@@ -944,6 +944,7 @@ exports[`AvatarScreen renders screen 1`] = `
`944`	`944`	`undefined,`
`945`	`945`	`undefined,`
`946`	`946`	`undefined,`
	`947`	`+ undefined,`
`947`	`948`	`false,`
`948`	`949`	`[`
`949`	`950`	`{`
@@ -1199,6 +1200,7 @@ exports[`AvatarScreen renders screen 1`] = `
`1199`	`1200`	`undefined,`
`1200`	`1201`	`undefined,`
`1201`	`1202`	`undefined,`
	`1203`	`+ undefined,`
`1202`	`1204`	`false,`
`1203`	`1205`	`[`
`1204`	`1206`	`{`
@@ -1467,6 +1469,7 @@ exports[`AvatarScreen renders screen 1`] = `
`1467`	`1469`	`undefined,`
`1468`	`1470`	`undefined,`
`1469`	`1471`	`undefined,`
	`1472`	`+ undefined,`
`1470`	`1473`	`false,`
`1471`	`1474`	`[`
`1472`	`1475`	`[`
@@ -1659,6 +1662,7 @@ exports[`AvatarScreen renders screen 1`] = `
`1659`	`1662`	`undefined,`
`1660`	`1663`	`undefined,`
`1661`	`1664`	`undefined,`
	`1665`	`+ undefined,`
`1662`	`1666`	`false,`
`1663`	`1667`	`[`
`1664`	`1668`	`{`
@@ -1914,6 +1918,7 @@ exports[`AvatarScreen renders screen 1`] = `
`1914`	`1918`	`undefined,`
`1915`	`1919`	`undefined,`
`1916`	`1920`	`undefined,`
	`1921`	`+ undefined,`
`1917`	`1922`	`false,`
`1918`	`1923`	`[`
`1919`	`1924`	`{`
@@ -2039,6 +2044,7 @@ exports[`AvatarScreen renders screen 1`] = `
`2039`	`2044`	`undefined,`
`2040`	`2045`	`undefined,`
`2041`	`2046`	`undefined,`
	`2047`	`+ undefined,`
`2042`	`2048`	`false,`
`2043`	`2049`	`{`
`2044`	`2050`	`"backgroundColor": "#FFC50D",`
@@ -2213,6 +2219,7 @@ exports[`AvatarScreen renders screen 1`] = `
`2213`	`2219`	`undefined,`
`2214`	`2220`	`undefined,`
`2215`	`2221`	`undefined,`
	`2222`	`+ undefined,`
`2216`	`2223`	`false,`
`2217`	`2224`	`[`
`2218`	`2225`	`{`
@@ -2441,6 +2448,7 @@ exports[`AvatarScreen renders screen 1`] = `
`2441`	`2448`	`undefined,`
`2442`	`2449`	`undefined,`
`2443`	`2450`	`undefined,`
	`2451`	`+ undefined,`
`2444`	`2452`	`false,`
`2445`	`2453`	`[`
`2446`	`2454`	`{`
@@ -2669,6 +2677,7 @@ exports[`AvatarScreen renders screen 1`] = `
`2669`	`2677`	`undefined,`
`2670`	`2678`	`undefined,`
`2671`	`2679`	`undefined,`
	`2680`	`+ undefined,`
`2672`	`2681`	`false,`
`2673`	`2682`	`[`
`2674`	`2683`	`{`
@@ -2855,6 +2864,7 @@ exports[`AvatarScreen renders screen 1`] = `
`2855`	`2864`	`undefined,`
`2856`	`2865`	`undefined,`
`2857`	`2866`	`undefined,`
	`2867`	`+ undefined,`
`2858`	`2868`	`false,`
`2859`	`2869`	`[`
`2860`	`2870`	`{`
Original file line number	Diff line number	Diff line change
@@ -2770,6 +2770,7 @@ exports[`TextField Screen renders screen 1`] = `
`2770`	`2770`	`undefined,`
`2771`	`2771`	`undefined,`
`2772`	`2772`	`undefined,`
	`2773`	`+ undefined,`
`2773`	`2774`	`false,`
`2774`	`2775`	`[`
`2775`	`2776`	`{`
@@ -3443,6 +3444,7 @@ exports[`TextField Screen renders screen 1`] = `
`3443`	`3444`	`undefined,`
`3444`	`3445`	`undefined,`
`3445`	`3446`	`undefined,`
	`3447`	`+ undefined,`
`3446`	`3448`	`false,`
`3447`	`3449`	`[`
`3448`	`3450`	`{`
Original file line number	Diff line number	Diff line change
@@ -1134,6 +1134,7 @@ exports[`Button container size should have no padding of button is an icon butto
`1134`	`1134`	`undefined,`
`1135`	`1135`	`undefined,`
`1136`	`1136`	`undefined,`
	`1137`	`+ undefined,`
`1137`	`1138`	`false,`
`1138`	`1139`	`[`
`1139`	`1140`	`{`
@@ -2266,6 +2267,7 @@ exports[`Button icon should apply color on icon 1`] = `
`2266`	`2267`	`undefined,`
`2267`	`2268`	`undefined,`
`2268`	`2269`	`undefined,`
	`2270`	`+ undefined,`
`2269`	`2271`	`false,`
`2270`	`2272`	`[`
`2271`	`2273`	`{`
@@ -2342,6 +2344,7 @@ exports[`Button icon should apply color on icon 2`] = `
`2342`	`2344`	`undefined,`
`2343`	`2345`	`undefined,`
`2344`	`2346`	`undefined,`
	`2347`	`+ undefined,`
`2345`	`2348`	`false,`
`2346`	`2349`	`[`
`2347`	`2350`	`{`
@@ -2418,6 +2421,7 @@ exports[`Button icon should include custom iconStyle provided as a prop 1`] = `
`2418`	`2421`	`undefined,`
`2419`	`2422`	`undefined,`
`2420`	`2423`	`undefined,`
	`2424`	`+ undefined,`
`2421`	`2425`	`false,`
`2422`	`2426`	`[`
`2423`	`2427`	`{`
@@ -2499,6 +2503,7 @@ exports[`Button icon should return icon style according to different variations
`2499`	`2503`	`undefined,`
`2500`	`2504`	`undefined,`
`2501`	`2505`	`undefined,`
	`2506`	`+ undefined,`
`2502`	`2507`	`false,`
`2503`	`2508`	`[`
`2504`	`2509`	`{`
@@ -2575,6 +2580,7 @@ exports[`Button icon should return icon style according to different variations
`2575`	`2580`	`undefined,`
`2576`	`2581`	`undefined,`
`2577`	`2582`	`undefined,`
	`2583`	`+ undefined,`
`2578`	`2584`	`false,`
`2579`	`2585`	`[`
`2580`	`2586`	`{`
@@ -2651,6 +2657,7 @@ exports[`Button icon should return icon style according to different variations
`2651`	`2657`	`undefined,`
`2652`	`2658`	`undefined,`
`2653`	`2659`	`undefined,`
	`2660`	`+ undefined,`
`2654`	`2661`	`false,`
`2655`	`2662`	`[`
`2656`	`2663`	`{`
@@ -3901,6 +3908,7 @@ exports[`Button labelColor should return undefined color if this is an icon butt
`3901`	`3908`	`undefined,`
`3902`	`3909`	`undefined,`
`3903`	`3910`	`undefined,`
	`3911`	`+ undefined,`
`3904`	`3912`	`false,`
`3905`	`3913`	`[`
`3906`	`3914`	`{`