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

[Bug] GTK: reenabling button doesn't work if page is a not-initial NavigationPage #9092

Closed
knocte opened this issue Jan 6, 2020 · 0 comments
Closed

Comments

@knocte
Copy link
Contributor

@knocte knocte commented Jan 6, 2020

Steps to Reproduce

  1. Clone https://github.com/knocte/geewallet , switch to branch bugreport/gtkButtonDisabling.
  2. Run the GTK frontend.
  3. Click on the button "FOO" to go to next page.
  4. Click on the button "BAR": an ALERT modal dialog will show the text "BAZ".
  5. Click "OK" to dismiss the alert.

Expected Behavior

(What happens if you run any other frontend, such as the Android one: ) The button "BAR" was disabled and reenabled once the dialog was shown, and its text changed to "reenabled". The button can be clicked again. (If you don't use navigation pages, this is also the actual behaviour in GTK.)

Actual Behavior

The button is still disabled and can't be clicked.

Basic Information

  • Version with issue: 4.3.0.991211
  • Last known good version: ?
  • IDE: VS4Mac 8.3.11
  • Platform Target Frameworks:
    • Android: v9.0
  • Android Support Library Version: 28.0.0.3
  • Affected Devices: GTK

Reproduction Link

https://github.com/knocte/geewallet/tree/bugreport/gtkButtonDisabling

@pauldipietro pauldipietro added this to New in Triage Jan 6, 2020
@samhouts samhouts added the p/gtk label Jan 6, 2020
@samhouts samhouts added this to To do in Other Ready For Work via automation Jan 6, 2020
@samhouts samhouts removed this from New in Triage Jan 6, 2020
@z3ut z3ut mentioned this issue Jan 14, 2020
0 of 2 tasks complete
@rmarinho rmarinho closed this in 3fa35e9 Feb 4, 2020
Other Ready For Work automation moved this from To do to Done Feb 4, 2020
knocte added a commit to nblockchain/geewallet that referenced this issue Feb 6, 2020
knocte added a commit to nblockchain/geewallet that referenced this issue Feb 6, 2020
knocte added a commit to nblockchain/geewallet that referenced this issue Feb 11, 2020
In order to get the bugfix for:
xamarin/Xamarin.Forms#9092

# Conflicts:
#	src/GWallet.Frontend.XF.Android/packages.config
#	src/GWallet.Frontend.XF.Gtk/GWallet.Frontend.XF.Gtk.fsproj
#	src/GWallet.Frontend.XF.Gtk/packages.config
#	src/GWallet.Frontend.XF.Mac/packages.config
#	src/GWallet.Frontend.XF.iOS/packages.config
@samhouts samhouts added this to In Progress in vNext (4.5.0) Feb 12, 2020
@samhouts samhouts moved this from In Progress to Done in vNext (4.5.0) Feb 12, 2020
knocte added a commit to nblockchain/geewallet that referenced this issue Feb 12, 2020
In order to get the bugfix for:
xamarin/Xamarin.Forms#9092

# Conflicts:
#	src/GWallet.Frontend.XF.Android/packages.config
#	src/GWallet.Frontend.XF.Gtk/GWallet.Frontend.XF.Gtk.fsproj
#	src/GWallet.Frontend.XF.Gtk/packages.config
#	src/GWallet.Frontend.XF.Mac/packages.config
#	src/GWallet.Frontend.XF.iOS/packages.config
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
vNext (4.5.0)
  
Done
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.