fix(SetContacts ): Remove unnecessary scroll

[DorraJaouad]

☑️ Resolves

• Fix ContactSelectionBubble causes unnecessary scroll #10755

Close Button has height 44px in ContactSelectionBubble whereas selected contacts div goes lower ( 40.8px), causing the extra scroll.

🖌️ UI Checklist

🖼️ Screenshots / Screencasts

🏚️ Before	🏡 After

🏁 Checklist

• 🌏 Tested with Chrome, Firefox and Safari or should not be risky to browser differences
• 🖌️ Design was reviewed, approved or inspired by the design team
• ⛑️ Tests are included or not possible
• 📗 User documentation in https://github.com/nextcloud/documentation/tree/master/user_manual/talk has been updated or is not required

[DorraJaouad]

/backport to stable28

[Antreesy]

Still issue with 2 lines and more:

I remember the issue with 44px, yes =) How about we get rid of NcButton, and put a click event on a whole component?

[DorraJaouad]

Still issue with 2 lines and more:

I thought it is expected with more then one line

How about we get rid of NcButton, and put a click event on a whole component?
Hmm, wouldn't that be confusing for the user ?

[Antreesy]

I thought it is expected with more then one line

max-height is 90+px, so 2 lines should be shown without scroll

Hmm, wouldn't that be confusing for the user ?

I thought we have the same behavior for NcSelect chips as an example, but they have it separately... maybe force a height of NcButton to 22px?

[Antreesy]

Tested

[Antreesy]

/backport to stable27

[Antreesy]

/backport to stable26