mipops/dvrescue

Merge interface requests

dericed opened this issue · 10 comments

  • option to remove a file from the "Files to Merge" table
  • currently it's possible to place the same file in the "files to merge" table multiple times. A file should not occur in the table more than once.
  • place "Merge" before "Package" in left edge navigation bar
  • reorder Merge layout like this (more similar to Package UI)
    • top row: left side is "Input Files" and right side is "Files to Merge"
    • middle row is the settings parts (output directory and "Merge" button)
    • bottom row is the summary table
  • In the merge image pop up. Add a row at the top with Frame # and Timecode from table, and a < and > button to go to previous of next row (similar to the dvloupe popup)

@dericed how about something like this for the rearranging? This seems like it would be the easiest to drag and drop the files to merges and I think it would be the most intuitive flow for input, selecting files, merging and then viewing the results on the other side. Let me know what you think :)

dvmerge-GUI-design_2023-12-02

@dericed what if we did something like this for the icons in the summary table:
Screenshot 2023-12-03 at 10 28 20 AM
Or we could use the file numbers so that people aren't thrown off by 0 = file 1:
dvmerge-icons-2

dvmerge-icons-3

Color coding in the summary table is super helpful! The only suggestion would be to not use red or green since we use those to represent other things for the GUI.

Instead of the icons above, we could use dots to represent the P, M and frames selected. P (problem frames) would be a gray dot, M (missing frames) would be a red dot and frames that are fine would be a green dot. For example if this is the current display:

dvmerge-summary-chart-current

It could be represented like this instead:

dvmerge-summary-chart-design-idea-1

@elder, can you try out the circles ideas that @libbyhopfauf designed. Take the Status value and append spaces so the length of Status equals the number of inputs. Then replace a space with a green circle, an "M" with a gray circle, and "P" with a red circle. Could we make a tooltip that summaries like "For ${timecode}: Input X (missing); Input X+1 (problem), ..."

I think this issue is complete. @libbyhopfauf, more advice on making this layout more intuitive?

I think this is great! The only other thing would be to remove red and green as colors from the summary list/corresponding highlighting in dvloupe (see quoted suggestion below). Based on feedback we've had from several users using these colors is really confusing.

Color coding in the summary table is super helpful! The only suggestion would be to not use red or green since we use those to represent other things for the GUI.

Sure, can you update the color list. It's in two places

property var colors: 'red green blue magenta yellow cyan'.split(' ')

COLOR=(red green blue magenta yellow cyan)

The two lists need to be in sync to correlate the dvplay image or the merged result with the filelist.

You can pick colors from https://ffmpeg.org/ffmpeg-utils.html#Color

See PRs #825 and #826 :)

I was also thinking that to help with the confusion that we could change the highlight color for all of the input file lists (for analysis, merge and packager) to be inverted vs. being green. See example below:

Current

Screenshot 2024-02-14 at 5 38 52 PM

Suggested

invert-selected-row-file

Just another small note about the merge gui -- the space where you can specify the path to package the file into should have a button to the right (probably a ...) to open a file selection window. It's there and it works but it's invisible!

invisible_button