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

Feature/fulltext search #4994

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

Conversation

gheber
Copy link
Member

@gheber gheber commented Oct 22, 2024

Restore fulltext search based on Google PSE.

Menu item for fulltext search.
Updated Google PSE ID.
@loricooperhdf
Copy link
Contributor

I would rather replace the corner doxygen-generated search box with the fully functional google search box.

@brtnfld
Copy link
Contributor

brtnfld commented Oct 22, 2024

If I'm searching in the RM, I don't want search results outside the RM. Maybe that is just me. Is that what this accomplishes? Or the default is to search the RM but have the option to do an expanded search.

@loricooperhdf
Copy link
Contributor

If I'm searching in the RM, I don't want search results outside the RM. Maybe that is just me. Is that what this accomplishes? Or the default is to search the RM but have the option to do an expanded search.

No--Gerd's PR here adds a link in the sidebar for Full Text Search. I set up that search to look at the following "directories" and everything under them. For the record, it's here: https://cse.google.com/cse?cx=f671574fa55cc44e6

I would like to change the box in the upper right to function the same way. I think people believe that a search box looks at all of the content on a website. No PR for that yet.

We COULD provide search boxes in certain places, like somewhere in the RM that searches RM only and specifically states that usage. Let's discuss at an upcoming Monday meeting.

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.

3 participants