[PR #8484] fix: improve settings button hover consistency and shortcut layout #8356

Open
opened 2026-03-23 23:23:49 +00:00 by mirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/AppFlowy-IO/AppFlowy/pull/8484
Author: @brianyoon0
Created: 2/17/2026
Status: 🔄 Open

Base: mainHead: fix-settings-button-consistency-8222


📝 Commits (1)

  • 208cca6 fix: improve settings button hover consistency and shortcut layout (#8222)

📊 Changes

1 file changed (+15 additions, -10 deletions)

View changed files

📝 frontend/appflowy_flutter/lib/workspace/presentation/settings/pages/settings_shortcuts_view.dart (+15 -10)

📄 Description

…8222)

  • Fixed reset button hover effect to match other settings buttons
  • Changed shortcut tile layout from 50/50 to space-between alignment
  • Improved visual consistency across settings UI

Fixes #8222

Feature Preview


PR Checklist

  • My code adheres to AppFlowy's Conventions
  • 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.

Summary by Sourcery

Improve the settings shortcuts UI for better hover feedback and shortcut layout.

Enhancements:

  • Update the reset shortcuts button to visually highlight icon and label on hover for consistency with other settings buttons.
  • Adjust the shortcut settings tile layout to space shortcuts and editors more evenly within the row.

🔄 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/8484 **Author:** [@brianyoon0](https://github.com/brianyoon0) **Created:** 2/17/2026 **Status:** 🔄 Open **Base:** `main` ← **Head:** `fix-settings-button-consistency-8222` --- ### 📝 Commits (1) - [`208cca6`](https://github.com/AppFlowy-IO/AppFlowy/commit/208cca6a512406fb05d37bcf6c05bb6ff3552de4) fix: improve settings button hover consistency and shortcut layout (#8222) ### 📊 Changes **1 file changed** (+15 additions, -10 deletions) <details> <summary>View changed files</summary> 📝 `frontend/appflowy_flutter/lib/workspace/presentation/settings/pages/settings_shortcuts_view.dart` (+15 -10) </details> ### 📄 Description …8222) - Fixed reset button hover effect to match other settings buttons - Changed shortcut tile layout from 50/50 to space-between alignment - Improved visual consistency across settings UI Fixes #8222 <!--- Thank you for submitting a pull request to AppFlowy. The team will dedicate their best efforts to reviewing and approving your pull request. If you have any questions about the project or feedback for us, please join our [Discord](https://discord.gg/wdjWUXXhtw). --> <!--- If your pull request adds a new feature, please drag and drop a video into this section to showcase what you've done! If not, you may delete this section. --> ### Feature Preview <!--- List at least one issue here that this PR addresses. If it fixes the issue, please use the [fixes](https://docs.github.com/en/get-started/writing-on-github/working-with-advanced-formatting/using-keywords-in-issues-and-pull-requests) keyword to close the issue. For example: fixes https://github.com/AppFlowy-IO/AppFlowy/pull/2106 --> --- <!--- Before you mark this PR ready for review, run through this checklist! --> #### PR Checklist - [ ] My code adheres to [AppFlowy's Conventions](https://docs.appflowy.io/docs/documentation/software-contributions/conventions) - [ ] 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. ## Summary by Sourcery Improve the settings shortcuts UI for better hover feedback and shortcut layout. Enhancements: - Update the reset shortcuts button to visually highlight icon and label on hover for consistency with other settings buttons. - Adjust the shortcut settings tile layout to space shortcuts and editors more evenly within the row. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
AppFlowy-IO/AppFlowy#8356
No description provided.