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

Stats, Achievement menu options link to Profile modal; Edit Avatar option is sometimes wrong #11337

Closed
Alys opened this issue Sep 1, 2019 · 5 comments · Fixed by #11412
Closed

Comments

@Alys
Copy link
Contributor

Alys commented Sep 1, 2019

Clicking on User Icon > Stats or User Icon > Achievement opens the Profile modal. You then have to click on the Stats or Achievement subheading to get to the screen you want.
The correct behaviour is that those two menu options should link directly to the Stats or Achievement modal.

The User Icon > Edit Avatar menu option works correctly most of the time (i.e., it takes you to the avatar editing modal with the Body section open). However if you use the User Icon > Backgrounds menu option and then any time later you go to User Icon > Edit Avatar, you're taken to the Backgrounds screen again (I'm pretty sure this is new, incorrect behaviour). You have to reload to make User Icon > Edit Avatar direct you to the Body screen automatically. <- This has been fixed

@Scens
Copy link
Contributor

Scens commented Sep 1, 2019

Hi!
I've just checked and Stat and Achievement menu options work good in the most recent develop version.

@Alys
Copy link
Contributor Author

Alys commented Sep 2, 2019

@Scens Thanks for finding that and for the PR! I've marked this as in progress for you.

@paglias
Copy link
Contributor

paglias commented Sep 13, 2019

I've edited the description to say that a part of the issue has been resolved

@ResamVi
Copy link
Contributor

ResamVi commented Oct 8, 2019

I have tinkered with it to see if I could pin down the error and I seem to have fixed it for me. Is help still wanted on this issue?

@Alys
Copy link
Contributor Author

Alys commented Oct 8, 2019

@ResamVi Sure, thanks! I've marked it as in progress for you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants