[PR #172] [MERGED] [core] Wrap Actions in SafeArea Widget #171

Closed
opened 2025-10-31 16:58:46 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/feeddeck/feeddeck/pull/172
Author: @ricoberger
Created: 6/8/2024
Status: Merged
Merged: 6/8/2024
Merged by: @ricoberger

Base: mainHead: wrap-actions-in-safearea-widget


📝 Commits (1)

  • 21079be [core] Wrap Actions in SafeArea Widget

📊 Changes

4 files changed (+207 additions, -199 deletions)

View changed files

📝 app/lib/widgets/item/details/utils/item_videos.dart (+43 -41)
📝 app/lib/widgets/item/preview/utils/item_actions.dart (+64 -62)
📝 app/lib/widgets/settings/accounts/utils/settings_accounts_actions.dart (+50 -48)
📝 app/lib/widgets/settings/profile/settings_profile_signout.dart (+50 -48)

📄 Description

With this commit we are wrapping all action widgets in a SafeArea widget, to improve the alignment of the actions.


🔄 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/feeddeck/feeddeck/pull/172 **Author:** [@ricoberger](https://github.com/ricoberger) **Created:** 6/8/2024 **Status:** ✅ Merged **Merged:** 6/8/2024 **Merged by:** [@ricoberger](https://github.com/ricoberger) **Base:** `main` ← **Head:** `wrap-actions-in-safearea-widget` --- ### 📝 Commits (1) - [`21079be`](https://github.com/feeddeck/feeddeck/commit/21079be8e2731d7d68e15e43e7c98ed91242d881) [core] Wrap Actions in `SafeArea` Widget ### 📊 Changes **4 files changed** (+207 additions, -199 deletions) <details> <summary>View changed files</summary> 📝 `app/lib/widgets/item/details/utils/item_videos.dart` (+43 -41) 📝 `app/lib/widgets/item/preview/utils/item_actions.dart` (+64 -62) 📝 `app/lib/widgets/settings/accounts/utils/settings_accounts_actions.dart` (+50 -48) 📝 `app/lib/widgets/settings/profile/settings_profile_signout.dart` (+50 -48) </details> ### 📄 Description With this commit we are wrapping all action widgets in a `SafeArea` widget, to improve the alignment of the actions. <!-- Keep PR title verbose enough and add prefix telling about what source it touches e.g "[rss] Add feature xyz" or if the the PR is not realated to a source use "[core]", e.g. "[core] Fix xyz". If you add a breaking change within your PR you should add ":warning:" to the title, e.g. ":warning: [core] My breaking change" --> <!-- Description of what have been changed. Please also reference an issue, when available. --> --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the pull-request label 2025-10-31 16:58:46 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/feeddeck#171