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

[UI] Improve download speed chart #3244

Merged

Conversation

RawToast225
Copy link
Contributor

@RawToast225 RawToast225 commented Nov 19, 2023

Smoothed out the download status bar and fixed the width of the real time download status


Use the following Checklist if you have changed something on the Backend or Frontend:

  • Tested the feature and it's working on a current and clean install.
  • Tested the main App features and they are still working on a current and clean install. (Login, Install, Play, Uninstall, Move games, etc.)
  • Created / Updated Tests (If necessary)
  • Created / Updated documentation (If necessary)

…ent the graph in downloads to be a higher number, changed the numbers to not jump around so much on the same row
@RawToast225
Copy link
Contributor Author

#3242 My attempt to address this issue

@RawToast225 RawToast225 marked this pull request as draft November 19, 2023 22:48
@RawToast225 RawToast225 marked this pull request as ready for review November 19, 2023 22:49
Copy link
Member

@flavioislima flavioislima left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested here and works pretty good.
Thanks for that. 🗡️ 🛡️

@flavioislima flavioislima changed the title Fix/jumping download UI [UI] Improve download speed chart Nov 22, 2023
@flavioislima flavioislima merged commit 630e523 into Heroic-Games-Launcher:main Nov 22, 2023
13 checks passed
@RawToast225 RawToast225 deleted the fix/jumping_download_ui branch January 15, 2024 03:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants