Skip to content

Conversation

RGLie
Copy link
Contributor

@RGLie RGLie commented Nov 14, 2024

Overview

Dark Mode 완료 하였습니다.
아직 다크 모드 설정을 캐시로 불러오는 기능은 없습니다.

Changes

Implementaion Method

After Changes

Related Issues

Rollback Scenario

TODO

@RGLie RGLie linked an issue Nov 14, 2024 that may be closed by this pull request
@RGLie RGLie marked this pull request as ready for review November 14, 2024 14:25
sangohkim and others added 2 commits November 14, 2024 23:57
set initial state to 'true' instead of 'false', which fixes a lot of network-related errors.
Copy link
Contributor

@thomaskim1130 thomaskim1130 left a comment

Choose a reason for hiding this comment

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

정말 수고하셨습니다! dialog 바로 반영해 주신 점 감사합니다.
다만 페이지를 리프레쉬할 때 발생하는 'RefreshIndicator.adaptive'(현재 제가 개편 중)에서 color, backgroundColor option을 변경할 수 있는데, 여기에도 다크테마를 반영해 주시면 좋을 것 같습니다.
https://api.flutter.dev/flutter/material/RefreshIndicator/RefreshIndicator.adaptive.html

@RGLie RGLie marked this pull request as draft November 21, 2024 13:07
@RGLie RGLie marked this pull request as ready for review November 21, 2024 13:10
@thomaskim1130 thomaskim1130 self-assigned this Nov 21, 2024
Copy link
Contributor

@thomaskim1130 thomaskim1130 left a comment

Choose a reason for hiding this comment

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

LGTM

@thomaskim1130 thomaskim1130 removed their assignment Nov 21, 2024
@thomaskim1130 thomaskim1130 merged commit f33baed into dev Nov 21, 2024
1 of 2 checks passed
@thomaskim1130 thomaskim1130 deleted the feature@/3.24-dark-mode branch November 21, 2024 14:02
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.

[Feature] 3.24 Dark Mode
3 participants