[PR #8261] Fix settings shortcuts UI improvements 8222 #8315

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

📋 Pull Request Information

Original PR: https://github.com/AppFlowy-IO/AppFlowy/pull/8261
Author: @RishiAhuja
Created: 10/7/2025
Status: 🔄 Open

Base: mainHead: feature/settings-shortcuts-ui-improvements-8222


📝 Commits (4)

  • 17072ab fix: shortcut tile layout spacing
  • 10d3fbb fix: shortcut tile layout spacing
  • 91942ab fix: improve hover effect for reset button in shortcuts settings
  • 7d9b36a fix: adjust layout for shortcut setting tiles

📊 Changes

1 file changed (+21 additions, -12 deletions)

View changed files

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

📄 Description

Feature Preview

This PR improves the user experience in the shortcuts settings page by fixing layout spacing issues and enhancing hover effects for better visual feedback.
fixes https://github.com/AppFlowy-IO/AppFlowy/issues/8222

Issue 1 fixed

Screenshot 2025-10-07 at 10 46 37 AM

Issue 2 fixed

Screenshot 2025-10-07 at 10 46 54 AM

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 visual feedback and layout spacing in the shortcuts settings UI

Enhancements:

  • Add persistent hover state and custom hover color to the reset button
  • Update reset button to dynamically change icon and text colors on hover
  • Adjust flex ratios in shortcut setting tiles to give more space to keybinding fields

🔄 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/8261 **Author:** [@RishiAhuja](https://github.com/RishiAhuja) **Created:** 10/7/2025 **Status:** 🔄 Open **Base:** `main` ← **Head:** `feature/settings-shortcuts-ui-improvements-8222` --- ### 📝 Commits (4) - [`17072ab`](https://github.com/AppFlowy-IO/AppFlowy/commit/17072ab16ba4948cc9fe07d9f230e546d1b7917e) fix: shortcut tile layout spacing - [`10d3fbb`](https://github.com/AppFlowy-IO/AppFlowy/commit/10d3fbb557b424995a461e5c1a381c83ac498171) fix: shortcut tile layout spacing - [`91942ab`](https://github.com/AppFlowy-IO/AppFlowy/commit/91942abd118d8e031a34c150a656eb67c30733a3) fix: improve hover effect for reset button in shortcuts settings - [`7d9b36a`](https://github.com/AppFlowy-IO/AppFlowy/commit/7d9b36a2779f2b7429bc48faf630044bf066062d) fix: adjust layout for shortcut setting tiles ### 📊 Changes **1 file changed** (+21 additions, -12 deletions) <details> <summary>View changed files</summary> 📝 `frontend/appflowy_flutter/lib/workspace/presentation/settings/pages/settings_shortcuts_view.dart` (+21 -12) </details> ### 📄 Description <!--- 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 This PR improves the user experience in the shortcuts settings page by fixing layout spacing issues and enhancing hover effects for better visual feedback. fixes https://github.com/AppFlowy-IO/AppFlowy/issues/8222 ### Issue 1 fixed <img width="720" height="128" alt="Screenshot 2025-10-07 at 10 46 37 AM" src="https://github.com/user-attachments/assets/2236a080-8f1b-46c0-b074-c4d55b47c1f8" /> ### Issue 2 fixed <img width="151" height="92" alt="Screenshot 2025-10-07 at 10 46 54 AM" src="https://github.com/user-attachments/assets/d58356cb-2252-4388-9a12-6b49aac4bfdf" /> <!--- Before you mark this PR ready for review, run through this checklist! --> #### PR Checklist - [x] My code adheres to [AppFlowy's Conventions](https://docs.appflowy.io/docs/documentation/software-contributions/conventions) - [x] 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. - [x] All existing tests are passing. ## Summary by Sourcery Improve the visual feedback and layout spacing in the shortcuts settings UI Enhancements: - Add persistent hover state and custom hover color to the reset button - Update reset button to dynamically change icon and text colors on hover - Adjust flex ratios in shortcut setting tiles to give more space to keybinding fields --- <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#8315
No description provided.