Pavilion Header Search Theme Component

Same here. I can repro this on theme creator and discourse development.

3 Likes

@angus perhaps the component needs to be improved? In light of recent changes:

https://github.com/discourse/discourse/commit/58fe45ffd95dc89bf008eeef28509a4fbc72263b

There is some strange behavior in the drop-down menu.

7 Likes

Broken.

1 Like

Thanks @Stranik and @nexo. I’ve added support for the new event added by https://github.com/discourse/discourse/commit/58fe45ffd95dc89bf008eeef28509a4fbc72263b.

https://github.com/angusmcleod/discourse-header-search-theme/commit/94276e5bef25255496529e1287105bf11353d0d5

The theme will continue to work with older Discourse versions as well.

8 Likes

User and tags go off of the search modal. Could you check if this happens to your instance?

It’s probably something wrong with your other themes. Long usernames are cut off with “…” when only using this component.
29%20PM

4 Likes

Testing this theme component now and I see that once a search is executed the top of the results are getting cut off, whole element is getting shifted up or something. Anyone else seeing this?

It seems to happen even when I switch to a default theme…but @angus I don’t see this active in your sandbox at the moment so not 100% sure if it’s a bug or something weird with my setup.

1 Like

We don’t have space for a search bar on mobile. Has anyone ever seen a search bar integrated into the forum itself, rather than the header? I guess it could also be a footer search bar

Have you looked at this component?

7 Likes

thats exactly what i’m looking for. thank you @Johani.

We’ll modify it to work on tag-pages, which is the same for pretty much every plugin right now.

3 Likes

I noticed this issue too, made a PR with some updates

https://github.com/angusmcleod/discourse-header-search-theme/pull/3

7 Likes

Thanks! Merged :tada:

5 Likes

A nice setting would be to place it exactly in the center of the header :slight_smile:

1 Like

Hey @angus

are there any plans to update the theme component? In theme preview it seems to have some layout issues
I think it’s a great feature that animates user more to use search function more frequently.

4 Likes

I submitted this pr: FIX: deprecations for ember-cli by pfaffman · Pull Request #4 · angusmcleod/discourse-header-search-theme · GitHub

Hopefully it will serve as more than a notice that this deprecation exists. :man_shrugging:

3 Likes

I made a PR that fixes the layout issues.

4 Likes

What are the differences between this #theme-component and the one linked below?

1 Like

hey, for some reason, I’m not being successful with moving the search bar below my header. are there any suggestions? I’ve been trying to nest it within a div class that I created in the after header html

thanks,

Jacob

Hello,

Thank you for the great component.

How can I change the color of the search box?

I’d like to make it white.

Thank you

I’m always pleasantly surprised to find that this component still basically works :slight_smile: I’ve made a few updates to bring this more up to date with the latest Discourse.

Not sure you’re using this component, but it you are add this to your theme CSS

title .search-header #search-term {
  background-color: pink;
}

2 Likes