-
Notifications
You must be signed in to change notification settings - Fork 314
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
Cleanup Autosuggest description and change title for autosuggest and instant results #3207
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@burhandodhy in addition to my comments below, there is also a visual problem we will need to address. This is how I'm seeing it in my local:
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@burhandodhy some last changes before we merge it. Also, I don't know if it is something in my computer but the logo looks "pixelated" to me here (see below.) Even the full-size svg looks a bit pixelated to me.
'</a>', | ||
'<a href="' . esc_url( $epio_autosuggest_kb_link ) . '">', | ||
'</a>', | ||
'<a href="' . esc_url( admin_url( 'site-health.php?tab=debug' ) ) . '">', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@burhandodhy can you please change this link to point to the Status Report page? Thanks.
Description of the Change
This PR updates the titles for the Autosuggest and Instant Results feature if the ElasticPress is connected with ep.io. Also it removed the old checks from the autosuggest description.
Closes #3191
How to test the Change
Changelog Entry
Credits
Props @burhandodhy @felipeelia @NV607FOX
Checklist: