fix right-click img tags

[szaimen]

Follow-up to #35484

Fix #36832
Fix #36154

Before	After

[szaimen]

/compile amend /

[rotdrop]

Are you sure that this fixes #36154? We have still the width and height of the image tag set to 16px and its padding set to 14px. And box-sizing border-box. But this means that regardless of your change the following lines:

server/core/css/apps.scss

Lines 1144 to 1147 in 86c102e

	> img {
	width: $popovericon-size;
	padding: #{math.div($popoveritem-height - $popovericon-size, 2)};
	}

will still result in total size of 28px ("border-box" means that content + padding make up for the size, doesn't it?)

I have only tested your changes with NC25 and it does not seem to change anything. BTW, also the "Edit locally" menu item is affected and does not show the icon.

Is maybe the ":not()" wrong? At least #36154 talks about the very filesActionMenu and the :not() excludes it. So your CSS rules changes do not affect the filesActionMenu. If I remove the not(...) then the issues with the files-action menu described in #36154 seem to be fixed for NC 25.

[szaimen]

/compile amend /

[szaimen]

/backport to stable25

[rotdrop]

This seems to fix the issue. However, I suppose the lines

server/core/css/apps.scss

Lines 1144 to 1147 in 86c102e

	> img {
	width: $popovericon-size;
	padding: #{math.div($popoveritem-height - $popovericon-size, 2)};
	}

should be revisited as my impression is that these settings do not make sense with box-sizing border-box. Just mentioned it here because that rule makes the reviewed changes "necessary" and maybe are bogus. But this is probably something for a separate new issue.