Not entirely sure what’s causing this, but it seems like it is probably related to Themeable site settings are now available
Which theme are you using? How are these settings for the theme configured?
I think if you have search field in site header set and this component installed, you end up in the state you describe where search is only in the banner and doesn’t appear in the header at all… does that sound accurate?
Enig idee van de hoofdoorzaak en of er een componentfix of een discoursefix nodig is?
I updated one of my sites and this theme component got… squished.
Unfortunately, I don’t have a screenshot of what the homepage looked like before the update, but I’ll post a screenshot of what it looks like now. It’s basically been smashed so it’s not as tall as it used to be.
And here’s the original banner, the search bar used to be in the open area that’s now missing.
Het lijkt erop dat het geen probleem is met de zoekbanner op zich.
Ik heb ervoor gezorgd dat het vakje om de themacomponent bij te werken met Discourse-updates niet was aangevinkt, daarna heb ik mijn andere productiesite bijgewerkt, en die verkleinde de banner ook.
Hoe krijg ik de oorspronkelijke bannergrootte terug? Deze kleinere zijn lelijk.
Found a screenshot of pre-update on my 2nd site.
What it used to look like:
And now:
The admin notice is for an unrelated bit of code and does not exist on my other site.
Same here, for some reason my banner broken
try this temporary fix => desktop => common
.search-banner {
background-image: url(https://segredin.com/uploads/default/original/1X/3f044a906d2c358806610fc756e09991d25323ed.webp);
background-size: cover;
background-repeat: no-repeat;
border-radius: 15px;
margin-bottom: 10px !important;
height: 17em;
padding-top: 50px
}
@edit
Vastgezet met padding-top: 50px
Natuurlijk is dit nog steeds 100% kapot op mobiel.
Wat is uw site? Op mobiel verdwijnt deze zoekbanner gewoon, misschien is er een klasse veranderd, ik weet het niet.
Looks like a variable’s name was changed and that broke the padding, I’ve just added a fix FIX: restore padding broken due to missing variable by awesomerobot · Pull Request #100 · discourse/discourse-search-banner · GitHub
So you should be able to restore the previous spacing by updating the component.
Yeah there’s a simpler version of the banner included in Discourse by default now, and there’s some CSS to hide it on mobile… it can be restored with some custom CSS
@use "lib/viewport";
@include viewport.until(sm) {
.welcome-banner {
display: block;
}
}
Perfect, hartelijk dank voor het herstellen van deze component!
Volgt dit onderdeel gebruiksgegevens? We willen graag zien hoe effectief het daadwerkelijk is voor onze community.
@bartv Nee, dat geloof ik niet.
Het lijkt het probleem niet te hebben opgelost. De zoekbalk in de bovenste sitekopsectie verschijnt gewoon niet. Ik heb zojuist bijgewerkt naar de nieuwste 3.6.0 Beta1-Dev en het probleem bestaat nog steeds. De enige manier waarop ik de zoekbalk in de bovenste sitekop terug kan krijgen, is door deze component uit te schakelen en de Discourse Header Search component in te schakelen.
Enig idee waarom dit is gestopt met werken? Dit was een van de mooiste componenten die ik gebruikte, ik mis de functies ervan echt.
I guess that answers my question. After the latest update to the Welcome banner, I just enabled that and it seems to replace the functionality of this component as well as the Discourse Header Search.
The welcome banner is still missing some nice customization this component has like the ability to add a subhead text (extremely helpful as it can prompt users to use the search bar rather than just lie there like an empty box). Hopefully that will make it’s way into the Welcome banner.
On the flip side, is there a way to use the CSS customization feature in discourse to edit the subheading text in the search banner ?
Are you referring to this Advanced Search Banner component? If so, I believe this setting does this:
You can change the text variable js.search.title in /admin/customize/site_texts.
But as you mentioned in Creating a banner to display at the top of your site - #23 by RBoy, the more specific variable should be js.welcome_banner.search- but it doesn’t seem to work right now.
