fix(files): right click actions menu

[skjnldsv]

After #43090

The root changed

server/apps/files/src/components/FileEntry.vue

Lines 402 to 407 in 1b0eb2b

	const root = this.$root.$el as HTMLElement
	const contentRect = root.getBoundingClientRect()
	// Using Math.min/max to prevent the menu from going out of the AppContent
	// 200 = max width of the menu
	root.style.setProperty('--mouse-pos-x', Math.max(contentRect.left, Math.min(event.clientX, event.clientX - 200)) + 'px')
	root.style.setProperty('--mouse-pos-y', Math.max(contentRect.top, event.clientY - contentRect.top) + 'px')

[skjnldsv]

/backport! to backport/43090/stable28

[ShGKme]

Isn't it the same? Do we need root class here at all?

Suggested change

	.content[style*="mouse-pos-x"] .v-popper__popper {
	[style*="mouse-pos-x"] .v-popper__popper {

[skjnldsv]

Isn't it far less efficient?

[ShGKme]

Isn't it far less efficient?

Probably it is

[emoral435]

Overall good - however, it seems that firefox has had yet another regression (😓😓):

left: chrome | right: firefox

I remember this being mentioned in a PR before, but I cannot remember for the life of me which PR it was! If you believe this can be ignored for now / chrome is where most people use NC, I can go ahead and submit an approval

[susnux]

#44139