Skip to content

Add option to show on lock screen#1346

Merged
gerhardol merged 2 commits into
jonasoreland:masterfrom
candrew66:show-on-lock-screen
May 9, 2026
Merged

Add option to show on lock screen#1346
gerhardol merged 2 commits into
jonasoreland:masterfrom
candrew66:show-on-lock-screen

Conversation

@candrew66
Copy link
Copy Markdown
Contributor

@candrew66 candrew66 commented Apr 30, 2026

This option shows the recording screen while the device is locked for easier access to controls and is enabled by default.

Related: #439

@candrew66 candrew66 force-pushed the show-on-lock-screen branch 2 times, most recently from 341fd14 to 94f6525 Compare May 1, 2026 03:59
@candrew66 candrew66 changed the title Add show on lock screen option Add option to show on lock screen May 1, 2026
@gerhardol gerhardol requested a review from Copilot May 1, 2026 06:05
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a new “Show on lock screen” setting to let the recording screen remain visible while the device is locked, improving access to workout controls (related to #439).

Changes:

  • Introduces new UI strings and a new “Lock screen visibility” settings category.
  • Adds a pref_show_on_lock_screen preference key and a corresponding checkbox in Controls settings (default enabled).
  • Updates RunActivity to apply “show when locked” behavior across API levels.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 4 comments.

File Description
common/src/main/res/values/strings.xml Adds user-facing strings for the new lock screen visibility setting/category.
app/src/main/org/runnerup/view/RunActivity.java Applies the “show on lock screen” behavior via setShowWhenLocked/window flags and hooks it into lifecycle/back handling.
app/res/xml/settings_controls.xml Adds a new settings category and checkbox to control lock screen visibility.
app/res/values/pref_keys.xml Defines the new preference key pref_show_on_lock_screen.

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

Comment thread app/res/xml/settings_user_interface.xml
Comment thread app/src/main/org/runnerup/view/RunActivity.java Outdated
Comment thread app/src/main/org/runnerup/view/RunActivity.java Outdated
Comment thread app/src/main/org/runnerup/view/RunActivity.java Outdated
@candrew66 candrew66 force-pushed the show-on-lock-screen branch 3 times, most recently from fff769c to 77ad117 Compare May 2, 2026 01:58
@candrew66
Copy link
Copy Markdown
Contributor Author

candrew66 commented May 2, 2026

Would it be ok to rename the Controls settings category to User interface?
I'm thinking of doing a future PR to address #439 using FLAG_KEEP_SCREEN_ON and it makes more sense as a name, just wondering if there's a policy against changing settings names.

@gerhardol
Copy link
Copy Markdown
Collaborator

Would it be ok to rename the Controls settings category to User interface? I'm thinking of doing a future PR to address #439 using FLAG_KEEP_SCREEN_ON and it makes more sense as a name, just wondering if there's a policy against changing settings names.

Renaming is fine. This used to have options to control with play/stop (functionality in Adroid removed in 4.4 or so).

To always keep the screen on is something that Play store reviews tend to block...

Copy link
Copy Markdown
Collaborator

@gerhardol gerhardol left a comment

Choose a reason for hiding this comment

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

seem fine, have not run

Comment thread app/src/main/org/runnerup/view/RunActivity.java Outdated
@candrew66 candrew66 force-pushed the show-on-lock-screen branch from 77ad117 to dc5ec85 Compare May 4, 2026 01:36
@candrew66
Copy link
Copy Markdown
Contributor Author

candrew66 commented May 4, 2026

Renaming is fine. This used to have options to control with play/stop (functionality in Adroid removed in 4.4 or so).

Thanks, it's renamed in a second commit.

To always keep the screen on is something that Play store reviews tend to block...

I was going to add a Keep screen on option (off by default). I assume this is fine?

@candrew66
Copy link
Copy Markdown
Contributor Author

I've left the old translations for the Controls settings category up. Would it be better to remove them?

This option shows the recording screen while the device is locked for
easier access to controls and is enabled by default.

Related: jonasoreland#439
@candrew66 candrew66 force-pushed the show-on-lock-screen branch from dc5ec85 to 16947f5 Compare May 4, 2026 03:03
@gerhardol
Copy link
Copy Markdown
Collaborator

I've left the old translations for the Controls settings category up. Would it be better to remove them?

Please remove

To always keep the screen on is something that Play store reviews tend to block...

I was going to add a Keep screen on option (off by default). I assume this is fine?

The remark is on the permission itself. But we can try.

@gerhardol
Copy link
Copy Markdown
Collaborator

And translated files are updated on Transifex. I will push the new source files, will invalidate the old strings so old strings are removed and new are empty (using default language).

User interface better describes the contents of the settings category.
@candrew66 candrew66 force-pushed the show-on-lock-screen branch from 16947f5 to 421fe09 Compare May 4, 2026 08:37
@candrew66
Copy link
Copy Markdown
Contributor Author

Right, translations are gone now

@candrew66
Copy link
Copy Markdown
Contributor Author

I might keep this PR with only the show on lock screen option to keep it simpler. It should be pretty easy for someone else to do the keep screen on option if they want it.

@gerhardol
Copy link
Copy Markdown
Collaborator

I might keep this PR with only the show on lock screen option to keep it simpler. It should be pretty easy for someone else to do the keep screen on option if they want it.

If you prefer, you can do that in a follow up. I will just revert if it is a problem.

@gerhardol gerhardol merged commit f6b47dd into jonasoreland:master May 9, 2026
1 check passed
@candrew66 candrew66 deleted the show-on-lock-screen branch May 11, 2026 07:30
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.

3 participants