GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — settings: Add setting for auto-opening apps by vkareh · Pull Request #2004 · InfiniTimeOrg/InfiniTime

Conversation

@vkareh

@vkareh vkareh commented Feb 6, 2024 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Contributor

MusicService: Autostart music app on music start
When starting music on a paired device, the music service will automatically trigger the music app to open.
edit: Repurposed this PR to a more general feature for auto-opening apps.

Certain apps can be configured to auto-open based on component events:

  • battery: when charging the watch
  • music: when playing music from the paired device
  • navigation: when opening a maps app on the paired device

InfiniSim_2024-02-09_142706

Fixes #1619

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
@github-actions

github-actions Bot commented Feb 6, 2024 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown

Build checks have not completed. Possible reasons for this are:

  1. The checks need to be approved by a maintainer
  2. The branch has conflicts
  3. The firmware build has failed
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@vkareh
vkareh force-pushed the music-app-autostart branch from 6d8e732 to e9df27a Compare February 7, 2024 14:14
@mark9064

mark9064 commented Feb 7, 2024 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Member

I like the sound of this feature, but I think it should be optional. I have my phone and computers all connected with KDE connect, so any audio I play on any device syncs to my phone which sends it to InifiniTime. This is great, but making it switch to the music app every time any audio/media is played anywhere would drive me insane as it'd be switching all the time

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@FintasticMan

FintasticMan commented Feb 8, 2024

Copy link
Copy Markdown
Member

I personally don't think that this would be a very nice user experience. As Mark says, I play media on my phone quite a lot, which will have it start and stop quite frequently. If that causes it to switch to the music app each time, it would get quite annoying. I think that this would be better implemented as a quick-switch action, where you can configure an app to get to quickly.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@vkareh
vkareh force-pushed the music-app-autostart branch from e9df27a to dfb4e34 Compare February 9, 2024 19:26
@vkareh

vkareh commented Feb 9, 2024

Copy link
Copy Markdown
Contributor Author

I've added the ability to configure which apps auto-open based on certain events:

  • battery: when charging the phone (code directly from Open battery app when charging #1876)
  • music: when playing music on paired device (tested with Gadgetbridge / Spotify Lite)
  • navigation: when starting map navigation on paired device (tested with Gadgetbridge / OsmAnd~)

InfiniSim_2024-02-09_142706

KaffeinatedKat and mark9064 reacted with heart emoji
All reactions
  • ❤️ 2 reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@vkareh
vkareh force-pushed the music-app-autostart branch 2 times, most recently from aef969d to da53d2d Compare February 9, 2024 19:32
@vkareh vkareh changed the title MusicService: Autostart music app on music start settings: Add setting for auto-opening apps Feb 9, 2024
@mark9064

mark9064 commented Feb 10, 2024

Copy link
Copy Markdown
Member

I like this generic solution a lot, my preferred watch face has battery percentage on it so being able to turn off auto battery show would be nice

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

FintasticMan
FintasticMan requested changes Feb 10, 2024
View reviewed changes

@FintasticMan FintasticMan left a comment

Copy link
Copy Markdown
Member

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

I like it!

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
Comment thread src/components/ble/NavigationService.h Resolved

Uh oh!

There was an error while loading. Please reload this page.

@vkareh
vkareh force-pushed the music-app-autostart branch from da53d2d to d77f02c Compare February 10, 2024 15:35
@vkareh
vkareh requested a review from FintasticMan February 10, 2024 15:36
FintasticMan
FintasticMan approved these changes Feb 10, 2024
View reviewed changes
@JF002

JF002 commented Feb 11, 2024 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Collaborator

I understand that such a feature might be useful in some cases but... I would like to challenge it for the following reasons:

  • UX : when enabled, the Navigation/Music/Battery app would be automatically loaded when a event (new direction, new music data, new charging state) is received. It means that the state of the application that is currently running would be erased. Let's say I'm playing a game, changing some settings or using the chrono -> I lost everything I was doing before the event. That would be kind of infuriating to me.

  • Code : I don't think this implementation is compatible with the new compile time configuration of applications and watch faces : what happens in the Navigation or Music app are not built into the firmware?

  • InfiniTime Vision : In my opinion, this feature is not consistent with a few core principles of the InfiniTime project. For example, we avoid adding new settings whenever possible and we favor implementing a single solid default behavior instead.

As I said, I understand why some users request this kind of feature, but I'm not sure this is the best way to implement it. Feel free to share your opinions :)

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@vkareh
vkareh force-pushed the music-app-autostart branch 2 times, most recently from a319aec to d5d6c62 Compare February 12, 2024 15:19
@ghost

ghost commented Feb 12, 2024

Copy link
Copy Markdown

Some of my thoughts about this feature:

It should only automatically open apps while the smartwatch is not used for something else; so, basically only while it is on the watch-face.

Considering that most of this feature depends on interacting with a companion-app, i think that it may be better to have a way for a companion-app to request opening any app on the smartwatch and keep the settings for wether or not to do it in that companion-app. It would also be good if the companion-app can check for the availability of a smartwatch-app, so it can decide wether or not to offer such a setting, or tell the user how to make it available.

Automatically opening an app can make the usage-flow much easier.
For example: just start navigation in a smartphone-app, and one is ready to go; instead of having to change things on 2 devices, going through several screens on each device.
I also consider it to be an accessibility-feature; it basically makes such functionality usable/discoverable to begin with, for various people who do not have toying-with-tech as a hobby.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@vkareh
vkareh force-pushed the music-app-autostart branch from d5d6c62 to 1da02a1 Compare February 12, 2024 15:50
@vkareh

vkareh commented Feb 12, 2024

Copy link
Copy Markdown
Contributor Author

@JF002

UX : [...] It means that the state of the application that is currently running would be erased [...]

I updated the patchset to only load the app if there's no app loaded currently.

Code : I don't think this implementation is compatible with the new compile time configuration of applications and watch faces : what happens in the Navigation or Music app are not built into the firmware?

Great question. Testing this now to see what happens.

[...] we avoid adding new settings whenever possible and we favor implementing a single solid default behavior instead.

Fair point. I think this concern is the only one we cannot address in code and would require a decision from the InfiniTime core team (InfiniTeam? 😛 )

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@vkareh
vkareh force-pushed the music-app-autostart branch from 1da02a1 to 9c8432f Compare February 12, 2024 19:10
@vkareh

vkareh commented Feb 12, 2024

Copy link
Copy Markdown
Contributor Author

@JF002

  • Code : I don't think this implementation is compatible with the new compile time configuration of applications and watch faces : what happens in the Navigation or Music app are not built into the firmware?

I've accounted for this now. The DisplayApp verifies if the app is available before attempting to perform the action.

@MoonlightWave-12

It should only automatically open apps while the smartwatch is not used for something else; so, basically only while it is on the watch-face.

This is now fixed.

[...] have a way for a companion-app to request opening any app on the smartwatch [...]

This could be interesting... Gadgetbridge has device-specific settings. InfiniTime could implement a service that allows it to interact with apps.

[...] or tell the user how to make it available [...]

General users won't be able to decide which app gets installed on their watch.

Automatically opening an app can make the usage-flow much easier.

I agree. This feature's goal is to increase usability. When I play music/podcasts, I expect my watch to know so that I can see/interact. Same with navigation. My companion app (Gadgetbridge) is smart enough that it doesn't just randomly sends any sound/notification as though it were media, and limits it only to actual media playing on my phone (Spotify Lite / AntennaPod). Things like movies from streaming services do not get sent to the watch (which is what I expect anyway), so the behaviour here is really what I would expect of a connected device.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@vkareh
vkareh force-pushed the music-app-autostart branch 2 times, most recently from 79ea5ec to f9c2dce Compare February 19, 2024 01:10
@vkareh
vkareh force-pushed the music-app-autostart branch from f9c2dce to 87c6973 Compare March 18, 2024 12:51
@vkareh
vkareh force-pushed the music-app-autostart branch from 87c6973 to 6987f23 Compare March 26, 2024 12:15
@vkareh
vkareh force-pushed the music-app-autostart branch from 6987f23 to 9d34d25 Compare April 12, 2024 13:00
@vkareh
vkareh force-pushed the music-app-autostart branch from 9d34d25 to 25d2f45 Compare May 13, 2024 14:55
@vkareh
vkareh force-pushed the music-app-autostart branch from 25d2f45 to efe295e Compare June 10, 2024 14:34
@vkareh
vkareh force-pushed the music-app-autostart branch from efe295e to 777badf Compare June 19, 2024 15:30
@tituscmd

tituscmd commented Sep 3, 2024

Copy link
Copy Markdown
Contributor

I was just thinking... what if the app doesn't open right up, but gets placed to the right of the watchface?

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@vkareh
vkareh force-pushed the music-app-autostart branch 2 times, most recently from 9da21a2 to c73f8a9 Compare September 23, 2024 13:06
@vkareh
vkareh force-pushed the music-app-autostart branch from c73f8a9 to d7f2e15 Compare October 3, 2024 12:35
@vkareh
vkareh force-pushed the music-app-autostart branch 2 times, most recently from 5ee8054 to 1c569fd Compare October 28, 2024 14:46
@mark9064 mark9064 added the new feature This thread is about a new feature label Nov 18, 2024
@vkareh
vkareh force-pushed the music-app-autostart branch from 1c569fd to 28bed03 Compare November 20, 2024 14:18
@vkareh
vkareh force-pushed the music-app-autostart branch from 28bed03 to c225068 Compare December 4, 2024 18:18
@vkareh
vkareh force-pushed the music-app-autostart branch from c225068 to a514e9e Compare December 16, 2024 15:42
@vkareh
vkareh force-pushed the music-app-autostart branch from a514e9e to f099871 Compare January 6, 2025 14:24
@vkareh
vkareh force-pushed the music-app-autostart branch from f099871 to cdf46c3 Compare January 29, 2025 18:25
@vkareh
vkareh force-pushed the music-app-autostart branch from cdf46c3 to 96959c5 Compare February 28, 2025 14:38
@vkareh
vkareh force-pushed the music-app-autostart branch from 96959c5 to e2d7036 Compare May 14, 2025 14:23
@vkareh
vkareh force-pushed the music-app-autostart branch from e2d7036 to 871e7f2 Compare May 27, 2025 14:49
@mark9064

mark9064 commented May 27, 2025

Copy link
Copy Markdown
Member

Rebase to fix build CI, sorry (same for other PRs)

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@vkareh
vkareh force-pushed the music-app-autostart branch from 871e7f2 to d56505b Compare May 30, 2025 16:31
@mark9064

mark9064 commented May 30, 2025

Copy link
Copy Markdown
Member

After thinking about this for a while I'm in a similar space to JF. I think this isn't a good fit in terms of the vision and that either

  1. integrations (e.g current track playing) should be a feature of watchfaces themselves
  2. if an app should load by default (e.g the firmware update app), there need be no configuration option
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@vkareh
vkareh force-pushed the music-app-autostart branch from d56505b to 6f039a9 Compare June 4, 2025 17:18
@vkareh
vkareh force-pushed the music-app-autostart branch 2 times, most recently from 543bf02 to 84edc2f Compare June 19, 2025 17:50
@vkareh
vkareh force-pushed the music-app-autostart branch from 84edc2f to d11a774 Compare June 30, 2025 14:20
KaffeinatedKat and others added 5 commits July 31, 2025 08:36
@KaffeinatedKat @vkareh
feat: open battery app when charging
1d2864c
@vkareh
DisplayApp: Add function to determine if app is available
e356c16
@vkareh
MusicService: Autostart music app on music start
88086c3
When starting music on a paired device, the music service will
automatically trigger the music app to open if there is no app already
loaded.
@vkareh
NavigationService: Autostart navigation app on nav command
e5bebfa
When a paired device sends a navigation message, the navigation service
will automatically trigger the navigation app to open if there is no app
already loaded.
@vkareh
settings: Add setting for auto-opening apps
bec7853
Certain apps can be configured to auto-open based on component events:
- battery: when charging the watch
- music: when playing music from the paired device
- navigation: when opening a maps app on the paired device
@vkareh
vkareh force-pushed the music-app-autostart branch from d11a774 to bec7853 Compare July 31, 2025 12:36
@vkareh

vkareh commented Nov 17, 2025

Copy link
Copy Markdown
Contributor Author

This does not align with InfiniTime's vision (and to be honest, I've stopped maintaining this branch since there are conflicts, as I don't use this feature myself). Closing.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@vkareh vkareh closed this Nov 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@FintasticMan FintasticMan FintasticMan approved these changes

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

new feature This thread is about a new feature

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

6 participants

@vkareh @mark9064 @FintasticMan @JF002 @tituscmd @KaffeinatedKat
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.