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

Removed loading screen tips, fixes #11834 #12063

Merged
merged 3 commits into from Apr 14, 2020

Conversation

anoopravella
Copy link
Contributor

@anoopravella anoopravella commented Apr 6, 2020

Fixes #11834

Removed the div that displayed tips on the loading screen from the app.vue file. Also removed the code that selected the random tip to be displayed, also in app.vue.

When I ran the test suite locally, 11 tests failed, but the same 11 tests also failed when I ran them on the develop (untouched) branch locally.

Could not screenshot the change (as per the development guide) because the loading screen finished too quickly.

@paglias
Copy link
Contributor

paglias commented Apr 7, 2020

thanks @anoopravella ! could you also remove the website/common/locales/en/loadingScreenTips.json file as it contains the loading tips that we don't need anymore so that it's not loaded?

@Alys do we want to add a note about the file being removed so that we'll remember it's in the git history if we want to add these back?
@SabreCat quick question about Weblate, when removing a .json file is it okay to remove it only for English? or should it be removed for every language? Not sure how it's handled

@Alys
Copy link
Contributor

Alys commented Apr 7, 2020

do we want to add a note about the file being removed so that we'll remember it's in the git history if we want to add these back?

Good idea! The wiki's Loading Page Tips page already had some information about the code so I've updated that and added a temporary note, linking to the PR (it's all in the Information for Developers section at the bottom - click a button there to reveal it). Once this PR is merged and live, I'll make further changes to the page, and also link to the final commit in develop. I'm labelling this as "notify helpers" as a reminder to me. I'll also want to tell the Socialites so they can answer questions about why the tips were removed.

@paglias
Copy link
Contributor

paglias commented Apr 14, 2020

thanks @anoopravella ! This will go live by the end of the week, if you have an Habitica UUID i'll be happy to award you a contribution tier

@anoopravella anoopravella deleted the remove-load-tips branch April 17, 2020 16:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

loading tips can't be read when loading is too quick
3 participants