Inconsistent size of 2FA-type icons

[MariusBluem]

Do not know wheter this is an issue with this app, the server or the notifications-2FA. 🤔 @ChristophWurst

[ChristophWurst]

Good point. Is this a specific browser or does it happen with all?

[ChristophWurst]

cc @nickvergessen @rullzer for the notifications app

[ChristophWurst]

on my dev instance, without the notifications provider

[nickvergessen]

Which size are you using? Is it documented?

[nickvergessen]

Should add the recommended size to this block:
https://github.com/nextcloud/server/blob/45cf2eef56549f4c360609f82aed8b38e7bb0335/lib/public/Authentication/TwoFactorAuth/IProvidesIcons.php#L37-L53

Or try to fix it with CSS

[MariusBluem]

Tested with Safari and Microsoft Edge but I think it is happening for all.

[rullzer]

I saw this as well on my private instance. We do not provide an icon I think for the notifications app. So I think the default is just used...

[rullzer]

Mmm I can't trigger it anymore.

[ChristophWurst]

Should add the recommended size to this block

I think we should have a required minimum + some css that limit the max size

[ChristophWurst]

Mmm I can't trigger it anymore.

Did we fix it for 16 maybe?

[rullzer]

Did we fix it for 16 maybe?

Also can't trigger it on 15 ;)

[ChristophWurst]

@MariusBluem is that a specific setup you were testing on?

[MariusBluem]

It was our company cloud :)