Fix: misplaced border ncselect

[hamza221]

Fix #9445

b	a

[GretaD]

i thought the issue was that the dropdown not having a top border.

[hamza221]

i thought the issue was that the dropdown not having a top border.

Yes, I realised that later after checking pre-vue8 , that's why I turned it into draft

Pre vue8 there was no border and no slot for no results

[hamza221]

pre vue-8

[hamza221]

I left the No result in

[ChristophWurst]

/backport to stable3.6

[hamza221]

PHP-CS unrelate to this PR, might be cause by this 7af56c9

[ChristophWurst]

#9552

[GVodyanov]

Works!

[GretaD]

when i setup an account for the first time and i have no search results, it is shown like this.
If i have at least one search result, it is shown correctly.

[ChristophWurst]

The borderless design does not look very appealing. Could we fully switch to the default ncselect styling with borders? We have bordered and stacked inputs in other places too:

So I guess it would be fine to also have the borders in the Mail composer. @jancborchardt objections? if so, please provide mockups for a borderless select component

New suggestions from Christoph

[hamza221]

@nextcloud/designers any suggestions?

[marcoambrosini]

I think it makes sense to use our components as they natively look as @ChristophWurst suggests. Border-with will decrease to 1 pixel with the next release so they won't look as prominent.

If with time we realise that it's still too much, I would prefer finding a solution upstream, since these dense forms are ubiquitous in Nextcloud.