Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Android GoBack with forcefully close the app even if the keyboard is up #4257

Closed
2 tasks done
alexballas opened this issue Sep 16, 2023 · 3 comments
Closed
2 tasks done
Labels
blocker Items that would block a forthcoming release bug Something isn't working

Comments

@alexballas
Copy link
Contributor

alexballas commented Sep 16, 2023

Checklist

  • I have searched the issue tracker for open issues that relate to the same problem, before opening a new one.
  • This issue only relates to a single bug. I will open new issues for any other problems.

Describe the bug

Hello,
Since v2.4.0 GoBack action will forcefully close the app even if they keyboard is up.

How to reproduce

  1. Create an app with a textbox
  2. Select the textbox. The keyboard appears.
  3. Issue a back action

Expected:
The keyboard should collapse.

What happens:
The app exits.

Screenshots

No response

Example code

package main

import (
	"log"

	"fyne.io/fyne/v2/app"
	"fyne.io/fyne/v2/container"
	"fyne.io/fyne/v2/widget"
)

func main() {
	myApp := app.New()
	myWindow := myApp.NewWindow("Entry Widget")

	input := widget.NewEntry()
	input.SetPlaceHolder("Enter text...")

	content := container.NewVBox(input, widget.NewButton("Save", func() {
		log.Println("Content was:", input.Text)
	}))

	myWindow.SetContent(content)
	myWindow.ShowAndRun()
}

Fyne version

v2.4.0

Go compiler version

1.21.1

Operating system and version

Ubuntu 18.04 (compiling) / Android 10

Additional Information

No response

@alexballas alexballas added the unverified A bug that has been reported but not verified label Sep 16, 2023
@Jacalz
Copy link
Member

Jacalz commented Sep 16, 2023

It looks like you forgot to add the example code. Would you mind updating the description?

@alexballas
Copy link
Contributor Author

Hello @Jacalz ,I copy pasted the example from https://developer.fyne.io/widget/entry
Any app with an entry widget should do (and assuming you don't do any special handling with w.Canvas().SetOnTypedKey)

@andydotxyz andydotxyz added bug Something isn't working blocker Items that would block a forthcoming release and removed unverified A bug that has been reported but not verified labels Sep 19, 2023
@andydotxyz andydotxyz added this to the D fixes (v2.4.x) milestone Sep 19, 2023
andydotxyz added a commit to andydotxyz/fyne that referenced this issue Dec 10, 2023
@andydotxyz
Copy link
Member

Fixed on develop and will be in v2.4.3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocker Items that would block a forthcoming release bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants