Improove Video Mod

Collaboration
#1

This video-Mod is a template-version:

https://anymod.com/mod/youtube-banner-modal-lllamb

An improovment-suggestion can be found here:

https://anymod.com/merge/lllamb/oroaab

I am aware there are better ways to code the improovement but I decided to stick with the workaround and merely report my status quo.

PS: I wanted to ensure Tyler will be able to look at the MERGE URL. Thus I invited Tyler to this Team: https://anymod.com/projects/8P795P/team

This is the improoved Mod:
https://anymod.com/mod/youtube-banner-modal-oroaab?cloneComplete=true&preview=true

A: found the issue and its cause
B: solved the issue with a workaround, and
C: tried to report the issue best I can

Question 1: Does Tyler need to be a team-member for him to be able to look into my newly created Merge-Comparision-URL?

Question 2: Is there a better way to report future-issues?

#2

I am grateful for any update.

#3

Hi Stefan,
Is the goal to make it so that you can have multiple videos on the page?

#4

Yes.

I missed to mention what the ‘improovement’ is ment for. It will avoid modal-Layers from identical mods ( on the same page ) to be mistakenly adressed. The unique naming convention will prevent any potential interference. I ran into such an issue last week.

#5

We’ve merged a change similar to the one you proposed, using mod.selector instead of the video ID.

https://anymod.com/mod/youtube-banner-modal-lllamb?preview=true

Thank you for letting us know and for proposing a solution!

1 Like