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

Improve search system #532

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Improve search system #532

wants to merge 1 commit into from

Conversation

mbkma
Copy link
Member

@mbkma mbkma commented Jun 6, 2021

shell: Add EvFindSidebar widget to show search results
see https://gitlab.gnome.org/GNOME/evince/-/commit/940ce8048e4fc6503dca03c1ce090ff4a11dface

eggfindbar: don't close the findbar when it loses the focus
see https://gitlab.gnome.org/GNOME/evince/-/commit/a2d781fb124bd19861035fd41dacc976e00d19ef

shell: Add the findbar to the main box instead of the view box

libview: Add ev_view_find_set_result

Fixes #238

shell: Add EvFindSidebar widget to show search results
see https://gitlab.gnome.org/GNOME/evince/-/commit/940ce8048e4fc6503dca03c1ce090ff4a11dface

eggfindbar: don't close the findbar when it loses the focus
see https://gitlab.gnome.org/GNOME/evince/-/commit/a2d781fb124bd19861035fd41dacc976e00d19ef

shell: Add the findbar to the main box instead of the view box

libview: Add ev_view_find_set_result
@mbkma mbkma requested a review from a team June 6, 2021 08:27
@raveit65
Copy link
Member

I tried to back port the same functionality 2 years ago or so. So i like this.
But i recall that there was a lot of more commits which are related to the new search sidebar, but i am not sure.

@@ -72,6 +72,9 @@ void ev_view_zoom (EvView *view,
/* Find */
void ev_view_find_next (EvView *view);
void ev_view_find_previous (EvView *view);
void ev_view_find_set_result (EvView *view,
gint page,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Indents

@@ -5729,6 +5771,15 @@ attachment_bar_menu_popup_cb (EvSidebarAttachments *attachbar,
return TRUE;
}

static void
find_sidebar_result_activated_cb (EvFindSidebar *find_sidebar,
gint page,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Indents

@raveit65
Copy link
Member

Beside from a few code-style issues it works fine, thanks.

@mbkma
Copy link
Member Author

mbkma commented Jul 10, 2021

This does not work with epub documents yet...

@raveit65
Copy link
Member

@mbkma
Any chance to fix code-style issues?

@raveit65
Copy link
Member

Opps, cross posting :)

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.

[feature request] backport search functionality from evince
2 participants