Video overlay should not nudge people to accept all cookies

Can one of the maintainers please explain, if the "Alle zulassen" / "Accept All" is really what we want to display as button overlay to filtered content?
What do others think? Should we "accept all" cookies with this button or should we only accept the video related cookie settings?

@vistree could you please add a marked screenshot for clarification? Thanks :)

@Anybody - sure: on the attached screenshot you find the standard video filter overlay (cookies-filter-overlay-button.png). The button "Accept all Cookies" will trigger all cookis from all "groups".
So, it is the same behaviour as in the main cookies dialog (cookies-main-dialog.png)
From my point of view it should instead only set the cookie for the "video" group (cookie-video-dialog.png)

Thanks, I think I got your point!

At you suggest showing the smaller text below as larger button and the button as smaller text?

So that you're not being "nudged" to accept all cookies, right?

Regarding the other screenshots I think all is fine, as "Save" is the primary action and all others are equally weighted?

Regarding privacy, I agree we can do that and should of course allow site owners to override this by CSS.

Did I get you right?

If yes, please prepare a MR for that.

To be honest ... I even did not realize, that the smaller text is there and can do what I was missing ;-)
So, yes. You are right ;-) We should make the "Video only" accept link a button and make the "Accept all cookies" less prominent - or we can also remove it??
What do you think?
I can then try to create a MR ;-)

Thanks, I'd think we should switch them. Happy to review your MR! :)

@Anybody: can you have look at the MR? Errors seem not to be related to my changes …

Thanks yes, I'll ask @Grevil to check this, when he reviews the other issues!

Not related to your MR!
Tests are currently failing on the main branch (probably due to a recent hotfix). On it.

Tests are green again on dev! A quick rebase on current 1.2.x should do the trick.

Thanx @Grevil and @Anybody,
my MR now was build successfully.

Alright, I also removed the full stops, since they look quite weird on the big button:

Otherwise, this should be good to go as long as the tests suceed! :)

All green! Let's merge this!

Thank you Grevil ;-)