Skip to content

Conversation

@ACTCD
Copy link
Collaborator

@ACTCD ACTCD commented Jun 29, 2025

@ACTCD ACTCD requested a review from Copilot June 29, 2025 03:45
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 adds a new liquid glass app icon across the project, updates CI to a newer macOS runner, and replaces the old PNG-based logo with a Svelte SVG component.

  • Registers AppIcon.icon in the Xcode project and adds its configuration file
  • Introduces LogoIcon.svelte and updates App.svelte to use it instead of a PNG
  • Updates the GitHub Actions workflow to target a newer macOS runner and Xcode version

Reviewed Changes

Copilot reviewed 5 out of 7 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
xcode/Userscripts.xcodeproj/project.pbxproj Added build-file, file-reference, and group entries for AppIcon
xcode/App-Shared/AppIcon.icon/icon.json New JSON config describing the liquid glass icon appearance
src/shared/Components/LogoIcon.svelte New Svelte component rendering the SVG-based app icon
src/app/App.svelte Replaced <img> with <LogoIcon /> and adjusted related styles
.github/workflows/deployment.yaml Updated runs-on label and xcode-select path in CI workflow
Comments suppressed due to low confidence (2)

.github/workflows/deployment.yaml:12

  • Verify that GitHub Actions supports the macos-15 runner label; if not, switch to macos-latest or another valid image to avoid pipeline failures.
    runs-on: macos-15 # https://github.com/actions/runner-images

.github/workflows/deployment.yaml:41

  • The path /Applications/Xcode_26.0.app may not exist on the runner; update this to the actual installed Xcode version (e.g., Xcode_16.2) or make it configurable.
        run: sudo xcode-select -s "/Applications/Xcode_26.0.app" # https://github.com/actions/runner-images/blob/main/images/macos/macos-14-arm64-Readme.md#xcode

@ACTCD ACTCD requested a review from quoid June 29, 2025 04:00
Copy link
Owner

@quoid quoid left a comment

Choose a reason for hiding this comment

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

🤩

@ACTCD ACTCD merged commit 43ab052 into main Jul 1, 2025
1 check passed
@ACTCD ACTCD deleted the new-app-icon branch July 1, 2025 02:18
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.

iOS 18-26 dark icon and macOS 26

3 participants