Skip to content

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

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

Closed
flavioislima opened this issue Mar 23, 2022 · 8 comments · Fixed by #1815 or #1891
Closed

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

flavioislima opened this issue Mar 23, 2022 · 8 comments · Fixed by #1815 or #1891
Assignees
Labels

Comments

@flavioislima
Copy link
Member

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 flavioislima added enhancement This feature needs upgrades. good first issue Good for newcomers. labels Mar 23, 2022
@flavioislima flavioislima changed the title Replace Native Dialogs with HTML Dialogs [General UI/UX] Replace Native Dialogs with HTML Dialogs Mar 23, 2022
@flavioislima flavioislima pinned this issue Mar 23, 2022
@arielj
Copy link
Collaborator

arielj commented Apr 18, 2022

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.

@flavioislima flavioislima removed the good first issue Good for newcomers. label Apr 22, 2022
@flavioislima
Copy link
Member Author

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.

@Nocccer
Copy link
Collaborator

Nocccer commented Sep 12, 2022

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.

This was linked to pull requests Sep 14, 2022
This was unlinked from pull requests Sep 14, 2022
@Nocccer Nocccer linked a pull request Sep 14, 2022 that will close this issue
4 tasks
@flavioislima flavioislima removed a link to a pull request Sep 24, 2022
4 tasks
@flavioislima
Copy link
Member Author

flavioislima commented Sep 24, 2022

@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
Copy link
Member Author

@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.

@Nocccer Nocccer linked a pull request Sep 30, 2022 that will close this issue
4 tasks
@BrettCleary BrettCleary self-assigned this Oct 10, 2022
@BrettCleary BrettCleary linked a pull request Oct 11, 2022 that will close this issue
4 tasks
@BrettCleary
Copy link
Collaborator

Addressed by #1891

@biliesilva
Copy link

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

@flavioislima
Copy link
Member Author

flavioislima commented Oct 20, 2022

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.

@arielj arielj unpinned this issue Oct 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
5 participants