Skip to content

Commit

Permalink
Fix repeated search weird behavior (#605)
Browse files Browse the repository at this point in the history
Accept the first match after initiating new search.

Signed-off-by: Jonas Dujava <jonas.dujava@gmail.com>
  • Loading branch information
jdujava authored and sebastinas committed Jun 22, 2024
1 parent d3b083c commit cd6679e
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion zathura/shortcuts.c
Original file line number Diff line number Diff line change
Expand Up @@ -873,6 +873,7 @@ bool sc_search(girara_session_t* session, girara_argument_t* argument, girara_ev
const unsigned int num_pages = zathura_document_get_number_of_pages(zathura->document);
const unsigned int cur_page = zathura_document_get_current_page_number(zathura->document);
GtkWidget* cur_page_widget = zathura_page_get_widget(zathura, zathura_document_get_page(zathura->document, cur_page));
bool new_search = argument->data != NULL;
bool nohlsearch = false;
bool first_time_after_abort = false;

Expand Down Expand Up @@ -911,7 +912,7 @@ bool sc_search(girara_session_t* session, girara_argument_t* argument, girara_ev
continue;
}

if (first_time_after_abort == true || (tmp + num_pages) % num_pages != cur_page) {
if (new_search == true || first_time_after_abort == true || (tmp + num_pages) % num_pages != cur_page) {
target_page = page;
target_idx = diff == 1 ? 0 : num_search_results - 1;
break;
Expand Down

0 comments on commit cd6679e

Please sign in to comment.