Skip to content

Macro operator enhancements #6109

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.

Sign up for GitHub

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

Jump to bottom
Open
JV0812 wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Macro operator enhancements #6109

JV0812 wants to merge 6 commits into from

Conversation

@JV0812
Copy link
Collaborator

@JV0812 JV0812 commented Dec 10, 2025 •
edited
Loading

Purpose of this pull request

This pull request is to add details about the below new enhancements to the macro docs:

  • Match by name
  • Highlighting and showing a tooltip
  • Macro recommendations

Select the type of change

  • Minor Changes - Typos, formatting, slight revisions
  • Update Content - Revisions, updating sections
  • New Content - New features, sections, pages, tutorials
  • Site and Tools - .clabot, version updates, maintenance, dependencies, new packages for the site (Docusaurus, Gatsby, React, etc.)

Ticket (if applicable)

Macros new feature documentations

@cla-bot cla-bot bot added the cla-signed Contributor approved, listed in .clabot file label Dec 10, 2025
@JV0812 JV0812 self-assigned this Dec 10, 2025
@JV0812 JV0812 added the do-not-merge Pull requests that should not be merged label Dec 10, 2025
@jpipkin1 jpipkin1 changed the title Macro operator enhncements Macro operator enhancements Dec 10, 2025
jpipkin1
jpipkin1 reviewed Dec 10, 2025
View reviewed changes
jpipkin1
jpipkin1 reviewed Dec 10, 2025
View reviewed changes
jpipkin1
jpipkin1 reviewed Dec 10, 2025
View reviewed changes
jpipkin1
jpipkin1 reviewed Dec 10, 2025
View reviewed changes
jpipkin1
jpipkin1 approved these changes Dec 10, 2025
View reviewed changes
Copy link
Collaborator

@jpipkin1 jpipkin1 left a comment

Choose a reason for hiding this comment

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

Looks good, @JV0812. I have only a few minor suggestions.

kimsauce
kimsauce approved these changes Dec 12, 2025
View reviewed changes
Copy link
Collaborator

@kimsauce kimsauce left a comment

Choose a reason for hiding this comment

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

I cropped the middle space out of static/img/search/searchquerylanguage/search-operators/macro-in-line-suggestions.png and uploaded that version

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@kimsauce kimsauce kimsauce approved these changes

@jpipkin1 jpipkin1 jpipkin1 approved these changes

@mafsumo mafsumo Awaiting requested review from mafsumo mafsumo is a code owner

@amee-sumo amee-sumo Awaiting requested review from amee-sumo amee-sumo is a code owner

Assignees

@JV0812 JV0812

Labels

cla-signed Contributor approved, listed in .clabot file do-not-merge Pull requests that should not be merged

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@JV0812 @kimsauce @jpipkin1