[PR #262] [MERGED] Improve openUrl Function #251

Closed
opened 2025-10-31 17:00:29 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/feeddeck/feeddeck/pull/262
Author: @ricoberger
Created: 5/11/2025
Status: Merged
Merged: 5/11/2025
Merged by: @ricoberger

Base: mainHead: improve-openurl-function


📝 Commits (1)

📊 Changes

1 file changed (+19 additions, -14 deletions)

View changed files

📝 app/lib/utils/openurl.dart (+19 -14)

📄 Description

Instead of using an external browser on Android, we are now using the
In-App-Browser to open all urls. For all other platforms we keep the
current behaviour and just set the launch mode explicitly, for the case
the the default is changed in the used package.

Closes #261


🔄 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/262 **Author:** [@ricoberger](https://github.com/ricoberger) **Created:** 5/11/2025 **Status:** ✅ Merged **Merged:** 5/11/2025 **Merged by:** [@ricoberger](https://github.com/ricoberger) **Base:** `main` ← **Head:** `improve-openurl-function` --- ### 📝 Commits (1) - [`be8fa8b`](https://github.com/feeddeck/feeddeck/commit/be8fa8bf7b6254ab6885507d7099929cdaaeb47e) Improve `openUrl` Function ### 📊 Changes **1 file changed** (+19 additions, -14 deletions) <details> <summary>View changed files</summary> 📝 `app/lib/utils/openurl.dart` (+19 -14) </details> ### 📄 Description Instead of using an external browser on Android, we are now using the In-App-Browser to open all urls. For all other platforms we keep the current behaviour and just set the launch mode explicitly, for the case the the default is changed in the used package. Closes #261 <!-- 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 17:00:29 -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#251