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

Implement a search counter, next to the page counter #644

Open
wants to merge 5 commits into
base: develop
Choose a base branch
from

Conversation

laomuon
Copy link
Contributor

@laomuon laomuon commented Jul 28, 2024

This PR is the first step at implementing the search counter.
Fixes #450

@ldwgchen ldwgchen mentioned this pull request Jul 29, 2024
Copy link
Member

@sebastinas sebastinas left a comment

Choose a reason for hiding this comment

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

Not tested yet, but here are some initial comments.

unsigned int number_of_pages = zathura_document_get_number_of_pages(zathura->document);
unsigned int current_page_number = zathura_document_get_current_page_number(zathura->document);
zathura->global.total_search_results = 0;
zathura->global.current_search_result = 0;
Copy link
Member

Choose a reason for hiding this comment

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

Add an empty line in between and avoid the reformatting of the two lines above.

Copy link
Member

Choose a reason for hiding this comment

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

Please skip the reformatting of lines 413 and 414.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

.gitignore Outdated Show resolved Hide resolved
zathura/commands.c Outdated Show resolved Hide resolved
zathura/commands.c Outdated Show resolved Hide resolved
zathura/commands.c Outdated Show resolved Hide resolved
zathura/shortcuts.c Show resolved Hide resolved
zathura/zathura.h Outdated Show resolved Hide resolved
zathura/zathura.h Show resolved Hide resolved
zathura/zathura.c Outdated Show resolved Hide resolved
zathura/zathura.c Outdated Show resolved Hide resolved
@laomuon
Copy link
Contributor Author

laomuon commented Aug 5, 2024

@sebastinas Sorry for the late reply. I made the changes that you requested, but actually all these previous changes came from clang-format, which I ran on all these files

@laomuon laomuon requested a review from sebastinas August 5, 2024 12:07
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.

[Question] How to display search match counters?
2 participants