@delvh included the rename poping to popping here in the second commit, I ran:

perl -p -i -e 's#poping#popping#g' templates/**/* web_src/js/index.js

If we are already at it: Is there a reason why these two are two separate classes?
Because as far as I can see, they could be combined into one style class pop-up.
Then everyone knows without a doubt what they are meant to do.
Doing this makes everything more readable, smaller, and less error-prone in my opinion.
Unless of course that style class is already used somewhere else…
Even though doing this might help reduce bugs in the end as those things are also already intended to popup and we then have a more unified code base…

I think that class can probably be removed entirely. I'll probably move the renaming commit to another PR as it's not directly related.

I am fine with both.
I prefer removing it entirely if possible because less code means fewer bugs…

Backed out of the renaming change here, will follow up with a separate PR later.

🚀