Skip to content

targetSDK 35 대응 #445

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

Open
wants to merge 11 commits into
base: develop
Choose a base branch
from
Open

targetSDK 35 대응 #445

wants to merge 11 commits into from

Conversation

plgafhd
Copy link
Collaborator

@plgafhd plgafhd commented Jun 22, 2025

공식 문서 내용 중, 창 인셋 변경 사항 내용만 대응이 필요했다.

  • setStatusBarColor()와 setNavigationBarColor()가 deprecated 되었으므로, 사용하지 않아야 함
  • SDK 35가 되면서 더 넓은 화면을 표시하도록 콘텐츠가 statusBar와 navigationBar까지 그려지는 것이 default가 됨
  • 이에 따라 statusBar와 navigationBar는 투명하게 하는 것이 권장이 되고, 위의 deprecated 된 함수는 영향을 주지 못함
  • enableEdgeToEdge() 사용하고, inset을 적당히 주어서 기존 화면 구성을 유지해야 한다.

대응이 됐으나 google play console의 경고는 유지되었는데 이에 대해서는 코멘트로 자세히 설명

@plgafhd plgafhd requested a review from a team as a code owner June 22, 2025 14:01
@plgafhd
Copy link
Collaborator Author

plgafhd commented Jun 22, 2025

앱에서 더 넓은 화면용으로 지원 중단된 API 또는 파라미터를 사용합니다

이 경고와 함께 어디서 쓰이는지를 알려줬는데 R8 mapping을 까본 결과

  • com.wafflestudio.snutt2.views.RootActivity$setWindowAppearance -> 이게 유일하게 우리 코드이고, 대응해서 없어짐
  • com.facebook.react.modules.statusbar.StatusBarModule (RN 관련)
  • com.facebook.react.modules.statusbar.StatusBarModule.getTypedExportedConstants (RN 관련)
  • com.google.android.gms.dynamite.zzb 내의 일부 함수 (RN 관련)
  • com.google.android.material.datepicker.l.M
    • 이건 com.google.android.material:material 라이브러리를 쓰는 과정에서, 그 내부 코드에 있어서 감지된 것 (우리 잘못 아님)
  • com.swmansion.rnscreens.ScreenWindowTraits$setColor$1 (RN 관련)
  • enableEdgeToEdge()
    • SDK 35 대응한다고 이 함수 쓰라고 해놓고 이 함수 내부에 SDK 35에서 deprecated 되는 함수를 쓰고 있다 (...)

그래서 우리 잘못이 없다...

@plgafhd
Copy link
Collaborator Author

plgafhd commented Jun 22, 2025

일부 사용자에게는 더 넓은 화면이 표시되지 않을 수 있습니다

enableEdgeToEdge() 쓰라고 해서 쓰고 있고, SDK 26까지 내려가서 테스트 해봤는데도 잘 되기 때문에 문제는 없을 것 같다.

@JuTaK97
Copy link
Collaborator

JuTaK97 commented Jun 22, 2025

단순 deprecate 어노테이션만 붙고 아예 없어지지는 않는 거면 냅둬도 될듯

Comment on lines -556 to -560
val primaryColor = ContextCompat.getColor(this@RootActivity, if (isDarkMode) R.color.black_dark else R.color.white)
window.apply {
setBackgroundDrawableResource(if (isDarkMode) R.color.black_dark else R.color.white)
statusBarColor = primaryColor
navigationBarColor = primaryColor
Copy link
Collaborator Author

@plgafhd plgafhd Jun 22, 2025

Choose a reason for hiding this comment

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

이 부분만 해당되겠네
일단 없어지지 않고 어노테이션만 붙는건 맞는것 같은데
코드가 있어도 어차피 무시될거라 없앤거긴 해
없애지 말고 그대로 둘까?

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