JoeyDeVries / LearnOpenGL

Code repository of all OpenGL chapters from the book and its accompanying website https://learnopengl.com
https://learnopengl.com
Other
10.82k stars 2.77k forks source link

Ad is in an annoying place #375

Closed ArcE404 closed 9 months ago

ArcE404 commented 10 months ago

image This Ad appears on the introduction page and makes it difficult to navigate through the page. image

JoeyDeVries commented 10 months ago

Shit, this is definitely not the right behavior, sorry! Second time I've been hearing of this and thought it was fixed.

Do you have some extra info for me to see if I can try and reproduce?

Thanks!

ArcE404 commented 10 months ago

Sure thing: I'm using chrome on a Windows 10 computer (desktop), it appears in a full-size window on a 1920×1080px screen I'm from Costa Rica, I'm doing a Spanish translation, that's why I also know that the same behaviour is on the translations page 😅 hope it helps, please let me know if you need something else :) image

JoeyDeVries commented 10 months ago

This is perfect, thank you; I'm going to look into it. Will keep you updated!

JoeyDeVries commented 10 months ago

Hey, I can't seem to be able to figure out what's happening. If you're familiar with browser dev tools, would you be able to tell me the following?

I'll keep debugging on my end to see if I can reproduce/figure-it-out. Sorry for the inconvenience!

ArcE404 commented 10 months ago

Is not an inconvenience, I'm happy to help :D

here is the HTML component of the ad:

<div id="waldo-tag-15905" class="waldo-display-unit up-show" style="min-height: 600px;" data-google-query-id="CJ24tNW81YIDFaaj0QQd8JsCjA"></div>

The ad is 160x600px.

Pasted image 20231121110852

The bug constantly appears in the learnopengl.com path, the learnopengl.com/Introduction path and the translations' path. I don't see it in other places, only these.

If it helps, as you can see, the add, and its iframe component, is inside the div component with the id="nav", the same place where the list of options is located Pasted image 20231121110748

Tell me if you need something else ;)

JoeyDeVries commented 10 months ago

This is super helpful, thank you! It looks like an ad is overruling my default-hide behavior (unless content is large enough) which is why they're visible without positioning logic happening yet. I'm currently discussing this with my ad agency. I've pushed a hotfix for now which hopefully makes it a little less annoying, but not sure if it'll be enough. Hopefully you can at least use the menu (:

I'll keep you posted

ArcE404 commented 10 months ago

Thank you, for taking care and finding a solution! :)

JoeyDeVries commented 9 months ago

Okay, been some back and forth; and had to rewrite some backend logic to make it all work. Turns out some javascript magic of hiding ads when content size isn't big enough can mess with ads sometimes. Should be visible at all times so I reworked this logic to the server. I expect things'll be a lot better and it should be fixed. Let me know if it still happens!

ArcE404 commented 9 months ago

Everything works correctly now! And yes, JS can be a mess sometimes, tell me about it. Thank you for resolving this issue, and thank you for all the information that is on the page, I'm really love all this stuff about graphics programming, shaders and stuff. I think I found a new passion within my passion (which is programming)

Thanks, Augusto

JoeyDeVries commented 9 months ago

Of course, thanks for letting me know! And wonderful to hear ❤️ there's truly something special about creating your own graphics/stuff from nothing; enjoy the process! :D