From 3cfe17676699f031bcb12d70dcb0705c1353f4d5 Mon Sep 17 00:00:00 2001 From: adids1221 Date: Wed, 20 Aug 2025 14:19:17 +0300 Subject: [PATCH 1/6] update: blue 30 system color hex, test fix --- .../__snapshots__/AvatarScreen.spec.js.snap | 12 ++++++------ .../__snapshots__/TextFieldScreen.spec.js.snap | 4 ++-- src/style/__tests__/colors.spec.js | 6 +++--- src/style/colorsPalette.ts | 2 +- 4 files changed, 12 insertions(+), 12 deletions(-) diff --git a/demo/src/screens/__tests__/__snapshots__/AvatarScreen.spec.js.snap b/demo/src/screens/__tests__/__snapshots__/AvatarScreen.spec.js.snap index 6285de6837..de7d9b1696 100644 --- a/demo/src/screens/__tests__/__snapshots__/AvatarScreen.spec.js.snap +++ b/demo/src/screens/__tests__/__snapshots__/AvatarScreen.spec.js.snap @@ -776,7 +776,7 @@ exports[`AvatarScreen renders screen 1`] = ` { "alignItems": "center", "alignSelf": "flex-start", - "backgroundColor": "#116DFF", + "backgroundColor": "#0A62EE", "borderRadius": 999, "justifyContent": "center", "overflow": "hidden", @@ -1034,7 +1034,7 @@ exports[`AvatarScreen renders screen 1`] = ` { "alignItems": "center", "alignSelf": "flex-start", - "backgroundColor": "#116DFF", + "backgroundColor": "#0A62EE", "borderRadius": 999, "justifyContent": "center", "overflow": "hidden", @@ -1292,7 +1292,7 @@ exports[`AvatarScreen renders screen 1`] = ` { "alignItems": "center", "alignSelf": "flex-start", - "backgroundColor": "#116DFF", + "backgroundColor": "#0A62EE", "borderRadius": 999, "justifyContent": "center", "overflow": "hidden", @@ -1759,7 +1759,7 @@ exports[`AvatarScreen renders screen 1`] = ` { "alignItems": "center", "alignSelf": "flex-start", - "backgroundColor": "#116DFF", + "backgroundColor": "#0A62EE", "borderRadius": 999, "justifyContent": "center", "overflow": "hidden", @@ -2617,7 +2617,7 @@ exports[`AvatarScreen renders screen 1`] = ` "top": 0, }, { - "backgroundColor": "#116DFF", + "backgroundColor": "#0A62EE", }, { "bottom": 1, @@ -3142,7 +3142,7 @@ exports[`AvatarScreen renders screen 1`] = ` { "alignItems": "center", "alignSelf": "flex-start", - "backgroundColor": "#116DFF", + "backgroundColor": "#0A62EE", "borderRadius": 999, "justifyContent": "center", "overflow": "hidden", diff --git a/demo/src/screens/__tests__/__snapshots__/TextFieldScreen.spec.js.snap b/demo/src/screens/__tests__/__snapshots__/TextFieldScreen.spec.js.snap index e2e3a1217e..f887a30c71 100644 --- a/demo/src/screens/__tests__/__snapshots__/TextFieldScreen.spec.js.snap +++ b/demo/src/screens/__tests__/__snapshots__/TextFieldScreen.spec.js.snap @@ -4387,7 +4387,7 @@ exports[`TextField Screen renders screen 1`] = ` "textAlign": "right", }, { - "color": "#116DFF", + "color": "#0A62EE", }, ], ] @@ -5144,7 +5144,7 @@ exports[`TextField Screen renders screen 1`] = ` "lineHeight": 24, }, { - "color": "#116DFF", + "color": "#0A62EE", }, undefined, undefined, diff --git a/src/style/__tests__/colors.spec.js b/src/style/__tests__/colors.spec.js index 9dc13f19dd..3941bc2d01 100644 --- a/src/style/__tests__/colors.spec.js +++ b/src/style/__tests__/colors.spec.js @@ -278,17 +278,17 @@ describe('style/Colors', () => { it('should return the system color, if the color is included in the validColors array', () => { expect(uut.getSystemColorByHex('#FFFFFF', GetColorsByHexOptions)).toEqual('white'); expect(uut.getSystemColorByHex('#000000', GetColorsByHexOptions)).toEqual('black'); - expect(uut.getSystemColorByHex('#116DFF', {validColors: [...SYSTEM_COLORS, 'blue']})).toEqual('blue30'); + expect(uut.getSystemColorByHex('#0A62EE', {validColors: [...SYSTEM_COLORS, 'blue']})).toEqual('blue30'); expect(uut.getSystemColorByHex('#FB6413', {validColors: [...SYSTEM_COLORS, 'orange']})).toEqual('orange30'); }); it('should return undefined if the color is not included in validColors', () => { - expect(uut.getSystemColorByHex('#116DFF', GetColorsByHexOptions)).toEqual(undefined); + expect(uut.getSystemColorByHex('#0A62EE', GetColorsByHexOptions)).toEqual(undefined); expect(uut.getSystemColorByHex('#00A87E', GetColorsByHexOptions)).toEqual(undefined); }); it('without validColors array the function will return the first system color name match the color', () => { - expect(uut.getSystemColorByHex('#116DFF')).toEqual('blue30'); + expect(uut.getSystemColorByHex('#0A62EE')).toEqual('blue30'); expect(uut.getSystemColorByHex('#00A87E')).toEqual('green30'); }); diff --git a/src/style/colorsPalette.ts b/src/style/colorsPalette.ts index 4ae7766499..ffdf7ddc13 100644 --- a/src/style/colorsPalette.ts +++ b/src/style/colorsPalette.ts @@ -15,7 +15,7 @@ const colorsPalette = { blue5: '#0C3E9D', blue10: '#0F4DC4', blue20: '#0F59E6', - blue30: '#116DFF', + blue30: '#0A62EE', blue40: '#3582FF', blue50: '#81B4FF', blue60: '#97C7FF', From 6d34dec15301b5e21000921842f0f468cecdf094 Mon Sep 17 00:00:00 2001 From: adids1221 Date: Wed, 20 Aug 2025 14:31:30 +0300 Subject: [PATCH 2/6] update: textSuccess, iconSuccess set to green5 --- src/style/designTokens.ts | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/style/designTokens.ts b/src/style/designTokens.ts index c265f0a7f7..4bda1581ee 100644 --- a/src/style/designTokens.ts +++ b/src/style/designTokens.ts @@ -40,7 +40,7 @@ export default { $textDefaultLight: colorsPalette.white, $textPrimary: colorsPalette.violet30, $textGeneral: colorsPalette.blue30, - $textSuccess: colorsPalette.green10, + $textSuccess: colorsPalette.green5, $textSuccessLight: colorsPalette.green30, $textMajor: colorsPalette.orange10, $textDanger: colorsPalette.red10, @@ -55,7 +55,7 @@ export default { $iconPrimaryLight: colorsPalette.violet50, $iconGeneral: colorsPalette.blue30, $iconGeneralLight: colorsPalette.blue50, - $iconSuccess: colorsPalette.green10, + $iconSuccess: colorsPalette.green5, $iconSuccessLight: colorsPalette.green30, $iconMajor: colorsPalette.orange10, $iconDanger: colorsPalette.red10, From 2440a7134179eb3313831bfc50c85be86aaf8925 Mon Sep 17 00:00:00 2001 From: adids1221 Date: Wed, 20 Aug 2025 14:33:00 +0300 Subject: [PATCH 3/6] update: textDanger, iconDanger set to red5 --- src/style/designTokens.ts | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/src/style/designTokens.ts b/src/style/designTokens.ts index 4bda1581ee..3f5bc101cc 100644 --- a/src/style/designTokens.ts +++ b/src/style/designTokens.ts @@ -43,8 +43,7 @@ export default { $textSuccess: colorsPalette.green5, $textSuccessLight: colorsPalette.green30, $textMajor: colorsPalette.orange10, - $textDanger: colorsPalette.red10, - //textDangerLight should be deprecated + $textDanger: colorsPalette.red5, $textDangerLight: colorsPalette.red10, // ICON @@ -58,8 +57,7 @@ export default { $iconSuccess: colorsPalette.green5, $iconSuccessLight: colorsPalette.green30, $iconMajor: colorsPalette.orange10, - $iconDanger: colorsPalette.red10, - //iconDangerLight should be deprecated + $iconDanger: colorsPalette.red5, $iconDangerLight: colorsPalette.red10, $iconDisabled: colorsPalette.grey50, From 8a7874ae73022eeaec10f296b800797dbd6d1d5f Mon Sep 17 00:00:00 2001 From: adids1221 Date: Wed, 20 Aug 2025 14:35:04 +0300 Subject: [PATCH 4/6] update: change textMajor and iconMajor colors to orange5 --- .../screens/__tests__/__snapshots__/AvatarScreen.spec.js.snap | 4 ++-- src/style/designTokens.ts | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/demo/src/screens/__tests__/__snapshots__/AvatarScreen.spec.js.snap b/demo/src/screens/__tests__/__snapshots__/AvatarScreen.spec.js.snap index de7d9b1696..aaf4cdc8d3 100644 --- a/demo/src/screens/__tests__/__snapshots__/AvatarScreen.spec.js.snap +++ b/demo/src/screens/__tests__/__snapshots__/AvatarScreen.spec.js.snap @@ -477,11 +477,11 @@ exports[`AvatarScreen renders screen 1`] = ` }, { "backgroundColor": "transparent", - "color": "#CD4700", + "color": "#9E3900", "lineHeight": undefined, }, { - "color": "#CD4700", + "color": "#9E3900", }, ], ] diff --git a/src/style/designTokens.ts b/src/style/designTokens.ts index 3f5bc101cc..8913312a77 100644 --- a/src/style/designTokens.ts +++ b/src/style/designTokens.ts @@ -42,7 +42,7 @@ export default { $textGeneral: colorsPalette.blue30, $textSuccess: colorsPalette.green5, $textSuccessLight: colorsPalette.green30, - $textMajor: colorsPalette.orange10, + $textMajor: colorsPalette.orange5, $textDanger: colorsPalette.red5, $textDangerLight: colorsPalette.red10, @@ -56,7 +56,7 @@ export default { $iconGeneralLight: colorsPalette.blue50, $iconSuccess: colorsPalette.green5, $iconSuccessLight: colorsPalette.green30, - $iconMajor: colorsPalette.orange10, + $iconMajor: colorsPalette.orange5, $iconDanger: colorsPalette.red5, $iconDangerLight: colorsPalette.red10, $iconDisabled: colorsPalette.grey50, From 7e3d29e4b138d25c4b3c3c20e71fee4249004cda Mon Sep 17 00:00:00 2001 From: adids1221 Date: Wed, 27 Aug 2025 14:13:37 +0300 Subject: [PATCH 5/6] update: change blue30 color hex to #0F62E6 and update related tests --- .../__snapshots__/AvatarScreen.spec.js.snap | 12 ++++++------ .../__snapshots__/TextFieldScreen.spec.js.snap | 4 ++-- src/style/__tests__/colors.spec.js | 4 ++-- src/style/colorsPalette.ts | 2 +- 4 files changed, 11 insertions(+), 11 deletions(-) diff --git a/demo/src/screens/__tests__/__snapshots__/AvatarScreen.spec.js.snap b/demo/src/screens/__tests__/__snapshots__/AvatarScreen.spec.js.snap index aaf4cdc8d3..3dda40d4bd 100644 --- a/demo/src/screens/__tests__/__snapshots__/AvatarScreen.spec.js.snap +++ b/demo/src/screens/__tests__/__snapshots__/AvatarScreen.spec.js.snap @@ -776,7 +776,7 @@ exports[`AvatarScreen renders screen 1`] = ` { "alignItems": "center", "alignSelf": "flex-start", - "backgroundColor": "#0A62EE", + "backgroundColor": "#0F62E6", "borderRadius": 999, "justifyContent": "center", "overflow": "hidden", @@ -1034,7 +1034,7 @@ exports[`AvatarScreen renders screen 1`] = ` { "alignItems": "center", "alignSelf": "flex-start", - "backgroundColor": "#0A62EE", + "backgroundColor": "#0F62E6", "borderRadius": 999, "justifyContent": "center", "overflow": "hidden", @@ -1292,7 +1292,7 @@ exports[`AvatarScreen renders screen 1`] = ` { "alignItems": "center", "alignSelf": "flex-start", - "backgroundColor": "#0A62EE", + "backgroundColor": "#0F62E6", "borderRadius": 999, "justifyContent": "center", "overflow": "hidden", @@ -1759,7 +1759,7 @@ exports[`AvatarScreen renders screen 1`] = ` { "alignItems": "center", "alignSelf": "flex-start", - "backgroundColor": "#0A62EE", + "backgroundColor": "#0F62E6", "borderRadius": 999, "justifyContent": "center", "overflow": "hidden", @@ -2617,7 +2617,7 @@ exports[`AvatarScreen renders screen 1`] = ` "top": 0, }, { - "backgroundColor": "#0A62EE", + "backgroundColor": "#0F62E6", }, { "bottom": 1, @@ -3142,7 +3142,7 @@ exports[`AvatarScreen renders screen 1`] = ` { "alignItems": "center", "alignSelf": "flex-start", - "backgroundColor": "#0A62EE", + "backgroundColor": "#0F62E6", "borderRadius": 999, "justifyContent": "center", "overflow": "hidden", diff --git a/demo/src/screens/__tests__/__snapshots__/TextFieldScreen.spec.js.snap b/demo/src/screens/__tests__/__snapshots__/TextFieldScreen.spec.js.snap index f887a30c71..098a38c83b 100644 --- a/demo/src/screens/__tests__/__snapshots__/TextFieldScreen.spec.js.snap +++ b/demo/src/screens/__tests__/__snapshots__/TextFieldScreen.spec.js.snap @@ -4387,7 +4387,7 @@ exports[`TextField Screen renders screen 1`] = ` "textAlign": "right", }, { - "color": "#0A62EE", + "color": "#0F62E6", }, ], ] @@ -5144,7 +5144,7 @@ exports[`TextField Screen renders screen 1`] = ` "lineHeight": 24, }, { - "color": "#0A62EE", + "color": "#0F62E6", }, undefined, undefined, diff --git a/src/style/__tests__/colors.spec.js b/src/style/__tests__/colors.spec.js index 3941bc2d01..278aa1a1a0 100644 --- a/src/style/__tests__/colors.spec.js +++ b/src/style/__tests__/colors.spec.js @@ -278,7 +278,7 @@ describe('style/Colors', () => { it('should return the system color, if the color is included in the validColors array', () => { expect(uut.getSystemColorByHex('#FFFFFF', GetColorsByHexOptions)).toEqual('white'); expect(uut.getSystemColorByHex('#000000', GetColorsByHexOptions)).toEqual('black'); - expect(uut.getSystemColorByHex('#0A62EE', {validColors: [...SYSTEM_COLORS, 'blue']})).toEqual('blue30'); + expect(uut.getSystemColorByHex('#0F62E6', {validColors: [...SYSTEM_COLORS, 'blue']})).toEqual('blue30'); expect(uut.getSystemColorByHex('#FB6413', {validColors: [...SYSTEM_COLORS, 'orange']})).toEqual('orange30'); }); @@ -288,7 +288,7 @@ describe('style/Colors', () => { }); it('without validColors array the function will return the first system color name match the color', () => { - expect(uut.getSystemColorByHex('#0A62EE')).toEqual('blue30'); + expect(uut.getSystemColorByHex('#0F62E6')).toEqual('blue30'); expect(uut.getSystemColorByHex('#00A87E')).toEqual('green30'); }); diff --git a/src/style/colorsPalette.ts b/src/style/colorsPalette.ts index ffdf7ddc13..f24bdd563f 100644 --- a/src/style/colorsPalette.ts +++ b/src/style/colorsPalette.ts @@ -15,7 +15,7 @@ const colorsPalette = { blue5: '#0C3E9D', blue10: '#0F4DC4', blue20: '#0F59E6', - blue30: '#0A62EE', + blue30: '#0F62E6', blue40: '#3582FF', blue50: '#81B4FF', blue60: '#97C7FF', From 6bcd9cdf5bd396306b8da73a5e910398e349d474 Mon Sep 17 00:00:00 2001 From: Adi Mordo Date: Thu, 28 Aug 2025 11:19:04 +0300 Subject: [PATCH 6/6] Update src/style/__tests__/colors.spec.js Co-authored-by: Nitzan Yizhar --- src/style/__tests__/colors.spec.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/style/__tests__/colors.spec.js b/src/style/__tests__/colors.spec.js index 278aa1a1a0..46aeaec9ce 100644 --- a/src/style/__tests__/colors.spec.js +++ b/src/style/__tests__/colors.spec.js @@ -283,7 +283,7 @@ describe('style/Colors', () => { }); it('should return undefined if the color is not included in validColors', () => { - expect(uut.getSystemColorByHex('#0A62EE', GetColorsByHexOptions)).toEqual(undefined); + expect(uut.getSystemColorByHex('#0F62E6', GetColorsByHexOptions)).toEqual(undefined); expect(uut.getSystemColorByHex('#00A87E', GetColorsByHexOptions)).toEqual(undefined); });