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

Desktop: Re-ordering notelist items causes unwanted height change #6639

Closed
Daeraxa opened this issue Jul 4, 2022 · 3 comments · Fixed by #6640
Closed

Desktop: Re-ordering notelist items causes unwanted height change #6639

Daeraxa opened this issue Jul 4, 2022 · 3 comments · Fixed by #6640
Labels
bug It's a bug desktop All desktop platforms

Comments

@Daeraxa
Copy link
Collaborator

Daeraxa commented Jul 4, 2022

Environment

Joplin version: dev branch since commit fb9e78d
Platform: Linux
OS specifics: Fedora 36

Joplin 2.8.8 (dev, linux)
Client ID: f8a67ce4381c4d96943c96a0203aebc8
Sync Version: 3
Profile Version: 41
Keychain Supported: No
Revision: fb9e78d

Originally discovered by "theParadoxica" on the Joplin Discord server
PR for commit: #6542

Steps to reproduce

  1. Enable custom ordering in a notebook with at least two notes
  2. Drag note 2 above note 1 -> observe note height change
  3. Restart Joplin to reset note heights (changing notebook has no effect)

Describe what you expected to happen

Notelist items height should not change on reordering

Logfile

No log events appear to be generated but issue is easy to replicate

@Daeraxa Daeraxa added bug It's a bug desktop All desktop platforms labels Jul 4, 2022
@ken1kob
Copy link
Contributor

ken1kob commented Jul 4, 2022

This symptom doesn't happen when the height is styled.

Restart Joplin to reset note heights (changing notebook has no effect)

Also, they can be reset by showing option screen. It's a convenient emergency exit. 😀

@Daeraxa
Copy link
Collaborator Author

Daeraxa commented Jul 4, 2022

I also noticed it reacts to how long you "hold" the notebook at the top. If you just flash over it then it only slightly reduces, holding it makes it keeps reducing until it it gets to 8px when it stops shrinking.

@ken1kob
Copy link
Contributor

ken1kob commented Jul 4, 2022

Cause of the Symptom

When a note list item is dragged, its height is sometimes transiently being changed.

Treatment

When a note list item is dragged, its height should not be considered as its permanent height.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug It's a bug desktop All desktop platforms
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants