Improve trix and action text support

[Paul-Bob]

Feature

Trix field have some limitations:

1. File attachment is possible only on persistent records Avo demo error: "You can't upload files into the Trix editor until you save the resource" #2713
2. File attachment is possible only whit explicit attachment_key
3. File attachments are corrupt except images on action text. (has_rich_text) After uploading File in ActionText (Trix) it can not be found when rendering (missing attachable) #2538

Let's explore this possibility:

1. When attachment_key is set that means we need a persistent record with that attachment_key association. So limitation points 1 and 2 still applying.
2. When attachment_key is not set that means the user do not want to associate the attachments to a fixed key/association. We can generate a random key like here https://trix-editor.org/js/attachments.js and in this case the limitation points 1 and 2 will not apply, i.e there is no need for a record and neither attachment_key.

[davekruse]

Editing an ActionText field with Trix field separates captions from images.

1. Add an image
2. Click and change the caption
3. Save
4. Edit

In the trix edit view, the caption is rendered as regular text (a couple
line breaks and plain text). Saving the field causes that caption to become part of the body. Rendering the ActionText elsewhere also shows the text as part of the body - no longer attached to the image.

[sedubois]

I think a helpful first step would be to ensure models using has_rich_text are added to the test suite and to the demo website. Currently the use of has_rich_text does not seem to be tested/demonstrated/documented anywhere. See issue.

I can also confirm that captions are broken as noted by @davekruse.

[sedubois]

Should a separate issue be opened to fix the broken captions?

[adrianthedev]

Yes, please open a new issue, but first, can you please test if a new app with Trix and AS works well and the captions aren't broken?

[sedubois]

@adrianthedev your new demo app shows the issue:

• captions work with the Page resource which uses a plain-text column
• captions break with the Post resource which uses has_rich_text Action Text

[adrianthedev]

I wrote on the previous PR that that will require us to digg in deeper for such an edge-case. Would you be able to sponsor that fix?

[sedubois]

@adrianthedev could you please clarify what you mean by edge case? Maybe there is a misunderstanding. Using has_rich_text is to my knowledge the official way to use Action Text (and, by extension, Trix).

What I consider to be an edge (a.k.a unofficial) case is to use the Trix editor on top of a plain text DB column without using has_rich_text, as is currently done throughout the Avo documentation and demo code.

Currently I am trying to add Avo to a production system but in my free time, because it is just being evaluated; so in turn I am not able to sponsor a fix although I do try to support the project by documenting the issues found with reproduction steps.

I hope it's clearer this way. Although I can't sponsor I am happy to support the project in a more modest way and wish you much success.