[FEATURE REQUEST][OPINIONATED] Hiding unnecessary columns in a given context

[Hlsgs]

When opening the program via command line and selecting one or more metrics(e.g. -metric=VMAF), it would be great if the columns for the other metrics were hidden. The intention of the user is clear in this case and so they serve no purpose and take up space for no reason.

And while we're at it, can we also get a -no-playback parameter to hide the playback column. I never use it, but I do click on the play buttons from time to time by mistake.

I realise that these are kind of personal(hence the "opinionated" tag) but if such quality of life improvements are of interest to you, they would be really appreciated.

Thank you, regardless of what you decide!

[fifonik]

In v1.7.0 beta 2 (that I've just published) I added an ability to hide preview grid column using config option "ShowPreviewColumn": false

Unfortunately, at this stage I do not think that hiding metrics' columns is a good idea so not implementing it right now. Will need to think about this.

[Hlsgs]

In v1.7.0 beta 2 (that I've just published) I added an ability to hide preview grid column using config option "ShowPreviewColumn": false

Excellent! Thank you for that!

Unfortunately, at this stage I do not think that hiding metrics' columns is a good idea so not implementing it right now. Will need to think about this.

I've stayed away from the config file until now to facilitate easier updates(just delete and replace the program folder). But, now that I've looked into it and since I'll be using it moving forward, it would make more sense to hide the metrics columns via options like Metrics.PSNR.Hide:True or Metrics.PSNR.Enabled:False. While it's somewhat concievable that a user who started the program with -metric=VMAF might change their mind, I fail to see any downsides to the FFMetrics.conf approach.

Btw, speaking of FFMetrics.conf, it only needs to contain what we want to change, correct? For example, having it as follows would be correct?

{
	"ShowPreviewColumn": false
}

Or, does it need to also contain everything else that's in FFMetrics.conf.example?

[fifonik]

Btw, speaking of FFMetrics.conf, it only needs to contain what we want to change, correct?

Correct.