Skip to content

Navigate to selected bookmark#5157

Merged
kjk merged 1 commit into
sumatrapdfreader:masterfrom
DerDunkelheit:navigation-to-selected-bookmark
Dec 27, 2025
Merged

Navigate to selected bookmark#5157
kjk merged 1 commit into
sumatrapdfreader:masterfrom
DerDunkelheit:navigation-to-selected-bookmark

Conversation

@DerDunkelheit
Copy link
Copy Markdown
Contributor

@DerDunkelheit DerDunkelheit commented Oct 29, 2025

In this pull request I made FavTreeItemClicked working for currently selected tree item.

When there’s only one bookmark, you can navigate to it only once because navigation to the bookmark page happens in the FavTreeSelectionChanged callback. This callback is triggered only when a different tree element is selected.

In the code, there’s also an onClick handler, but it was commented out in this commit: 32b2981

because of two related issues.:
#1716 #2202

In this pull request I made FavTreeItemClicked working for currently selected tree item.

When there’s only one bookmark, you can navigate to it only once because navigation to the bookmark page happens in the FavTreeSelectionChanged callback. This callback is triggered only when a different tree element is selected.

In the code, there’s also an onClick handler, but it was commented out in this commit: sumatrapdfreader@32b2981#diff-84fd94c3d33b1f12ded82c18e0ebb3ad66dde124291369437cd2782c21171e82

because of two related issues.:
sumatrapdfreader#1716
sumatrapdfreader#2202
@kjk kjk merged commit 1b2fd3b into sumatrapdfreader:master Dec 27, 2025
@kjk
Copy link
Copy Markdown
Member

kjk commented Dec 27, 2025

Thanks!

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