• Resolved KZeni

    (@kzeni)


    I’ve enabled the setting to remove the Algolia logo as my search is no longer a free tier, but the search results page (instantsearch.php) still shows the logo.

    As an aside, the site does have autocomplete turned off to help reduce search operations for now, if that happens to change things in any way (doesn’t seem like it should.)

    Searching this plugin’s current files for the?powered_by_enabled?config setting (see:?https://github.com/search?q=repo%3AWebDevStudios%2Fwp-search-with-algolia+powered_by_enabled&type=code) showed it’s being used in the autocomplete.php template, but there’s no mention of it in the instantsearch.php template.

    Is there some way that the:

    /* Search powered-by widget */
    instantsearch.widgets.poweredBy({
    	container: '#algolia-powered-by'
    })

    code snippet in the instantsearch.php template could check for whether it should be shown via something like:

    if ( algolia.powered_by_enabled ) {
    	/* Insert whatever applicable code's needed here */
    }

    as the autocomplete.php template is currently doing?

    Just seems weird that the setting to remove the logo is seemingly only removing it from one of the two places it’s then shown (unless this issue is unique to this particular site, of course, which doesn’t really seem to be the case since the instantsearch.php doesn’t seem to check for that setting while the autocomplete.php template does it in a way that looks to only then affects the autocomplete output.)

    Of course, one could customize the instantsearch.php template via the theme files to manually remove it, but it’d ideally honor the setting without needing to do so per the autocomplete.php template already doing this (the?algolia?variable with those config values is being included on the search results page with the?powered_by_enabled?value being correct, but the template for the search results just doesn’t check for it, currently.)

    I’ve also posted this at https://github.com/WebDevStudios/wp-search-with-algolia/issues/395 to better facilitate/track this in the appropriate spots.

Viewing 2 replies - 1 through 2 (of 2 total)
Viewing 2 replies - 1 through 2 (of 2 total)
  • The topic ‘Instantsearch template is not honoring the setting to remove powered by logo’ is closed to new replies.