Skip to content

Conversation

@CatchABus
Copy link
Contributor

@CatchABus CatchABus commented Aug 11, 2025

PR Checklist

What is the current behavior?

Right now, views can only have a single box-shadow.

What is the new behavior?

Views can have multiple box-shadows.

Few improvements to note:

  • Android shadows will respect transparent background without the need of an overlay layer
  • Android shadows will render a solid shadow when blur radius is <= 0
  • Added multipliers to match native shadow blur with CSS shadow blur based on discussion in fix(ios): consistent box-shadow radius blur #10583

Fixes/Implements/Closes #[Issue Number].

BREAKING CHANGES:

Shadow blur changes to match the CSS shadow blur.
The formulas are based on these sources:
https://bjango.com/articles/matchingdropshadows

Apple UX guide also confirms the iOS formula:
https://microsoft.github.io/apple-ux-guide/Shadows.html

@nx-cloud
Copy link

nx-cloud bot commented Aug 11, 2025

View your CI Pipeline Execution ↗ for commit b7cae53

Command Status Duration Result
nx test apps-automated -c=android ✅ Succeeded 4m 30s View ↗
nx run-many --target=test --configuration=ci --... ✅ Succeeded 3s View ↗

☁️ Nx Cloud last updated this comment at 2025-11-05 19:14:20 UTC

@CatchABus CatchABus marked this pull request as ready for review August 11, 2025 11:35
@CatchABus CatchABus marked this pull request as draft August 11, 2025 14:02
@CatchABus CatchABus marked this pull request as ready for review August 11, 2025 14:07
@CatchABus CatchABus marked this pull request as draft August 11, 2025 15:05
@CatchABus CatchABus marked this pull request as ready for review August 11, 2025 15:10
@NathanWalker NathanWalker added this to the 9.0 milestone Aug 13, 2025
@NathanWalker NathanWalker changed the title feat(core): Multiple box-shadows support feat(core): multiple box-shadows support Nov 5, 2025
@NathanWalker NathanWalker changed the title feat(core): multiple box-shadows support feat(core): multiple css box-shadows support Nov 5, 2025
@NathanWalker NathanWalker merged commit cd6fccb into NativeScript:main Nov 5, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants