WzComparerR2 icon indicating copy to clipboard operation
WzComparerR2 copied to clipboard

Comparer: Show info for all files include merged, sub wz (Fix #166)

Open KENNYSOFT opened this issue 4 years ago • 5 comments

#166

image

We may provide some preference to collapse this long list.

KENNYSOFT avatar Sep 20 '21 07:09 KENNYSOFT

Do we really need to show them all?

And we can use <details> and <summary> to collapse file tree.

Kagamia avatar Oct 04 '21 04:10 Kagamia

@PirateIzzy Any suggestion? 🤔

KENNYSOFT avatar Oct 04 '21 12:10 KENNYSOFT

@KENNYSOFT

I'm thinking of a few options:

  1. Combine all of them into a single output, so the total size of the old Character.wz -> the total size of the new Character.wz
  2. Add a summary on the top (like the current version), and a full comparison on the bottom of the comparison report. 2.1. Maybe with this, we could change the layout of the comparison to show just the WZ files, rather than the full directory, and changing it to be a side-by-side WZ comparison, like so: Mockup

PirateIzzy avatar Oct 05 '21 00:10 PirateIzzy

My purpose is always to keep the sub files and folders transparent to wzlib users, so I prefer option 1.

For those who really interested in sub files, I think it's better to provide a expandable list in report file, but it is a little more difficult to collapse file list in a html table.

We still need to provide full path to wz file, because most of us have more than one version of Maplestory Client.

  file size ver
new + D:\kmst\data\character\character.wz 1,000,000,000 (total) 1129
  |--- character.wz 63
  |--- character_000.wz xx
  |--- accessory\accessory.wz 63
  |--- accessory\accessory_000.wz 63
old + D:\kms\character.wz 1,100,000,000 (total) 351
date 2021-10-05 xx:xx
params -xxx

Kagamia avatar Oct 05 '21 08:10 Kagamia

Alright, understood. And that can work.

PirateIzzy avatar Oct 05 '21 17:10 PirateIzzy