Skip to content

Conversation

emilk
Copy link
Member

@emilk emilk commented Oct 13, 2025

The most important thing for me is to easily be able to toggle between old/new in a flip-book like fashion. Previously that required holding down a key, which was annoying. Now you can press 2, then 3, then back to 2, etc. Easy. Clean. I think also the new UI is more self-explanatory, using radio buttons instead of checkboxes.

Let me know if I broke some existing workflow.

@emilk emilk added the enhancement New feature or request label Oct 13, 2025
@emilk
Copy link
Member Author

emilk commented Oct 13, 2025

CI is red on main; fixing that in

@lucasmerlin
Copy link
Member

Hmm, I'm not convinced that this is better.
It is twice as much "work" to switch between old / new now. Before, you could hold 1 to see the old image. Press 2 to see new. Release to see old.
Now, you have to press and release to see old, press and release (a different key!) to see new, press and release to see old again.

To me it just doesn't feel as good, but maybe I'm just used to the other controls now.

Also it's not possible anymore to see just the old or new image with the diff overlayed.

@emilk
Copy link
Member Author

emilk commented Oct 15, 2025

It is twice as much "work" to switch between old / new now. Before, you could hold 1 to see the old image. Press 2 to see new. Release to see old.

It's not about how much work it is as much as about about how intuitive it is.

Before:

  • Hold down 1 to see old image
  • Hold down both 1 and 2 to see new image (because if you only hold down 2, there may be a short frame in between when you will see the blended mode again!). Or wait, does holding down both show both in case of alpha? Probably. So maybe there was no way to actually toggle old and new image before?

After:

  • Press 1 to see old image
  • Press 2 to see old image

maybe I'm just used to the other controls now.

100%

Also it's not possible anymore to see just the old or new image with the diff overlayed.

Old with diff overlayed:
image

New with diff overlayed:
image

…but I can also add Old + Diff and New + Diff as their own view states with their own keyboard numbers?

@emilk
Copy link
Member Author

emilk commented Oct 15, 2025

image

Copy link
Member

@lucasmerlin lucasmerlin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

@emilk emilk merged commit e066013 into main Oct 15, 2025
25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants