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

Empty black box/alert shown #141

Open
ShrimpCryptid opened this issue Apr 17, 2024 · 2 comments
Open

Empty black box/alert shown #141

ShrimpCryptid opened this issue Apr 17, 2024 · 2 comments
Labels
bug Something isn't working frontend Pull requests that update frontend Javascript/Typescript code
Milestone

Comments

@ShrimpCryptid
Copy link
Owner

Description:

Players have reported being shown an empty black screen after voting or certain other alert interactions. I've also seen it happen on initial load when the role should be shown. There is no way to close the alert unless players refresh the page.

Reproduction:

None yet, see dev build: https://secret-hitler-online-dev0.web.app/

Notes:

May be caused by conflicting animations attempting to use the same AlertBox component?

@ShrimpCryptid ShrimpCryptid added the bug Something isn't working label Apr 17, 2024
@ShrimpCryptid ShrimpCryptid added this to the 2.0 Release milestone Apr 17, 2024
@ShrimpCryptid ShrimpCryptid added the frontend Pull requests that update frontend Javascript/Typescript code label Apr 17, 2024
@ShrimpCryptid ShrimpCryptid moved this from New to Backlog in Secret Hitler Online Apr 17, 2024
@ShrimpCryptid ShrimpCryptid moved this from Backlog to Ready in Secret Hitler Online Apr 18, 2024
@moltasthornblom
Copy link

I noticed this issue as well, both when changing icons in lobby and when ingame. The bug could be considered as gamebreaking due to me being unabled to "End Turn" while the box was visible. I'm inclined to think that this is a firefox issue due to only being able to reproduce the bug in firefox.

I did some debugging of the "showChangeIconAlert" function but everything works as expected. However I noticed that the box disappeared when using mozilla dev tools lol.

.appear-custom-alert{
    transition-timing-function: ease-in-out;
    -webkit-animation: appear-custom-alert 0.5s;
    animation: appear-custom-alert 0.5s;
    visibility: visible;
    transition: visibility 0.5s;
    opacity: 1;
}

.disappear-custom-alert {
    transition-timing-function: ease-in-out;
    -webkit-animation: disappear-custom-alert 1s;
    animation: disappear-custom-alert 1s;
    visibility: hidden;
    transition: visibility 0.5s;
}

My theory is that the animation for some reason does not fire in firefox. I was able to solve it by adding a opacity:0; to the disappear-class. I also added some transitioning.

.disappear-custom-alert {
    transition-timing-function: ease-in-out;
    -webkit-animation: disappear-custom-alert 1s;
    animation: disappear-custom-alert 1s;
    visibility: hidden;
    opacity:0;
    transition: visibility 0.5s, opacity 0.5s;
}

I

However there is still some tweaking to be done to get the timing right with the animations so I did not do a PR, instead i simply provide with some context and information.

Cheers!

@ShrimpCryptid
Copy link
Owner Author

ShrimpCryptid commented May 28, 2024

I noticed this issue as well, both when changing icons in lobby and when ingame. The bug could be considered as gamebreaking due to me being unabled to "End Turn" while the box was visible. I'm inclined to think that this is a firefox issue due to only being able to reproduce the bug in firefox.

I did some debugging of the "showChangeIconAlert" function but everything works as expected. However I noticed that the box disappeared when using mozilla dev tools lol.

...

Thanks for the repro, I'll take a look! I originally opened this issue because it was happening on the development branch after some changes I'd made, so it's interesting that there's a similar issue happening on the main production branch too. Hopefully it'll solve both!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working frontend Pull requests that update frontend Javascript/Typescript code
Projects
Status: Ready
Development

No branches or pull requests

2 participants