HeroicGamesLauncher icon indicating copy to clipboard operation
HeroicGamesLauncher copied to clipboard

[General UI/UX] Replace Native Dialogs with HTML Dialogs

Open flavioislima opened this issue 3 years ago • 5 comments

Problem description

Right now Heroic uses the native dialogs to show error messages or to prompt the user for uninstalling, repairing the game, warning in general. The problem is that those dialogs don't follow Heroic design and they feel like misplaced elements.

Feature description

Ideally, Heroic should have their own dialogs that work from inside it, just like the Install Dialog. So then it could follow the same design specifications.

Like on this example on Figma: Screenshot 2022-03-23 at 11 23 54

Link: https://www.figma.com/file/Zv89aBtLJ8yN4BwFS0AwNb/Heroic-Games?node-id=0%3A1

Alternatives

No response

Additional information

No response

flavioislima avatar Mar 23 '22 10:03 flavioislima

I think this requires a fair amount of changes, I think it shouldn't be labeled as good first issue.

I was checking and the current dialog we use comes from electron, not react. So, apart from a design change, this will also require to change how the frontend communicates with the backend when triggering those actions in many places.

arielj avatar Apr 18 '22 23:04 arielj

Yes, you're right, this is a big complex. One idea would be to have a PR only to implement the generic dialog and then we could replace the other dialogs in different PRs.

flavioislima avatar Apr 22 '22 14:04 flavioislima

So this should be not complex anymore. We have now the Dialog component and we can create as first step a Error Dialog and Message Dialog. We already have a ProgressDialog aswell which is used for winetricks and save-sync output.

Nocccer avatar Sep 12 '22 00:09 Nocccer

@biliesilva would need some examples to replace the one's bellow:

  • showMessageBox (Update info) image
  • showMessageBox but with a checkbox (uninstall windows game on Linux) image
  • Other example with a bigger message: image
  • One Button image
  • Three button (Stop installation) image

Some observations:

  • I think the dialogs don't need a title like the native ones.
  • The latest screenshot could have better UX:
    • This dialog is shown when we click on it to cancel the installation
    • The user has 3 options in this case.
    • Close the dialog and keep installing
    • Pause the installation (it is what keeping the files does)
    • Cancel the installation and remove the downloaded files.

So I believe we would need to support icons on the dialog buttons. Also, those dialogs could have an option to not close unless some of the options are chosen. So if we click on the backdrop it should do nothing. Or we can also use the default option when closing it.

flavioislima avatar Sep 24 '22 16:09 flavioislima

@Heroic-Games-Launcher/development I have increased the priority for this one since it really breaks the joystick experience in Heroic. Let me know if I am forgetting other dialogs.

flavioislima avatar Sep 24 '22 16:09 flavioislima

Addressed by #1891

BrettCleary avatar Oct 20 '22 18:10 BrettCleary

I just saw this message now. Do we still need the design? if yes, I can send it tomorrow.

biliesilva avatar Oct 20 '22 20:10 biliesilva

I just saw this message now. Do we still need the design? if yes, I can send it tomorrow.

I don't think so. This was already done you can check the designs on the merged PR.

flavioislima avatar Oct 20 '22 22:10 flavioislima