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

Account > Wallet Seed view is broken in legacy UI [0.15 XMR] #527

Closed
woodser opened this issue Jan 7, 2023 · 13 comments
Closed

Account > Wallet Seed view is broken in legacy UI [0.15 XMR] #527

woodser opened this issue Jan 7, 2023 · 13 comments
Labels
💰bounty There is a bounty on this issue

Comments

@woodser
Copy link
Contributor

woodser commented Jan 7, 2023

The Account > Wallet Seed view in the legacy UI is broken.

It's not displaying the mnemonic seed phrase, and the date input boxes grow indefinitely.

@woodser woodser changed the title Account > Wallet Seed view is broken in legacy UI Account > Wallet Seed view is broken in legacy UI [0.15 XMR] Jan 7, 2023
@woodser woodser added the 💰bounty There is a bounty on this issue label Jan 7, 2023
@github-actions
Copy link

github-actions bot commented Jan 7, 2023

There is a bounty on this issue, the amount is in the title. The reward will be awarded to the first person or group of people who resolves this issue.

If you are starting to work on this bounty, please write a comment, so that we can assign the issue to you. We expect contributors to provide a PR in a reasonable time frame or, in case of an extensive work, updates on their progresses. We will unassign the issue if we feel the assignee is not responsive or has abandoned the task.

Read the full conditions and details of our bounty system.

@woodser
Copy link
Contributor Author

woodser commented Jan 7, 2023

Related to bisq-network/bisq#5607

@napoly
Copy link
Contributor

napoly commented Jan 7, 2023

This issue seems to be linked to macOS javafx-controls-16-mac only as on Linux javafx-controls-16-linux.jar it seems OK.

@preland
Copy link
Contributor

preland commented Aug 18, 2023

I’ll look into this one

@preland
Copy link
Contributor

preland commented Jan 13, 2024

I was unable to reproduce the mnemonic seed not rendering; although I was able to reproduce and (somewhat) solve the growing date input boxes. Here are the notes I have so far, both regarding the datePicker issue, and some other... oddities I found with the Account Seed View:

P.S. whenever I say "datePicker" I am most likely referring to JFXDatePicker. DatePicker does not have the same issues.

-The issue occurs in both datePicker elements in Seed View. They are the only dataPicker elements currently present in Haveno.
-Width isn't modified at instantiation; this means that setting MaxWidth for the element changes nothing except for the starting point of the growth
-Growth isn't constrained or influenced by window size
-The bug occurs for all UI users regardless of user type (saying this in case the "legacy UI" mentioned in the issue title is implying that this issue only occurs for arbitrators)
-For some reason, the growth will become paused when moved from a higher resolution monitor to a lower resolution monitor. In addition, the issue doesn't seem to occur on the lower resolution at all (1080p). The growth continues Most (see next) of the time when moved back to a higher resolution.
-Sometimes, when moved from lower to higher resolution quickly, the growth will remain stopped. The growth continues once the window is updated in any way (a button is clicked, a pop-up occurs, etc.)
-I could not find a way to reduce or reset the size of the datePicker after growth. Moving to another screen would stop it, but it would continue from the same size once looked at again. Restarting is the only way to (temporarily) return the size back to normal.

-As is described in my PR, I was unable to use a seed to recover a wallet. The button would remain greyed out, even after all information was provided. I was also unable to see the date of the current wallet seed, which would also potentially be part of the issue.

-Using the control buttons to change tabs (Ctrl-{1-8}) from the Account tab to the Account tab again (so Ctrl-8, Ctrl-8) would cause the currently active view to unload entirely

That was what I was able to find looking at it today. I'll send the relevant findings over to JFoenix in regards to JFXDatePicker, although it seems that JFXDatePicker was made in March of 2016, and is still on version 1.0, which could be a sign that a fix isn't coming anytime soon on their end.

@preland
Copy link
Contributor

preland commented Jan 13, 2024

Just so we are on the same page as to this issue; this is what I see (pretend the datePickers are JFXDatePickers-the behavior is the same with both)

Should I be able to see a date in the top "Wallet date" field? Is this a bug, or intended/unimplemented behavior?

This behavior is also present when running a full "screen" testnet (1 seednode, 2 monerod, 2 users, and 1 arbitrator), and it also occurs on stagenet.

Screenshot 2024-01-13 at 5 28 25 PM

@woodser
Copy link
Contributor Author

woodser commented Jan 14, 2024

That behavior isn't implemented yet, so it's expected. I'm looking into it, though it will be nice when the date picker isn't growing :)

@preland
Copy link
Contributor

preland commented Jan 14, 2024

Ok, I was thinking that was the case since there wasn’t a usable function to get the date :)

I am unable to reproduce the mnemonic seed not appearing; is this still an issue?

@woodser
Copy link
Contributor Author

woodser commented Jan 14, 2024

I am unable to reproduce the mnemonic seed not appearing; is this still an issue?

No, the seed appears for me.

@preland
Copy link
Contributor

preland commented Jan 14, 2024

I am unable to reproduce the mnemonic seed not appearing; is this still an issue?

No, the seed appears for me.

In that case, is it okay if we treat that part of the issue as resolved unless someone else is still having the issue?

@woodser
Copy link
Contributor Author

woodser commented Jan 14, 2024

Yes, we can complete this issue and I've opened a separate issue for restoring from seed:

#752

@preland
Copy link
Contributor

preland commented Jan 14, 2024

Good, I'll look at that issue after this

@preland
Copy link
Contributor

preland commented Mar 22, 2024

(Payment was received, forgot to confirm)

@woodser woodser closed this as completed Mar 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
💰bounty There is a bounty on this issue
Projects
None yet
Development

No branches or pull requests

3 participants