Operating system
Windows
Joplin version
3.0.6
Desktop version info
Joplin 3.0.6 (prod, win32)
Client ID: 20ee7ddfafe148fbb49548a412501d20
Sync Version: 3
Profile Version: 47
Keychain Supported: Yes
Revision: 18b9f5c
Favorites: 1.3.2
Hotfolder: 1.1.1
Note list (Preview): 1.0.0
What issue do you have?
The menu item Go/Focus/Notelist does nothing if the note list is scrolled such that the current note is not visible in the note list. The focus simply remains where it is (e.g. in the note body). This can be observed by scrolling the list so the current item is not visible, select in the note title or body, then choose the menu item (or use Ctrl+Shift+L) -- the focus remains where it is. Using the up/down arrow keys at this point should change the current note, but does not (because the focus did not move to the list). If the current item IS visible in the list, then Go/Focus/Notelist does work correctly.
Also, imo the fix for this should include scrolling the note list so the current item is in view.
1 Like
Also got the same issue, will this be fixed ?
A fix has been merged, but is not yet in a release:
laurent22:dev
← personalizedrefrigerator:pr/desktop/fix-note-list-scroll
opened 08:41PM - 28 May 24 UTC
# Summary
This pull request fixes #10514 by scrolling the selected note list … item into view before attempting to focus it.
Previously, calling `focusNote` would not focus the selected note if not visible in the note list.
# Testing plan
1. Open a notebook with a large number of notes.
2. Select a note.
3. Press and hold the down arrow to scroll down.
4. Verify that the note list scrolls.
5. Scroll the note list such that the currently selected note is offscreen.
6. Click on the current notebook in the notebook list.
7. Press <kbd>tab</kbd>.
8. Verify that the note list has scrolled the selected note into view.
9. Scroll the selected note offscreen.
10. Open the command palette and run `:focusElementNoteList`
11. Verify that the selected note has been focused.
<!--
Please prefix the title with the platform you are targetting:
Here are some examples of good titles:
- Desktop: Resolves #123: Added new setting to change font
- Mobile, Desktop: Fixes #456: Fixed config screen error
- All: Resolves #777: Made synchronisation faster
And here's an explanation of the title format:
- "Desktop" for the Windows/macOS/Linux app (Electron app)
- "Mobile" for the mobile app (or "Android" / "iOS" if the pull request only applies to one of the mobile platforms)
- "CLI" for the CLI app
If it's two platforms, separate them with commas - "Desktop, Mobile" or if it's for all platforms, prefix with "All".
If it's not related to any platform (such as a translation, change to the documentation, etc.), simply don't add a platform.
Then please append the issue that you've addressed or fixed. Use "Resolves #123" for new features or improvements and "Fixes #123" for bug fixes.
AND PLEASE READ THE GUIDE: https://github.com/laurent22/joplin/blob/dev/readme/dev/index.md
-->
With this release, I can confirm the issue has been fixed !
Thanks so much @personalizedrefriger !
Joplin 3.0.10 (prod, darwin)
Client ID: eb2d1e1494234b77b3f9cd026a11b753
Sync Version: 3
Profile Version: 47
Keychain Supported: Yes
Revision: 45f8e27
Admonition markdown extension: 1.1.0
Agenda: 3.7.1
Auto Alarm: 1.0.8
Backup: 1.4.1
Enhancement: 1.2.1
History Panel: 1.0.2
Inline tags: 1.3.0
Inline TODO: 1.7.1
Markdown Table: Colorize: 1.2.0
Math Mode: 0.6.2
Note Link System: 0.8.0
Note Rename: 1.0.0
Note Tabs: 1.4.0
Outline: 1.5.13
Rich Markdown: 0.15.0
VS-Code Style Note Search: 0.0.1
Whitespacer: 1.0.1
system
Closed
25 July 2024 07:29
5
This topic was automatically closed 30 days after the last reply. New replies are no longer allowed.