Lbry-desktop: Shouldn't allow to open multiple export and choose file dialogs

Created on 24 Mar 2018  路  10Comments  路  Source: lbryio/lbry-desktop

The Issue

User can open many "Choose file" windows to select files to publish
Same for "Choose directory" in Settings page
Same for "EXPORT" in History page (transaction history) and Overview page

Steps to reproduce

  1. Click on "PUBLISH" button
  2. Click on "CHOOSE FILE" button ==> "Choose file" window will display
  3. Click on "CHOOSE FILE" button again ==> another "Choose file" window will display
  4. Repeat step 3 many time ==> many "Choose file" windows will display

Expected behaviour

User should open only one "Choose file" windows, and it will be in top selection, User cannot focus on the Publish page after selected file success or close the "Choose file" windows

Actual behaviour

User can open many "Choose file" windows by clicking on "Choose file" button.
Same for "Choose directory" in Settings page
Same for "EXPORT" in History page (transaction history)

System Configuration

  • LBRY Daemon version: 0.19.1
  • LBRY App version: 0.21.2
  • LBRY Installation ID: N/A
  • Operating system: Windows (Windows-10-10.0.16299)
  • Wallet (lbryum): 3.2.0

Anything Else


Same for "Choose directory" in Settings page
Same for "EXPORT" in History page (transaction history) and Overview page

Screenshots


N/A

creator ux good first issue help wanted 1 improvement

Most helpful comment

He's clicking it really quickly... I'll try on windows. This is an edge case though, someone won't click like that :)

All 10 comments

@namujan78 thanks for the detailed write-up! I've sent you a tip on Discord :)

Fixed and merged in master.

@tzarebczan This issue still happens when you quickly click multiple times on CHOOSE FILE after the PUBLISH screen appeared.
CC @btzr-io
image

I can confirm this is working on linux, not sure about macOS,
Maybe this is a windows-bug ?

He's clicking it really quickly... I'll try on windows. This is an edge case though, someone won't click like that :)

Clicking really quick freezes the window:
https://github.com/electron/electron/issues/11323

It鈥檚 not freeze, but the app become slow responsive in some seconds, then back to normal after open multiple choose file windows.

One more thing, it happen only for first time you go to Publish app.
Next time it doesn鈥檛 happen. Maybe the app load complete all components successfull.

It doesn't happen on macOS.

@tzarebczan How about the status of this issue?
I think we should change the Closed to open/in progress if you still want fix it.
If not we just ignore this case.

Procedure of this case is:

  • Click on "PUBLISH" button then quickly multi click on "Choose file" when you can see "Choose file" button ==> Multi windows will appear

I don't have macOS so that I cannot confirm it happen or not on macOS. As well as Ubuntu.

CC @btzr-io @seanyesmunt

Was this page helpful?
0 / 5 - 0 ratings

Related issues

tzarebczan picture tzarebczan  路  16Comments

tzarebczan picture tzarebczan  路  18Comments

btzr-io picture btzr-io  路  11Comments

fbruetting picture fbruetting  路  28Comments

kauffj picture kauffj  路  16Comments