Skip to content

Conversation

@okwasniewski
Copy link
Member

Summary

This PR uses border configuration for borders fixing #22

The animation is now properly applied.

Screen.Recording.2025-10-23.at.14.31.07.mov

@okwasniewski okwasniewski linked an issue Oct 23, 2025 that may be closed by this pull request
3 tasks
@okwasniewski okwasniewski force-pushed the feat/support-context-menu branch from 32017e6 to cee1564 Compare October 23, 2025 12:35
@okwasniewski okwasniewski requested a review from Copilot October 23, 2025 12:42
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR implements proper border configuration handling for LiquidGlassView using iOS's UICornerConfiguration API to fix animation issues with borders. The implementation now correctly responds to changes in border styles, radii, and colors.

Key changes:

  • Added border change detection and layer invalidation mechanism in the iOS native component
  • Implemented finalizeUpdates method to apply border configurations using UICornerConfiguration (iOS 26.0+)
  • Enhanced the example app with a new LiquidToolBar component to demonstrate border animations

Reviewed Changes

Copilot reviewed 4 out of 6 changed files in this pull request and generated 1 comment.

File Description
ios/LiquidGlassView.mm Implements border configuration handling with UICornerConfiguration API and layer invalidation mechanism
example/src/App.tsx Adds new toolbar component with dropdown menus to demonstrate border animation functionality
example/package.json Adds required dependencies for the new dropdown menu components
example/ios/LiquidGlassExample.xcodeproj/project.pbxproj Removes empty inputPaths/outputPaths arrays from CocoaPods build phases

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@okwasniewski okwasniewski force-pushed the feat/support-context-menu branch from cee1564 to a4fae06 Compare October 23, 2025 12:46
@okwasniewski okwasniewski merged commit cc8f290 into main Oct 23, 2025
5 checks passed
@denissdubinin
Copy link

Big thank you @okwasniewski!

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.

Border radius for LiquidGlassView with ContextMenu inside

3 participants