jiayouxjh / grafx2

Automatically exported from code.google.com/p/grafx2
0 stars 0 forks source link

Having the magnifier settings identical for main and spare page #327

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
"I really hate that zoom/position is separate for normal/spare page...in my
mind, it should be one and the same."

Suggested by DawnBringer

Original issue reported on code.google.com by yrizoud on 18 Mar 2010 at 12:20

GoogleCodeExporter commented 9 years ago
As I understand it, it (can) concern two states:
-When non-zoomed, but with very large picture: The viewport position
-When zoomed: The zoom level, separator position, and the viewport position

In my opinion, it's a good idea in some cases. I just had a lot of problem 
checking
differences in C64 pictures after re-saving them; if the Main and Spare had been
synchronized, it would have been much easier.
I think a setting would be fine for this. Accessable from within grafx2 if 
possible,
because you can need it sometimes, and the next time it's a pain in the neck 
(for
example when you're using the spare to store tiles, and the main for a mockup).
We can call it "Sync views: ON/OFF" or somesuch.

If the images don't have same size... it doesn't really work, and I can't think 
of a
good fallback behavior.

Original comment by yrizoud on 18 Mar 2010 at 12:29

GoogleCodeExporter commented 9 years ago
In this case I was thinking about the second state (C64 example). This is the 
way
Brilliance work. If the zoom-location of the first image (where zoom is set) 
lies
outside the size of the swap-to image - the zoom mode is simply exited when 
swapping.

But as you say, one can imagine situations when the current method could be 
useful
too ;) So a setting option sounds good.

Original comment by annas...@hotmail.com on 18 Mar 2010 at 3:37

GoogleCodeExporter commented 9 years ago
I'd think it's ok to have them synced only if the size of pictures are 
matching. But 
that would be some surprising behaviour...

Original comment by pulkoma...@gmail.com on 18 Mar 2010 at 7:15

GoogleCodeExporter commented 9 years ago

Original comment by pulkoma...@gmail.com on 9 Aug 2010 at 9:43

GoogleCodeExporter commented 9 years ago

Original comment by pulkoma...@gmail.com on 22 Aug 2010 at 1:39

GoogleCodeExporter commented 9 years ago

Original comment by pulkoma...@gmail.com on 22 Aug 2010 at 1:39

GoogleCodeExporter commented 9 years ago
Implemented in r1622.

Original comment by yrizoud on 17 Sep 2010 at 4:57