Skip to content

Navigation: Delegate the back navigation to the OS if there is no screen to navigate back to.#180

Closed
CodeRoutineDev wants to merge 1 commit into
pynicolas:mainfrom
CodeRoutineDev:fix-back-button-on-cam-screen
Closed

Navigation: Delegate the back navigation to the OS if there is no screen to navigate back to.#180
CodeRoutineDev wants to merge 1 commit into
pynicolas:mainfrom
CodeRoutineDev:fix-back-button-on-cam-screen

Conversation

@CodeRoutineDev
Copy link
Copy Markdown

This is a fix for the issue #179 which I immediately noticed after installing the app.

Since this is such a small PR, I figured simply opening it makes discussion easier than discussing any changes up front. Let me know what you think.

Heads up for the great work on this app. I am really excited about it.

@CodeRoutineDev
Copy link
Copy Markdown
Author

Oh yeah, and this is my first PR on GitHub ever, so let me know if something is missing.

@pynicolas
Copy link
Copy Markdown
Owner

Thanks for opening this PR!
Can you please describe the concrete problem that you're trying to fix?
If it's about the arrow icon at the top-left of the camera screen, an issue (#179) was created yesterday because this button doesn't trigger anything (this problem was introduced in version 1.21.0): I'm going to remove this button.
I believe that a system "back" (like a "back" gesture) works correctly: on the camera screen, it brings back to the OS.

@pynicolas
Copy link
Copy Markdown
Owner

Sorry, I read the description of your PR too fast. So, yes, you're trying to fix #179.
And as I wrote, I prefer to not display the arrow icon when it should bring the user back to the OS.
I created #181 for that.
Thanks again for your PR!

@pynicolas pynicolas closed this May 18, 2026
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