localgovdrupal / localgov_search

Default sitewide search implementation for LocalGov Drupal.
GNU General Public License v2.0
0 stars 1 forks source link

Make the header search field required in the block plugin rather than using a hook_form_alter #10

Closed stephen-cox closed 3 years ago

stephen-cox commented 3 years ago

Closes #6

stephen-cox commented 3 years ago

The issue (#6) this fixes has been reported in Slack by someone demoing LGD. Tagging more reviewers as getting this fixes is high priority.

ekes commented 3 years ago

If we're placing the form anyway it makes sense to edit the form directly ourselves.

I'm just baffled why hook_form_alter would cause an empty message, and adding it to the form wouldn't?

stephen-cox commented 3 years ago

I'm just baffled why hook_form_alter would cause an empty message, and adding it to the form wouldn't?

Same here - I'm thinking there's some sort of extra processing going on either before the search form gets form altered, but haven't spent much time delving into what's going on

ekes commented 3 years ago

Well if it's high priority and works when it's on the form when first passed in...

finnlewis commented 3 years ago

Well if @ekes is happy, merge it!