MAUI: first steps

[webwarrior-ws]

Supersedes #241

[webwarrior-ws]

Added "Upload APK as CI artifact" step

[knocte]

In PR241, the 1st commit passes CI (it's green), but not here. I think when you pushed on Thursday EOD, it was also green here, but something you must have changed today to not be green anymore.

[knocte]

UI nits I found:

• Please add some separation between the button "More info" and the button "Ok"
• Please add some separation between the DOB label and the DOB picker.

[knocte]

I tried debugging the Android MAUI app in VS4Mac today and it seems to hang after submitting the payment password (it never goes to next page: loadingPage). Have you tested the Android MAUI app?

[webwarrior-ws]

Yes. Android MAUI app was fine for me. I can clear data and launch the app to see if it hangs after submitting payment password.

[webwarrior-ws]

As for UI: I think style should be modified to fix things like no margins around buttons and input fields not distinguishable from background.

[knocte]

Yes. Android MAUI app was fine for me. I can clear data and launch the app to see if it hangs after submitting payment password.

Nevermind, it worked for me in the end; it was not hanging, just extremely slow. We'll look into this later.

Another thing I've noticed wrt UI that needs to be fixed (is not a nit):

See? ReceivePage doesn't have a "back" button.

[knocte]

I think style should be modified to fix things like no margins around buttons and input fields not distinguishable from background.

What do you mean?

[knocte]

More from above:

• Some separation between currency dropdown and amount textbox.
• I think I recall we already talked about borders of textboxes in the past? can you remind me what was the deal here.

[knocte]

Some separation between currency dropdown and amount textbox.

Also separation between "Amount:" label and "Not enough funds" label.

[webwarrior-ws]

The problem with textboxes is that they are basically invisible. For example, in previous screenshot there is input field below "Destination address:" label, but you can't see it. They same is for input field right to the "Password:".

[webwarrior-ws]

ReceivePage didn't have any extra back button. But you can always use standard Android back button or gesture.

[knocte]

The problem with textboxes is that they are basically invisible. For example, in previous screenshot there is input field below "Destination address:" label, but you can't see it. They same is for input field right to the "Password:".

I know that, why are you rephrasing the problem that I already described? I'm saying that we discussed this in the past (but not just the problem but we discussed solutions).

[knocte]

ReceivePage didn't have any extra back button. But you can always use standard Android back button or gesture.

what??? the point of using navigation is to have those buttons automatically. Don't we have this in RIM or DG?

[knocte]

what??? the point of using navigation is to have those buttons automatically. Don't we have this in RIM or DG?

See https://learn.microsoft.com/en-us/dotnet/maui/user-interface/pages/navigationpage?view=net-maui-8.0 (the "back button)

[webwarrior-ws]

I checked, and indeed we use navigation in GWallet, but no toolbar is shown for some reason. I'll investigate.

[knocte]

Don't we have this in RIM or DG?

Confirmed, it is in DG, after pressing button "Add event".

[knocte]

@webwarrior-ws so, summary for today's priority list (items are the top should be done earlier, bottom to be done later): 1) Fix root cause of MauiGtk bug jumping label (in order to get rid of the workaround) in CrossMaui (Mali). More details above. 2) Create a CI job called snap_alpha that uploads the Maui+Gtk app as a snap package. This job should upload the snap package to snap store directly. As you don't have permissions to upload binaries to my snaps, use a different snap name: geeMauiPreview, for now. 3) Resume work on Android's toolbar's back button crash, by trying to upgrade again to Maui8 (dropping the PkgRef as I told you in telegram).

Thanks

[knocte]

These two commits should obviously be squashed too:

• Fixup: commit to be squashed (cosmetic stuff)
• Forntend(XF,Maui): addressed comments

[knocte]

Superseded by https://github.com/nblockchain/geewallet/pull/279