Skip to content

feat: Open NetBird app when long-pressing the Quick Settings Tile - #222

Open
CodeName-Anti wants to merge 1 commit into
netbirdio:mainfrom
CodeName-Anti:feature/tile-longpress
Open

feat: Open NetBird app when long-pressing the Quick Settings Tile#222
CodeName-Anti wants to merge 1 commit into
netbirdio:mainfrom
CodeName-Anti:feature/tile-longpress

Conversation

@CodeName-Anti

@CodeName-Anti CodeName-Anti commented Jul 26, 2026

Copy link
Copy Markdown

This Pull request adds the ability to directly open the NetBird app by long-pressing the Quick Settings Tile.


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

Summary by CodeRabbit

  • New Features
    • Added support for opening quick settings tile preferences directly from the system quick settings panel.

Copilot AI review requested due to automatic review settings July 26, 2026 23:18
@coderabbitai

coderabbitai Bot commented Jul 26, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 0e894c8d-5839-40f2-b13d-e23fe5475b4f

📥 Commits

Reviewing files that changed from the base of the PR and between 99e998e and 3de3c86.

📒 Files selected for processing (1)
  • app/src/main/AndroidManifest.xml

📝 Walkthrough

Walkthrough

The Android manifest adds a Quick Settings tile preferences intent filter to MainActivity.

Changes

Quick Settings preferences

Layer / File(s) Summary
Register Quick Settings preferences action
app/src/main/AndroidManifest.xml
Adds android.service.quicksettings.action.QS_TILE_PREFERENCES handling to MainActivity.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related issues

  • netbirdio/android-client#216 — The manifest action supports the requested Quick Settings tile long-press behavior, but does not implement launching the app or displaying the active profile.

Suggested reviewers: copilot

Poem

I’m a quick little rabbit, hopping with cheer,
A tile preference path now appears here.
One manifest filter, neat and bright,
Guides the long-press into sight.
Hop, hop—Android knows the right door!

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly matches the change: opening the NetBird app from a long-press on the Quick Settings Tile.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR enables opening the NetBird app directly when the user long-presses the Quick Settings tile by registering an activity to handle the QS_TILE_PREFERENCES intent.

Changes:

  • Add an intent-filter to MainActivity for android.service.quicksettings.action.QS_TILE_PREFERENCES.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread app/src/main/AndroidManifest.xml
@CodeName-Anti

Copy link
Copy Markdown
Author

Related to #216

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants