Revamped replace/overwrite dialog

Ideas for improvements and requests for new features in XnView Classic

Moderators: XnTriq, helmut, xnview

User avatar
XnTriq
Moderator & Librarian
Posts: 6387
Joined: Sun Sep 25, 2005 3:00 am
Location: Ref Desk

Revamped replace/overwrite dialog

Post by XnTriq »

p127385.png
Green background indicates an exact match, red means… Well, you get the idea :mrgreen:

PS: I also like custom fields!
You do not have the required permissions to view the files attached to this post.
User avatar
xnview
Author of XnView
Posts: 44883
Joined: Mon Oct 13, 2003 7:31 am
Location: France

Re: Revamped replace/overwrite dialog

Post by xnview »

perhaps the font color in red/green only??
Pierre.
User avatar
XnTriq
Moderator & Librarian
Posts: 6387
Joined: Sun Sep 25, 2005 3:00 am
Location: Ref Desk

Re: Revamped replace/overwrite dialog

Post by XnTriq »

xnview wrote:perhaps the font color in red/green only??
That would be great, yes :D
p127406.png
  • File size in bytes (if Tools » Options... » File List » Misc. » Use always bytes for filesize is activated)
  • Timestamp including seconds
  • Optional checksum calculation
Do you think this could be incorporated into the dialog for cmd_Compare, too?
You do not have the required permissions to view the files attached to this post.
User avatar
xnview
Author of XnView
Posts: 44883
Joined: Mon Oct 13, 2003 7:31 am
Location: France

Re: Revamped replace/overwrite dialog

Post by xnview »

or perhaps simply red when values are different?
Pierre.
User avatar
XnTriq
Moderator & Librarian
Posts: 6387
Joined: Sun Sep 25, 2005 3:00 am
Location: Ref Desk

Re: Revamped replace/overwrite dialog

Post by XnTriq »

xnview wrote:or perhaps simply red when values are different?
At least for the checksum, I would prefer both (red & green).

Screenshots:
thibaud
Posts: 274
Joined: Sat Dec 02, 2006 12:41 am

Re: Revamped replace/overwrite dialog

Post by thibaud »

xnview wrote:or perhaps simply red when values are different?
good point.
a single color to indicate the error is what I would do too.
don't ask the user to process more information than needed and keep the ui consistent.
I'm not sure how relevant these md5 check app are in that regard.
User avatar
XnTriq
Moderator & Librarian
Posts: 6387
Joined: Sun Sep 25, 2005 3:00 am
Location: Ref Desk

Re: Revamped replace/overwrite dialog

Post by XnTriq »

Whoot! Implemented in version 2.34 8)
p127534.png
You do not have the required permissions to view the files attached to this post.
obelisk
Posts: 476
Joined: Fri Jan 09, 2009 9:54 am

Re: Revamped replace/overwrite dialog

Post by obelisk »

great work!
User avatar
XnTriq
Moderator & Librarian
Posts: 6387
Joined: Sun Sep 25, 2005 3:00 am
Location: Ref Desk

Re: Revamped replace/overwrite dialog

Post by XnTriq »

  1. As mentioned in a previous post, Use always bytes for filesize (Tools » Options... » File List » Misc.) should also take effect in the replace dialog, IMO.
  2. To prevent the 32-digit MD5 hash from being truncated, the right column of Replace and With could be moved a little farther to the left.
  3. Making use of PathCompactPathEx (or PathCompactPath?) would improve the display of truncated pathnames: X:\Eeny…Moe\Destination\ vs. X:\EenyMeenyMinyMoe\Des
  4. Although the files are not identical, XnView displays the same checksum for Replace (X:\Test\Destination\placeholder.jpg) as for With (X:\Test\Source\placeholder.jpg). :bug:

Code: Select all

Replace: X:\Test\DST\placeholder.jpg (Exif orientation 1) = 3126b90e4c94b4a19380baa7e567c273
   With: X:\Test\SRC\placeholder.jpg (Exif orientation 2) = aa00fea6eb31e37090fd21f21fa123b6
p127537.zip
You do not have the required permissions to view the files attached to this post.
User avatar
Karl02
Posts: 134
Joined: Mon Sep 03, 2007 1:00 pm
Location: Germany

Re: Revamped replace/overwrite dialog

Post by Karl02 »

For symmetry, there should also be functions for "Replace all" and "Skip all". But to avoid too many buttons, there should be a checkbox "[ ] Apply to all".
-- Karl
User avatar
xnview
Author of XnView
Posts: 44883
Joined: Mon Oct 13, 2003 7:31 am
Location: France

Re: Revamped replace/overwrite dialog

Post by xnview »

XnTriq wrote: [*]Making use of PathCompactPathEx (or PathCompactPath?) would improve the display of truncated pathnames:
it's edit textfield, so no need to shorten filename :)
Pierre.
User avatar
XnTriq
Moderator & Librarian
Posts: 6387
Joined: Sun Sep 25, 2005 3:00 am
Location: Ref Desk

Re: Revamped replace/overwrite dialog

Post by XnTriq »

xnview wrote:
XnTriq wrote:Making use of PathCompactPathEx (or PathCompactPath?) would improve the display of truncated pathnames
it's edit textfield, so no need to shorten filename :)
Don't tell me you're rejecting my valuable programming tips :P :wink:

But seriously, I see what you mean: Users can select the text (Directory & Name) with the cursor.
User avatar
xnview
Author of XnView
Posts: 44883
Joined: Mon Oct 13, 2003 7:31 am
Location: France

Re: Revamped replace/overwrite dialog

Post by xnview »

XnTriq wrote:
xnview wrote:it's edit textfield, so no need to shorten filename :)
Don't tell me you're rejecting my valuable programming tips :P :wink:
:P
Pierre.
User avatar
XnTriq
Moderator & Librarian
Posts: 6387
Joined: Sun Sep 25, 2005 3:00 am
Location: Ref Desk

Re: Revamped replace/overwrite dialog

Post by XnTriq »

XnTriq wrote:Although the files are not identical, XnView displays the same checksum for Replace (X:\Test\Destination\placeholder.jpg) as for With (X:\Test\Source\placeholder.jpg). :bug:

Code: Select all

Replace: X:\Test\DST\placeholder.jpg (Exif orientation 1) = 3126b90e4c94b4a19380baa7e567c273
   With: X:\Test\SRC\placeholder.jpg (Exif orientation 2) = aa00fea6eb31e37090fd21f21fa123b6
p127537.zip
placeholder_dst.jpg + placeholder_src.jpg
This bug hasn't been fixed in the latest release (v2.35).
User avatar
XnTriq
Moderator & Librarian
Posts: 6387
Joined: Sun Sep 25, 2005 3:00 am
Location: Ref Desk

Re: Revamped replace/overwrite dialog

Post by XnTriq »

The fix isn't listed in the changelog but since v2.36 this feature works as expected. Thank you, Pierre!