Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Simplify preview window #54

Closed
alex285 opened this issue Jun 22, 2019 · 2 comments
Closed

Simplify preview window #54

alex285 opened this issue Jun 22, 2019 · 2 comments
Labels
enhancement ideas to improve existing features

Comments

@alex285
Copy link

alex285 commented Jun 22, 2019

I believe all those controls take unnecessary space, without any real functionality

  1. The arrows on red circles, aren't really needed. You can move the preview area, either with mouse, or touch (Gesture)

  2. The zoom controls on blue circle (that isn't a circle) is redundant since the slider does the job perfectly

  3. Fit on screen (i guess thats the square, b/c doesn't work) would be better if it was outside the previewer, somewhere on main window

Screenshot from 2019-06-22 23-57-55
@alex285 alex285 added the enhancement ideas to improve existing features label Jun 22, 2019
@maoschanz
Copy link
Owner

See #49: it's complex because you turned on the devel-only setting. With 0.4 you should only have arrows, with 0.6 you will only have zoom controls

The slider is cool because it shows the current value and gives precise control, but i'm not sure it's as usable as the buttons, anyway you will probably control that with ctrl+scroll, not with the slider

@alex285
Copy link
Author

alex285 commented Jun 22, 2019

Oh, you have removed the "experimental" from the UI, and probably the flag was there from a previous installation :/

About the slider, personally I prefer it a million times more than buttons! Besides in all graphics programs we use sliders, and not buttons. Basically there is this thing (on professional apps), that we can slide even when there are buttons, with press-down. Point is that sliders work better :)

maoschanz added a commit that referenced this issue Sep 8, 2019
@maoschanz
Fix numerous zoom-related problems
8ed39d0 
that's cards in project n°4, and issue #78 (also it's mentionned in #54 and others)

- scrollbars change their capacity/size to match the zoom level so we can scroll with zoomed images
- the minimap is now produced with quite different methods to take into account the zoom level
- the minimap button now shows a translatable label which is updated when the user changes the active tab
- the coordinates used by the tools to draw are now correct according to the zoom level
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement ideas to improve existing features
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@alex285 @maoschanz