Conversation
opficdev
commented
Mar 5, 2026
- closed adapativeButtonStyle에 iOS 18 이하일 경우 + 라이트 모드일 경우 얇게 보이는 테두리를 추가한다 #190
Summary of ChangesHello, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! 이 PR은 adaptiveButtonStyle의 시각적 동작을 개선하는 데 중점을 둡니다. glassEffect 사용 시 클리핑 문제를 해결하고, 투명한 배경의 렌더링 방식을 표준화하여 UI의 전반적인 일관성과 미학을 향상시킵니다. Highlights
Changelog
Activity
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
| self.foregroundStyle(Color(.label)) | ||
| .padding(8) | ||
| .glassEffect(.regular.tint(color), in: shape) | ||
| .clipShape(shape) |
There was a problem hiding this comment.
이 코드가 포함된 if #available(iOS 26.0, *) 조건문은 glassEffect가 사용된 것으로 보아 visionOS를 대상으로 하는 것으로 보입니다. 하지만 iOS 26.0은 유효한 OS 버전이 아니므로 이 조건은 항상 거짓이 되어, 이 코드 블록은 절대로 실행되지 않습니다.
visionOS 용 코드를 분기하려면 #if os(visionOS) 전처리기 지시문을 사용하는 것이 올바른 방법입니다.
#if os(visionOS)
self.foregroundStyle(Color(.label))
.padding(8)
.glassEffect(.regular.tint(color), in: shape)
.clipShape(shape)
#else
// ... 다른 OS를 위한 fallback 코드
#endif현재 구현은 의도와 다르게 항상 else 블록의 코드를 실행하게 되므로, visionOS 특정 UI가 적용되지 않는 심각한 문제입니다. 상위의 if #available 문을 수정해주세요.
| if color == .clear { | ||
| shape | ||
| .fill(.ultraThinMaterial) | ||
| .fill((Color(.systemGray5))) |