fix(NcUserBubble): indicate that element is clickable
☑️ Resolves
- Show interactive cursor
pointerif an element has click event or a content to show in Popover
RFC:
Isn't &--clickable * too wide? I doubt anyone is planning to insert an input in name slot, but still
🖼️ Screenshots
| 🏚️ Before | 🏡 After |
|---|---|
🏁 Checklist
- [ ] ⛑️ Tests are included or are not applicable
- [x] 📘 Component documentation has been extended, updated or is not applicable
- [ ] 3️⃣ Backport to
nextrequested with a Vue 3 upgrade
here
NcUserBubbleis used as a clickablediv
Without additional context (spreed PR), clickable div itself should be already an issue. Maybe we should extend it to <a>/<button>/<div> at least?
here
NcUserBubbleis used as a clickabledivWithout additional context (spreed PR), clickable div itself should be already an issue. Maybe we should extend it to
<a>/<button>/<div>at least?
Component already supports <a> with href prop. We can also add to prop to support <RouterLink>.
With Popover it is still clickable <div>, but let's call it a separate issue
/backport to next