feature: add open attachment without download

[Nevelito]

Description

• Add file_field option - "openable"
• Change component to display field
• Add spec

Fixes # (issue)

#3441

Checklist:

• I have performed a self-review of my own code
• I have commented my code, particularly in hard-to-understand areas
• I have made corresponding changes to the documentation
• I have added tests that prove my fix is effective or that my feature works

Screenshots & recording

without hover:

with hover:

video (PS: its random pdf file from university):
Screencast from 21.02.2025 14:10:29.webm

[codeclimate]

Code Climate has analyzed commit f77cfe3 and detected 0 issues on this pull request.

View more on Code Climate.

[Paul-Bob]

Thanks for working on this one, @Nevelito.

The openable option name is a bit difficult to spell. Let's rename it.

I have a few suggestions: external_preview, preview_on_click, or click_to_open. Something along these lines. Let's discuss a better option name.

Also, let's ensure that the link doesn't trigger any downloads. I've tested it, and while PDFs don't trigger a download, CSV files do.

---

PS

Hi @Nevelito,

After discussing this with Adrian, I’m editing my review as we noticed some statements in the issue that could be misinterpreted.

The goal is not to provide an option to enable or disable this feature. Instead, let's remove the configuration setting and make it always clickable and "openable."

Additionally, we need to ensure proper authorization is enforced—if a user cannot download the file, they should not be able to open it either.

[Nevelito]

I think it is done and work like it should

[Paul-Bob]

I tested this, and it works fine with PDFs. However, when a new tab opens for CSV files, it simply triggers a download.

Let's ensure we render the link only for previewable file types. I believe representable? handles this check

[Nevelito]

Hi @Paul-Bob do you have any idea why it can not find css in system spec? Locally everything passes clean.

[Paul-Bob]

It's important to check can_download_file? to enforce authorization properly. Otherwise, users without download permissions could still open and inspect the PDF.

[Paul-Bob]

Hi @Paul-Bob do you have any idea why it can not find css in system spec? Locally everything passes clean.

Sometimes, tests may fail in specific environments. You can check each environment using the following command:

RAILS_VERSION=8.0 bundle exec appraisal rails-8.0-ruby-3.3.0 bundle exec rspec --color spec/system/avo/open_field_attachment_spec.rb

Replace 8.0 and 3.3.0 with the Rails and Ruby versions you want to test locally.

Let me know if this is the case or if the tests are still passing locally, even after replicating the failing environment from GitHub Actions.

[Paul-Bob]

@Nevelito I made a small tweak to how the link is built in the .erb file. Could you take a look and adjust the test accordingly?

[github-actions]

This PR has been marked as stale because there was no activity for the past 15 days.

[Nevelito]

I’ll try to finish my tasks in a few days—I’ve got an important new project to work on. Sorry!