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

impr(profile): add level percentage when hovering over progress bar (vjgtigers) #5276

Merged
merged 5 commits into from Apr 5, 2024

Conversation

vjgtigers
Copy link
Contributor

Description

Added code so when the users mouse hovers over the progress bar the percent complete of that level is displayed

Also:
added "data-balloon-pos="up"" so that the total user xp on profile view will new appear

Closes #

pull [#5269]
messed up the branch when changing and couldn't fix it

on account and profile view page:

Screenshot 2024-04-02 220658

on profile view page:

Screenshot 2024-04-02 220744

@monkeytypegeorge monkeytypegeorge added the frontend User interface or web stuff label Apr 3, 2024
@vjgtigers vjgtigers changed the title impr : added percentage till next level when hovering over progress bar (vjgtigers) impr: added percentage till next level when hovering over progress bar (vjgtigers) Apr 3, 2024
@monkeytypegeorge
Copy link
Collaborator

Continuous integration check(s) failed. Please review the failing check's logs and make the necessary changes. https://github.com/monkeytypegame/monkeytype/actions/runs/8532204264

@Miodec
Copy link
Member

Miodec commented Apr 3, 2024

From the screenshots it looks like the vertical alignment is off. Please try to fix it.

@Miodec Miodec added the waiting for update Pull requests or issues that require changes/comments before continuing label Apr 3, 2024
@vjgtigers vjgtigers closed this Apr 3, 2024
@vjgtigers vjgtigers reopened this Apr 3, 2024
@vjgtigers
Copy link
Contributor Author

vjgtigers commented Apr 3, 2024

fixed xp bar alignment

Updated account and profile view page:

updatedBar

Updated profile view page:

updatedTotal

@Miodec Miodec changed the title impr: added percentage till next level when hovering over progress bar (vjgtigers) impr(profile): add level percentage when hovering over progress bar (vjgtigers) Apr 3, 2024
@Miodec
Copy link
Member

Miodec commented Apr 3, 2024

I broke the PR somehow.... Can you hard reset to the "update styling a tiny bit" commit, then force push?

@monkeytypegeorge
Copy link
Collaborator

Continuous integration check(s) failed. Please review the failing check's logs and make the necessary changes. https://github.com/monkeytypegame/monkeytype/actions/runs/8548861329

@vjgtigers
Copy link
Contributor Author

Should be working again

@Miodec
Copy link
Member

Miodec commented Apr 5, 2024

Thanks

@Miodec Miodec merged commit e37bf19 into monkeytypegame:master Apr 5, 2024
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
frontend User interface or web stuff waiting for update Pull requests or issues that require changes/comments before continuing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants