Skip to content

Fix interaction of axis panels when scale factor is more than 1#3994

Merged
swharden merged 3 commits intoScottPlot:mainfrom
bforl:ScaleFactor
Jun 25, 2024
Merged

Fix interaction of axis panels when scale factor is more than 1#3994
swharden merged 3 commits intoScottPlot:mainfrom
bforl:ScaleFactor

Conversation

@bforl
Copy link
Contributor

@bforl bforl commented Jun 25, 2024

If you set the ScaleFactor to a value greater than 1, then interaction with the Axis Panels (mouse interactions) was incorrect.

To repro the issue (without these changes), set the scale factor to something like 1.5 and then try to perform a marquee select in the YAxis only in the area pointed to by the arrow. You'll notice it doesn't work.

image

These changes scale the mouse pos like we do with all the other functions.

I have also cleaned up some of the Scale factor code

  • removed unnecessary checks of Scale factor == 1 (multiply or divide by 1 wont change anything)
  • added helper methods to reduce some of the duplicate code.

@swharden
Copy link
Member

Thanks @bforlgreen, this looks great! I'll merge it in a few minutes 🚀

@swharden swharden enabled auto-merge (squash) June 25, 2024 22:16
@swharden swharden merged commit a1831e5 into ScottPlot:main Jun 25, 2024
KroMignon added a commit to KroMignon/ScottPlot that referenced this pull request Jun 26, 2024
* upstream/main:
  Fix interaction of axis panels when scale factor is more than 1 (ScottPlot#3994)
  Added ResetMinAndMaxValues() to DataLoggerSource.cs (ScottPlot#3993)
  CoordinateLine: add constructor overloads (ScottPlot#3987)
  Colormap.GetColors() (ScottPlot#3983)
  Added a constructor overload that accepts List<Coordinates> (ScottPlot#3982)
  Signal: improve support for IReadOnlyList<T> (ScottPlot#3978)
  Axes: improve sharpness of axis lines, tick marks, and grid lines (ScottPlot#3976)
  adding console write file name function (ScottPlot#3965)
  Color.ToColor()
  Sandbox: extend minimal API
  Sandbox: Create .NET API project
  SVG XML Updates (ScottPlot#3957)
  Repeat render if changes are made in invoked events (ScottPlot#3952)
  CI: autoformat
  Experimental DataLogger2 using a `CircularBuffer<T>` (ScottPlot#3946)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants