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.

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

[Feature Request] - Plot zooming #1258

Closed
FabianInostroza opened this Issue Dec 7, 2017 · 7 comments

Comments

Projects
None yet
2 participants
@FabianInostroza
Copy link
Contributor

FabianInostroza commented Dec 7, 2017

Details for the issue

It would be useful to be able to zoom in the plot pane. That feature could help reduce iterations
in reducing the the range for the X axis.

Useful extra information

QCustomPlot already support zooming, it's a matter of enabling it and defining or adding controls for
the used to control the interaction (zoom on X axis only, Y axis only, etc).

I'm opening this issue because:

  • DB4S is crashing
  • DB4S has a bug
  • DB4S needs a feature
  • DB4S has another problem

I'm using DB4S on:

  • Windows: ( version: ___ )
  • Linux: ( distro: ___ )
  • Mac OS: ( version: ___ )
  • Other: ___

I'm using DB4S version:

  • 3.10.1
  • 3.10.0
  • 3.9.1
  • Other: ___

I have also:

@mgrojo

This comment has been minimized.

Copy link
Contributor

mgrojo commented Dec 7, 2017

I'm glad to say that zooming is implemented in our latest nightly builds. That was my solution to issue #838. You can drag the plot for changing the view over the axis and zoom in and out with the mouse wheel.

If you'd like to try: https://nightlies.sqlitebrowser.org/latest

@FabianInostroza

This comment has been minimized.

Copy link
Contributor Author

FabianInostroza commented Dec 7, 2017

mmm, I compiled from sources yesterday, didn't found the feature. Will check again.

@mgrojo

This comment has been minimized.

Copy link
Contributor

mgrojo commented Dec 7, 2017

You can also click on a point to get the row selected in the associated table.

@FabianInostroza

This comment has been minimized.

Copy link
Contributor Author

FabianInostroza commented Dec 7, 2017

Thanks, the table and graph synchronization is a nice feature. Could a checkbox (or some other way) be added to limit zooming to the X axis only?.

I accidentally hit the ESC key with the focus on the plot pane and it was hidden. I was unable to bring it back.

@mgrojo

This comment has been minimized.

Copy link
Contributor

mgrojo commented Dec 7, 2017

Ok, let's leave this issue open for the enhancement about zooming to only one axis.

The problem with the ESC is an ugly bug 😄 The same happens to the Remote dock. Not sure why when all are QDialog. A different issue would be ideal.

mgrojo added a commit that referenced this issue Dec 8, 2017

Allow zooming and dragging only in one orientation
By selecting the axis or the labels associated to the axis,the user can
restrict the subsequent zoom or drag operation to that orientation.

The possible user interactions are documented in the Whats's This
information.

See issue #1258

@mgrojo mgrojo self-assigned this Dec 8, 2017

@mgrojo mgrojo added the enhancement label Dec 8, 2017

@mgrojo

This comment has been minimized.

Copy link
Contributor

mgrojo commented Dec 8, 2017

@FabianInostroza I've implemented your suggestion of zooming (and dragging) only in one orientation. You have to select first the axis (or associated labels) and then the wheel and drag operations will affect only to that axis.

I've also documented the possible interactions in the What's This information, so users can discover them. Well, if they ever use this feature 😄

If you'd like to test it, it will be available in the next nightly build: https://nightlies.sqlitebrowser.org/latest

@FabianInostroza

This comment has been minimized.

Copy link
Contributor Author

FabianInostroza commented Dec 8, 2017

Wonderful, many thanks.
I use sqlitebrowser mainly for data exploration and this is really useful to me, thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment