Fix: RichEditor spacing when no buttons

[johncarter-]

When there are no buttons the toolbar ->disableAllToolbarButtons() - the bar still shows, causing an extra line.

Before

After

This conditionally hides the bar is no buttons.

Please let me know if you prefer a different syntax for the conditional class.

[danharrin]

Does this also fix the toolbar overflow issues here? #2514

I assume it does, with the overflow-x-auto?

[johncarter-]

Does this also fix the toolbar overflow issues here? #2514

I assume it does, with the overflow-x-auto?

Something seems off here, looks like I have merged the wrong classes 🤦‍♂️

I'm not sure about the related issue.

[danharrin]

Can't we just wrap the toolbar in an @if (count($getToolbarButtons())) instead of using the class?

[johncarter-]

Much better solution 🤦‍♂️

[johncarter-]

Removing the bar with the if directive results in a console error and the form failing to save:

I assumed it was a diffing issue so wrapped the if in a div but no joy. Shall I revert to CSS hidden?

[danharrin]

Yeah thats fine, lets use hidden then. Trix must not like it

[danharrin]

Btw, this entire issue is Chrome-only 😅 Safari wins for once

[danharrin]

Also fixed #2514

Thanks John!

[johncarter-]

Cheers Dan, sorry to leave you hanging there - I'm full-time Dadding at the mo.

[danharrin]

No worries John! :)