fix(MessageButtonsBar): improve rendering performance

[Antreesy]

☑️ Resolves

• Always render MessageButtonsBar and Reactions EmojiPicker, get rid of event listeners, rely on CSS instead
• Replace NcActions with NcButton for less rendering/mounting overload
• Get rid of event listeners for code blocks, when there is no code blocks

🖌️ UI Checklist

🖼️ Screenshots / Screencasts

No visual changes, everything is in performance graph

🚧 Tasks

• Test in real client

🏁 Checklist

• 🌏 Tested with different browsers / clients:
  • Chromium (Chrome / Edge / Opera / Brave)
  • Firefox
  • Safari
  • Talk Desktop
  • Integrations with Files sidebar and other apps
  • Not risky to browser differences / client
• 🖌️ 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

[ShGKme]

Looks good, but want to test it tomorrow

[DorraJaouad]

Changes are fine by me X( testing now

[ShGKme]

tomorrow

s/tomorrow/monday

[DorraJaouad]

now

It was indeed now but the approval is not instant for some reasons (conflicts) :3

[Antreesy]

General thoughts per commit:

• 1st should help to speed up a render (so it's just 3 buttons and no popover logic initialised until user clicks), I would definitely keep that ✅
• 2 might drop in favor of upstream PR ❌
• 3 render EmojiPicker for each message with reactions although there are some optimisations done upstream, I'd put in under the table for now ❌
• 4 might add slight overhead (and would conflict with split chat view work), so I'd extract it to different PR for more precise approach ➡️ 🚧

What are your thoughts, @ShGKme @DorraJaouad ?

[ShGKme]

What are your thoughts

👍

[DorraJaouad]

Fine by me

[ShGKme]

Backport?

[Antreesy]

/backport to stable32