fix(NcSelect): Ensure component height is same as input element

[susnux]

☑️ Resolves

Currently the element was 36px instead of 34px which looks weird if using multiple input elements side by side (caused by search input being 34px + 2px border).
Also there was jumping because there are two different border width, so we need to adjust the input height based on the outer border width and the margin of the selected options.

🏁 Checklist

• ⛑️ Tests are included or are not applicable
• 📘 Component documentation has been extended, updated or is not applicable
• 3️⃣ Backport to next requested with a Vue 3 upgrade

[GVodyanov]

Is it supposed to be 44px though? Because that is what I'm seeing. Why that number?

[susnux]

Is it supposed to be 44px though? Because that is what I'm seeing. Why that number?

Where do you see that number? It should be always --default-clickable-area, so:
On Nextcloud 28 and 29 it should be 44px and on Nextcloud 30 it should be 34px.

[GVodyanov]

Is it supposed to be 44px though? Because that is what I'm seeing. Why that number?

Where do you see that number? It should be always --default-clickable-area, so: On Nextcloud 28 and 29 it should be 44px and on Nextcloud 30 it should be 34px.

Ok yeah my bad, tested it on 30 and looks good. But we will have to update some styling in calendar @ChristophWurst as it breaks

[susnux]

/backport to next

[ShGKme]

Seem now to become an issue on 28/29