Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Append visionOS support #90

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

florentmorin
Copy link

Motivation

Project has been updated to integrate Sidekick Spatial AI.

What's new

The code is now fully compatible with visionOS.

Changes

Significant updates were required to support visionOS.

Below is an overview of the key changes.

Swift Package version

To support visionOS target, Swift Package version has been updated to 5.9.

Material background

Most UI elements in visionOS require a Material background by default :

  • The code has been updated to allow an optional cell color (required in the current version) and an optional cell material.
  • By default:
    • macOS and iOS will use the existing default color.
    • visionOS will use a new default material (provided via a new extension).
  • macOS and iOS prioritize color over material, while visionOS does the opposite.

Button hover effect

The hover effect for the vote / unvote button has been enhanced for a cleaner user experience.

Fix on filters

Previously, filters behaved unpredictably due to a default value issue. This has been resolved, and the all value is now selected by default.

Sample code

Sample code has been updated to support visionOS.

Screenshots

Screenshot-12-04-2024-13 02 42

Screenshot-12-04-2024-13 02 30

Copy link
Collaborator

@hiddevdploeg hiddevdploeg left a comment

Choose a reason for hiding this comment

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

Looks great! Thanks for doing this 😊

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