Zoom pop working or not?

[R_Charger]

Since patch 3682 my zoom pop key is not working (which I like to use). Is anyone using this key and experienced anything similar?

[nine2]

They cleaned up the zoom pop feature.

This is a smart mouth joke. They tried to clean the code and broke the feature. I reported it and they left it broken. Not sure where it is at.

[R_Charger]

Hmmm 2 steps forward one step back. Can it please be fixed in a new patch? Also would be an idea to fix it so it works properly like in UI party!

[KeyBlue]

What is zoom pop feature anyway?
This is the first time I hear about it...

[R_Charger]

You press one key and it zooms in/zooms out to instantly. No rolling the mouse wheel. The version in Forged Alliance has always been a bit rubbish as it didn't exactly zoom in/out where the cursor was. This was fixed in UI Party and worked perfectly.Its not working at all has happened since after 3680 (it was working with the refactored UI party for 3680). Really the fix in UI party should be integrated as it corrects a fault rather than just changes the UI.

[KeyBlue]

alright, I fixed it. It was just some missing "/".
https://github.com/FAForever/fa/pull/2006

@Anihilnine, I couldn't find an issue about it on github.
And you just mentioned it on slack while complaining about having to move the modules files. (not even in bugreports channel)
Nobody seemed to respond to it and then it just got drowned out by other stuff.
That can hardly be called proper bug reporting.

It's not your duty to make good reports, so thats not an issue.
Just don't claim you reported it and make it look like the devs completely ignored you when you didn't put in much effort in letting them know either.

[R_Charger]

Ok thanks for finding that KeyBlue. I still think the fix in UI party (zoompopper.lua) should be implemented in FAF. No reason why it shouldn't be fixed for everyone.

[nine2]

Zoom pop was originally introduced in hotbuild, its not part of FA. My "fix" is the most unpleasant hack in the world.

[KeyBlue]

Well I'll look at what you got. And see if we can steal some of your ideas.
But if its actually a big hack, it probably will stay as it is for now.

I assume you don't mind if we did use some of your code?

[nine2]

Hi R_Charger, you can use any code from UI Party that you want - but I think it's a bad idea. I made it whilst learning fa + lua and I did some pretty noob things.

Hotbuild's zoom pop is meant to be a button where you can zoom in and out really quickly. Unfortunately it zooms in to an inexact place, which is really jarring because after zooming in, you don't know where you are, which makes it unusable.

Steps to illustrate problem:
- zoom right out on a big map.
- start placing a factory near a mex near a map corner, as in show the green building placeholder next to a mex
- zoom pop in
- at this point you would expect to be zoomed in with the factory placeholder near the mex still, but it's not.
- now install UI Party and repeat the experiment for a much better experience

UI Party fixes this by doing some very dodgy things. In the startup sequence it actually quickly zooms the camera in and out about the map to work out the pixel to worldspace ratio that results in better zoom popping. It comes up with a ratio that is much more accurate (but still not totally accurate). The hack also slightly changes the startup angle of the camera.

I would suggest that such a hack has no place in production code. Perhaps a real fix is possible but I imagine you guys have more important things to invest in. I personally personally don't even use the "fixed" version of zoom pop because it is still annoying.

What you could do though is get zoom pop working at all -> I believe the keybinding action is broken somehow. Like something got renamed in code but the action that invokes it did not. Then people would have the option of using hotbuild or UI Party zoompop. Edit: oh ok i see you have done that already, sweet thanks

I can make a video to explain it all if you really want but I don't think this is worth your effort.