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

fix issue#2511 #2514

Merged
merged 3 commits into from
Oct 21, 2023
Merged

Conversation

Petrificus-totalus
Copy link
Contributor

This fixes issue #2511
My wallet address is 0x4c35271cA4fF57adEc0E28cAc6AF5D51E37bF378

@vercel
Copy link

vercel bot commented Oct 19, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
ancientbeast ✅ Ready (Inspect) Visit Preview Oct 21, 2023 0:19am

@ghost
Copy link

ghost commented Oct 19, 2023

👇 Click on the image for a new way to code review

Review these changes using an interactive CodeSee Map

Legend

CodeSee Map legend

@DreadKnight
Copy link
Member

Screenshot_20231019_160707
@Petrificus-totalus I've still managed to get the logo to show up awkwardly by switching tabs via hotkeys. Needs more work.

@DreadKnight DreadKnight marked this pull request as draft October 19, 2023 17:45
@Petrificus-totalus
Copy link
Contributor Author

I modified the dev environment which runs at 'localhost:8080', so do I need to do the npm run build:dev and then do 'git push', thanks for your patience. :)

@Petrificus-totalus
Copy link
Contributor Author

I modified the dev environment which runs at 'localhost:8080', so do I need to do the npm run build:dev and then do 'git push', thanks for your patience. :)

I did the build-and-push, and it seems to be working on the website you are testing. @DreadKnight

@DreadKnight
Copy link
Member

DreadKnight commented Oct 19, 2023 via email

@Petrificus-totalus
Copy link
Contributor Author

I believe it might be an issue with browser and operating system compatibility. May I ask which hotkey you are using? Apologies for asking so many questions.

@DreadKnight
Copy link
Member

DreadKnight commented Oct 19, 2023 via email

@Petrificus-totalus
Copy link
Contributor Author

oh, got it. I have been testing my laptop using shift+tab. Thanks for your reply.

@Petrificus-totalus
Copy link
Contributor Author

@DreadKnight Hi, I think I solved it. I've chosen to listen for when the game webpage becomes hidden, which happens when switching between web pages. If it disappears, the logo will be hidden.

@DreadKnight DreadKnight marked this pull request as ready for review October 21, 2023 12:17
@DreadKnight DreadKnight merged commit f915f96 into FreezingMoon:master Oct 21, 2023
2 checks passed
@DreadKnight
Copy link
Member

@Petrificus-totalus Heya! Seems to be working properly now. You forgot to mark this ready for review :P Bounty sent 🪙

@Petrificus-totalus
Copy link
Contributor Author

Thanks a lot. Is there anything else to do for this issue, please let me know.

@DreadKnight
Copy link
Member

@Petrificus-totalus Heya! Found another scenario related to this kind of stuff, I've opened new issue for it -> #2521

@Petrificus-totalus
Copy link
Contributor Author

@DreadKnight Sorry, sir. I have to prepare my final exams these days.

@DreadKnight
Copy link
Member

@Petrificus-totalus No worries, good luck with those! 🐻

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