[PR #3289] [CLOSED] fix: refine settings dialog ui #5663

New issue

Closed

opened 2026-03-23 22:19:44 +00:00 by mirror · 0 comments

mirror commented

2026-03-23 22:19:44 +00:00

Owner

📋 Pull Request Information

Original PR: https://github.com/AppFlowy-IO/AppFlowy/pull/3289
Author: @Xazin
Created: 8/29/2023
Status: ❌ Closed

Base: main ← Head: fix/refine-settings-dialog

📝 Commits (2)

0d71de3 fix: refine settings dialog ui
c3d1159 fix: lints

📊 Changes

8 files changed (+138 additions, -101 deletions)

View changed files

📝 frontend/appflowy_flutter/lib/workspace/presentation/home/menu/sidebar/sidebar_user.dart (+1 -1)
📝 frontend/appflowy_flutter/lib/workspace/presentation/settings/settings_dialog.dart (+77 -36)
📝 frontend/appflowy_flutter/lib/workspace/presentation/settings/widgets/settings_appearance/brightness_setting.dart (+20 -22)
📝 frontend/appflowy_flutter/lib/workspace/presentation/settings/widgets/settings_appearance/color_scheme.dart (+3 -3)
📝 frontend/appflowy_flutter/lib/workspace/presentation/settings/widgets/settings_appearance/theme_setting_entry_template.dart (+4 -0)
📝 frontend/appflowy_flutter/lib/workspace/presentation/settings/widgets/settings_menu.dart (+6 -12)
📝 frontend/appflowy_flutter/lib/workspace/presentation/settings/widgets/settings_menu_element.dart (+17 -24)
📝 frontend/appflowy_flutter/packages/flowy_infra_ui/lib/style_widget/button.dart (+10 -3)

📄 Description

There is no issue related to this PR.

Feature Preview

PR Checklist

My code adheres to the AppFlowy Style Guide
I've listed at least one issue that this PR fixes in the description above.
I've added a test(s) to validate changes in this PR, or this PR only contains semantic changes.
All existing tests are passing.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/AppFlowy-IO/AppFlowy/pull/3289 **Author:** [@Xazin](https://github.com/Xazin) **Created:** 8/29/2023 **Status:** ❌ Closed **Base:** `main` ← **Head:** `fix/refine-settings-dialog` --- ### 📝 Commits (2) - [`0d71de3`](https://github.com/AppFlowy-IO/AppFlowy/commit/0d71de35c316ed0ab900e36ebe8aa173e82eb843) fix: refine settings dialog ui - [`c3d1159`](https://github.com/AppFlowy-IO/AppFlowy/commit/c3d1159eb8e646d142b86d1ac7ac7ddc154cd400) fix: lints ### 📊 Changes **8 files changed** (+138 additions, -101 deletions) <details> <summary>View changed files</summary> 📝 `frontend/appflowy_flutter/lib/workspace/presentation/home/menu/sidebar/sidebar_user.dart` (+1 -1) 📝 `frontend/appflowy_flutter/lib/workspace/presentation/settings/settings_dialog.dart` (+77 -36) 📝 `frontend/appflowy_flutter/lib/workspace/presentation/settings/widgets/settings_appearance/brightness_setting.dart` (+20 -22) 📝 `frontend/appflowy_flutter/lib/workspace/presentation/settings/widgets/settings_appearance/color_scheme.dart` (+3 -3) 📝 `frontend/appflowy_flutter/lib/workspace/presentation/settings/widgets/settings_appearance/theme_setting_entry_template.dart` (+4 -0) 📝 `frontend/appflowy_flutter/lib/workspace/presentation/settings/widgets/settings_menu.dart` (+6 -12) 📝 `frontend/appflowy_flutter/lib/workspace/presentation/settings/widgets/settings_menu_element.dart` (+17 -24) 📝 `frontend/appflowy_flutter/packages/flowy_infra_ui/lib/style_widget/button.dart` (+10 -3) </details> ### 📄 Description There is no issue related to this PR. ### Feature Preview ![image](https://github.com/AppFlowy-IO/AppFlowy/assets/42929161/4eb7db59-04d2-4db5-b3bb-f53134143113) #### PR Checklist - [x] My code adheres to the [AppFlowy Style Guide](https://appflowy.gitbook.io/docs/essential-documentation/contribute-to-appflowy/software-contributions/submitting-code/style-guides) - [ ] I've listed at least one issue that this PR fixes in the description above. - [x] I've added a test(s) to validate changes in this PR, or this PR only contains semantic changes. - [ ] All existing tests are passing. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>