Automation Editor point fine tuning with double click

[tecknixia]

Related to issue #5225 and old PR #5232 (closed)

Allows user to double click an automation point in the Automation Editor, and enter new number to adjust the y level for that point. It also improves mouse cursor interaction with the point, and shows the point's Y level when mouse cursor is over a point.

This PR involves the creation of doubleClickEvent, creation of variable m_pointYLevel, and the use of mouseMoveEvent and drawCross.

• A variable, m_pointYLevel is created and used to hold a point's Y level position.
• The value of m_pointYLevel is set when the mouse moves over a point.
• The value of m_pointYLevel is used when opening an input dialog box, for the current value.
• If the mouse moves away from a point, the value of m_pointYLevel returns to 0.
• When the value of m_pointYLevel is 0, the tool tip in drawCross displays the Y level position of the mouse.
• When the value of m_pointYLevel is greater than 0, the tool tip in drawCross displays the point's Y level position.

[LmmsBot]

🤖 Hey, I'm @LmmsBot from github.com/lmms/bot and I made downloads for this pull request, click me to make them magically appear! 🎩

Linux

• Linux (AppImage): lmms-1.2.1.569-linux-x86_64.AppImage (build link)

Windows

• Windows 32-bit: lmms-1.2.1.569-mingw-win32.exe (build link)
• Windows 64-bit: lmms-1.2.1.569-mingw-win64.exe (build link)
• Windows 32-bit: lmms-1.2.1-msvc2017-win32.exe (build link)
• Windows 64-bit: lmms-1.2.1-msvc2017-win64.exe (build link)

macOS

• macOS : lmms-1.2.1.569-mac10.13.dmg (build link)

🤖

{"platform_name_to_artifacts": {"Linux": [{"artifact": {"title": {"title": "(AppImage)", "platform_name": "Linux"}, "link": {"link": "https://5093-15778896-gh.circle-artifacts.com/0/lmms-1.2.1.569-linux-x86_64.AppImage"}}, "build_link": "https://circleci.com/gh/LMMS/lmms/5093?utm_campaign=vcs-integration-link&utm_medium=referral&utm_source=github-build-link"}], "Windows": [{"artifact": {"title": {"title": "32-bit", "platform_name": "Windows"}, "link": {"link": "https://5092-15778896-gh.circle-artifacts.com/0/lmms-1.2.1.569-mingw-win32.exe"}}, "build_link": "https://circleci.com/gh/LMMS/lmms/5092?utm_campaign=vcs-integration-link&utm_medium=referral&utm_source=github-build-link"}, {"artifact": {"title": {"title": "64-bit", "platform_name": "Windows"}, "link": {"link": "https://5090-15778896-gh.circle-artifacts.com/0/lmms-1.2.1.569-mingw-win64.exe"}}, "build_link": "https://circleci.com/gh/LMMS/lmms/5090?utm_campaign=vcs-integration-link&utm_medium=referral&utm_source=github-build-link"}, {"artifact": {"title": {"title": "32-bit", "platform_name": "Windows"}, "link": {"link": "https://ci.appveyor.com/api/buildjobs/k5m67s4pud0h95eg/artifacts/build/lmms-1.2.1-msvc2017-win32.exe"}}, "build_link": "https://ci.appveyor.com/project/Lukas-W/lmms/builds/28696442"}, {"artifact": {"title": {"title": "64-bit", "platform_name": "Windows"}, "link": {"link": "https://ci.appveyor.com/api/buildjobs/e5en8sjfpe06gjkv/artifacts/build/lmms-1.2.1-msvc2017-win64.exe"}}, "build_link": "https://ci.appveyor.com/project/Lukas-W/lmms/builds/28696442"}], "macOS": [{"artifact": {"title": {"title": "", "platform_name": "macOS"}, "link": {"link": "https://5094-15778896-gh.circle-artifacts.com/0/lmms-1.2.1.569-mac10.13.dmg"}}, "build_link": "https://circleci.com/gh/LMMS/lmms/5094?utm_campaign=vcs-integration-link&utm_medium=referral&utm_source=github-build-link"}]}, "commit_sha": "cf50b86bd1607b3988899292e62e2effdab4bc41"}

[tecknixia]

@russiankumar
#5592
Waiting for reviews and testing.

[superpaik]

@tecknixia
Hi, Sorry I'm new in the testing team. Is this PR ready to test? The builds done by the bot are no longer availaible.
I've downloaded the PR from github and build it locally. Is that the last code available for this feature?
First test is ok. Minor issues:

• the popup window show 0,000 as value. I think it should be great to have the actual value of the point.
• I'm using a spanish version of windows. So if I enter 100,89 it works ok, but if I do 100.89 (with dot) it translate into 10089. Maybe is related to local version.

[IanCaio]

I just adapted this PR to have #5712 as a base branch, since it changes quite a lot of the AutomationEditor/AutomationPattern and some tweaking would be necessary to make this PR compatible with it. I kept the original authorship by commiting with --author="tecknixia" and it's ready to be submitted as a separate PR once the changes from the AutomationEditor are merged. Once it's submitted this one can be closed!

IanCaio/lmms@feature/automationProgression...IanCaio:feature/automationProgressionFineTuning

[IanCaio]

Closing this as #5923 which rewrote this over the new AutomationPattern framework was already merged.