Skip to content

GC Search: Relocation of specific results section styles#2718

Merged
Garneauma merged 1 commit intowet-boew:masterfrom
mmoya064:639
Feb 10, 2026
Merged

GC Search: Relocation of specific results section styles#2718
Garneauma merged 1 commit intowet-boew:masterfrom
mmoya064:639

Conversation

@mmoya064
Copy link
Copy Markdown
Contributor

Related to WET-639
Relocation of specific results section styles to #wb-land

Copy link
Copy Markdown
Contributor

@tristandasilva tristandasilva left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The CSS was moved correctly, but because of an incorrect layout in the HTML it changes the font size of the result count text on a couple of the pages.

  • In results-advanced-en/fr (line 63-64) and results-filters-en/fr (line 47-48) we can just remove this: <h2 class=”wb-inv”>Search Results</h2>, and change the <p> tag under it to an <h2> and remove the class=”h4”
  • In results-en/fr can you please remove the <div> that the <h2> is wrapped in (lines 27-29)?
  • In results-en we can remove the id and class on the form element to make it consistent with the other pages (line 14).

Thank you :)

Copy link
Copy Markdown
Contributor

@tristandasilva tristandasilva left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Garneauma Ready for your review!

@Garneauma Garneauma self-assigned this Feb 9, 2026
@Garneauma
Copy link
Copy Markdown
Collaborator

This is a Patch change.

@BeraJosh For your review. Thanks!

@Garneauma Garneauma added this to the v18.3.0 milestone Feb 10, 2026
Copy link
Copy Markdown

@BeraJosh BeraJosh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved

@Garneauma Garneauma merged commit 97c04d8 into wet-boew:master Feb 10, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants