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

Browser: Add find-in-page UI #25580

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

Conversation

AvadhDPatel
Copy link

@AvadhDPatel AvadhDPatel commented Dec 25, 2024

The implementation adds a find-in-page feature to the browser, allowing users to search for text within the active web page, navigate between matches, and clear highlights. In Tab.cpp, methods find_in_page, find_next, and find_previous are added to handle search operations by interacting with the web view (WebView). In BrowserWindow.cpp, a search bar UI is integrated with a TextBox for input, "Next" and "Previous" buttons for navigation, and a "Close" button to hide the search bar. The search bar is triggered via a menu action (e.g., Ctrl+F). The WebView is extended with methods like search_for_text, find_next_match, and clear_find_highlights to manage the search functionality and highlighting within the web content. This cohesive integration provides seamless in-page search capabilities.

@github-actions github-actions bot added the 👀 pr-needs-review PR needs review from a maintainer or community member label Dec 25, 2024
@nico
Copy link
Contributor

nico commented Dec 26, 2024

What happened here? This isn't even close to compiling. Did you forgot to add a few files to the commit?

@nico nico added ⏳ pr-waiting-for-author PR is blocked by feedback / code changes from the author and removed 👀 pr-needs-review PR needs review from a maintainer or community member labels Dec 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⏳ pr-waiting-for-author PR is blocked by feedback / code changes from the author
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants