Merge Preview Dialog in Torrent Content View

[jagannatharjun]

I personally think that we don't need preview dialog anymore[1] and it should be removed
My idea is to provide a context menu entry "Preview" in Torrent Content view and maybe a checkbox like "only show to previewable file" to filter out files
something like this but maybe without the checkbox, ofc only show "Preview" for previewable files

[1]

tagging @Chocobo1 @glassez

[glassez]

👍 I like this idea.

[jagannatharjun]

@glassez How about the checkbox "Only show Previewable file", this is to filter out "Previewable file", do we need it? I don't like its placement and can't find any other place to place it.

[glassez]

do we need it?

I would say No. But I never used this feature. What "preview" does really mean there? Could we replace it with generic "open" action?

[jagannatharjun]

I would say No. But I never used this feature. What "preview" does really mean there? Could we replace it with generic "open" action?

a quick look seems it actually just open, so maybe we can consider fully removing this.

[FranciscoPombal]

I like the checkbox idea, or at least some sort of visual indicator near each file to show whether or not it is previewable.

Double-click action on files in the Content tab could then be:

• Previewable -> open file
• Not previewable -> open containing directory

The former could also probably be customizable via a setting, e.g. Always open containing folder on double-click, even for previewable files.