Skip to content

chore: [IOPLT-1906] Replace legacy shadow with boxShadow in the DS components#8221

Open
dmnplb wants to merge 25 commits into
masterfrom
IOPLT-1906-remove-legacy-shadow-ds
Open

chore: [IOPLT-1906] Replace legacy shadow with boxShadow in the DS components#8221
dmnplb wants to merge 25 commits into
masterfrom
IOPLT-1906-remove-legacy-shadow-ds

Conversation

@dmnplb

@dmnplb dmnplb commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

Short description

Replace legacy shadow with boxShadow in the DS components, such as FooterActions and FooterActionsInline.

List of changes proposed in this pull request

  • Add the new shadow as a valid BoxShadowValue object
  • Remove legacy shadow from FooterActions and FooterActionsInline

Dependencies

How to test

Check the DS section, in particular all the FooterActions… and FooterActionsInline cases

CrisTofani and others added 24 commits June 17, 2026 14:53
…egacy-shadow-ds

# Conflicts:
#	apps/main-app/ts/features/itwallet/common/components/ItwSkeumorphicCard/index.tsx
#	apps/main-app/ts/features/itwallet/machine/eid/actions.ts
#	apps/main-app/ts/features/itwallet/presentation/remote/utils/__tests__/itwRemotePresentationUtils.test.ts
@dmnplb dmnplb self-assigned this Jul 1, 2026
@dmnplb dmnplb added the Design System New visual language and reduction of previous UI clutter label Jul 1, 2026
@github-actions

github-actions Bot commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

Jira Pull Request Link

This Pull Request refers to Jira issues:

@github-actions

github-actions Bot commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

PR Title Validation for conventional commit type

All good! PR title follows the conventional commit type.

@dmnplb dmnplb marked this pull request as ready for review July 1, 2026 13:02
@codecov

codecov Bot commented Jul 1, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 61.19403% with 78 lines in your changes missing coverage. Please review.
✅ Project coverage is 62.71%. Comparing base (2608bab) to head (6eeee5b).
⚠️ Report is 18 commits behind head on IOPLT-1792-migrate-ds-to-monorepo.

Files with missing lines Patch % Lines
...twallet/common/utils/itwCredentialIssuanceUtils.ts 16.66% 25 Missing ⚠️
.../ts/features/itwallet/machine/credential/actors.ts 5.55% 17 Missing ⚠️
...llet/credentialsCatalogue/store/selectors/index.ts 58.33% 5 Missing ⚠️
.../features/itwallet/machine/credential/selectors.ts 44.44% 5 Missing ⚠️
...screens/ItwIssuanceCredentialTrustIssuerScreen.tsx 20.00% 4 Missing ⚠️
...et/issuance/screens/ItwIssuanceEidResultScreen.tsx 78.94% 4 Missing ⚠️
...res/wallet/components/WalletEmptyScreenContent.tsx 0.00% 3 Missing ⚠️
...ures/itwallet/common/hooks/useItwAuthSourceName.ts 33.33% 1 Missing and 1 partial ⚠️
...atures/itwallet/common/utils/itwCredentialUtils.ts 77.77% 1 Missing and 1 partial ⚠️
...als/saga/handleItwCredentialsVaultCoherenceSaga.ts 90.00% 1 Missing and 1 partial ⚠️
... and 6 more
Additional details and impacted files

Impacted file tree graph

@@                          Coverage Diff                          @@
##           IOPLT-1792-migrate-ds-to-monorepo    #8221      +/-   ##
=====================================================================
+ Coverage                              62.50%   62.71%   +0.20%     
=====================================================================
  Files                                   1954     1951       -3     
  Lines                                  37010    37005       -5     
  Branches                                6183     6185       +2     
=====================================================================
+ Hits                                   23133    23207      +74     
+ Misses                                 12502    12419      -83     
- Partials                                1375     1379       +4     
Files with missing lines Coverage Δ
...omponents/screens/OperationResultScreenContent.tsx 93.75% <ø> (ø)
...features/fci/components/DocumentsNavigationBar.tsx 100.00% <ø> (ø)
.../features/fci/screens/loginL3/FciLoginL3Screen.tsx 84.61% <ø> (ø)
.../features/fci/screens/loginL3/LoginOptInScreen.tsx 80.00% <ø> (ø)
...llet/common/components/ItwCredentialCard/config.ts 89.36% <ø> (ø)
...let/common/components/ItwSkeumorphicCard/index.tsx 15.62% <ø> (ø)
...twallet/common/components/ItwWalletReadyBanner.tsx 90.90% <ø> (ø)
...es/itwallet/common/store/selectors/remoteConfig.ts 60.97% <100.00%> (+6.68%) ⬆️
...features/itwallet/common/utils/itwMetadataUtils.ts 60.00% <100.00%> (+17.14%) ⬆️
...ts/features/itwallet/common/utils/itwMocksUtils.ts 100.00% <ø> (ø)
... and 35 more

... and 20 files with indirect coverage changes


Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2608bab...6eeee5b. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Base automatically changed from IOPLT-1792-migrate-ds-to-monorepo to master July 13, 2026 13:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Design System New visual language and reduction of previous UI clutter

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants