[PR #4576] [MERGED] chore: inkwell mobile quick action sheets #6326

New issue

Closed

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

mirror commented

2026-03-23 22:22:43 +00:00

Owner

📋 Pull Request Information

Original PR: https://github.com/AppFlowy-IO/AppFlowy/pull/4576
Author: @richardshiue
Created: 2/1/2024
Status: ✅ Merged
Merged: 2/3/2024
Merged by: @appflowy

Base: main ← Head: feat/mobile-quick-actions

📝 Commits (3)

2a0344e chore: inkwell mobile quick action sheets
b438627 chore: more subtle tap effect
b46c497 chore: improve code style

📊 Changes

5 files changed (+107 additions, -119 deletions)

View changed files

📝 frontend/appflowy_flutter/lib/mobile/presentation/bottom_sheet/bottom_sheet_view_item_body.dart (+19 -27)
📝 frontend/appflowy_flutter/lib/mobile/presentation/bottom_sheet/bottom_sheet_view_page.dart (+19 -27)
📝 frontend/appflowy_flutter/lib/mobile/presentation/database/card/card_detail/mobile_card_detail_screen.dart (+32 -42)
📝 frontend/appflowy_flutter/lib/mobile/presentation/database/view/database_view_quick_actions.dart (+7 -9)
📝 frontend/appflowy_flutter/lib/mobile/presentation/widgets/flowy_mobile_quick_action_button.dart (+30 -14)

📄 Description

change the height of each item to be 52px, less padding of 4px vertically to have 44px height for the inkwell widget
change the thickness of divider to 0.5px
used in:
- view actions in homepage after sliding
- view actions in the more appbar button
- database card detail actions
- database view actions.

https://github.com/AppFlowy-IO/AppFlowy/assets/71320345/5536ce68-84a0-4761-8c2f-21f61e7996f5

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.

_{🔄 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/4576 **Author:** [@richardshiue](https://github.com/richardshiue) **Created:** 2/1/2024 **Status:** ✅ Merged **Merged:** 2/3/2024 **Merged by:** [@appflowy](https://github.com/appflowy) **Base:** `main` ← **Head:** `feat/mobile-quick-actions` --- ### 📝 Commits (3) - [`2a0344e`](https://github.com/AppFlowy-IO/AppFlowy/commit/2a0344e5eb277d2c3673630c209222a9aefbda8f) chore: inkwell mobile quick action sheets - [`b438627`](https://github.com/AppFlowy-IO/AppFlowy/commit/b4386278c4a6ba41cb218880b5cc3a541eae3e6a) chore: more subtle tap effect - [`b46c497`](https://github.com/AppFlowy-IO/AppFlowy/commit/b46c4978a943f1d88ea1e83f91c9d1a9875f5df1) chore: improve code style ### 📊 Changes **5 files changed** (+107 additions, -119 deletions) <details> <summary>View changed files</summary> 📝 `frontend/appflowy_flutter/lib/mobile/presentation/bottom_sheet/bottom_sheet_view_item_body.dart` (+19 -27) 📝 `frontend/appflowy_flutter/lib/mobile/presentation/bottom_sheet/bottom_sheet_view_page.dart` (+19 -27) 📝 `frontend/appflowy_flutter/lib/mobile/presentation/database/card/card_detail/mobile_card_detail_screen.dart` (+32 -42) 📝 `frontend/appflowy_flutter/lib/mobile/presentation/database/view/database_view_quick_actions.dart` (+7 -9) 📝 `frontend/appflowy_flutter/lib/mobile/presentation/widgets/flowy_mobile_quick_action_button.dart` (+30 -14) </details> ### 📄 Description - change the height of each item to be 52px, less padding of 4px vertically to have 44px height for the inkwell widget - change the thickness of divider to 0.5px - used in: - view actions in homepage after sliding - view actions in the more appbar button - database card detail actions - database view actions. https://github.com/AppFlowy-IO/AppFlowy/assets/71320345/5536ce68-84a0-4761-8c2f-21f61e7996f5 ### Feature Preview  ---  #### PR Checklist - [x] 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. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>