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.

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

1.3 Alpha - Ghost Notes aren't displayed properly after loading a project #5912

Closed
RoxasKH opened this issue Feb 17, 2021 · 2 comments · Fixed by #5916
Closed

1.3 Alpha - Ghost Notes aren't displayed properly after loading a project #5912

RoxasKH opened this issue Feb 17, 2021 · 2 comments · Fixed by #5916
Labels

Comments

@RoxasKH
Copy link
Contributor

RoxasKH commented Feb 17, 2021

Referring to: #5862 (comment)

Bug Summary

This ia a graphical bug , and as the title suggests, after saving a project that had a ghost notes pattern set, reloading it will make ghost notes not displaying as they should.

Steps to reproduce

  1. Create a new Project
  2. Open an instrument pattern and draw some notes
  3. Set the pattern as ghost notes
  4. Open a new pattern to see that ghost notes are displayed properly (optionally take a screenshot if you wanna do a comparison)
  5. Save the project
  6. Reload the project in LMMS
  7. Open the second pattern you made to see ghost notes
  8. Now ghost notes are not displayed properly as before

Setting again the pattern as ghost notes through songeditor will make ghost notes return as they should be, but it shouldn't be a forced passage.

Screenshot

Before Loading the project (as they should be)
prop

After Loading the project
unprop

Affected LMMS versions

LMMS 1.3.0-alpha.1.33+g4f74151f0-mingw-win64
OS: Windows 10 64-bit

@RoxasKH RoxasKH added the bug label Feb 17, 2021
@cyber-bridge
Copy link
Contributor

I did some digging and I guess this is due ghost notes don't store volume and panning info in the project file and the notes opacity is calculated with those values in void PianoRoll::drawNoteRect.

Possible fix could be to remove volume and panning info from ghost notes on set, I will check this and report soon.

@cyber-bridge
Copy link
Contributor

Well close .. it is due we don't store the volume for ghost notes in project files and on load of a the ghost notes from a project file it sets the volume to 0 since there is no such value and it's the default for asInt() when the value isn't found. Will create a PR for it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants