The regular label has line-height: 1.6, while the search-query label has line-height: normal plus a 2px margin-bottom. Should these be consistent, too? This would result in a slightly different look in TypeaheadSearch, which makes me think that we should maintain the current search-query line height but move those styles to TypeaheadSearch since they seem to be specific to that component.

Sorry I missed that! The line-height should be consistent too, even in TahS. I agree that the result of applying a line-height of 1.6 and removing the 2px bottom margin makes the items and menu look slightly different due to the increased white space, but I don't think that the difference is enough to the need to make an exception in this case. Also because, in the future – and depending on the outcome of our conversations around typographic styles – we might apply a different base line-height of 1.4 em*, which would make the menu look even closer to the current version.

In any case, I'll make sure to reach out to Alex to see if he could take a look at the changes before they are merged.

Current TahS	Version with consistent MenuItems	Version with new line-height*

Change 804422 had a related patch set uploaded (by Anne Tomasevich; author: Anne Tomasevich):

[design/codex@main] SearchResultTitle: remove styles for consistency with MenuItem label

https://gerrit.wikimedia.org/r/804422

No problem, thanks, @Sarai-WMDE!

Change 804422 merged by jenkins-bot:

[design/codex@main] SearchResultTitle, styles: remove properties for consistency with MenuItem label

https://gerrit.wikimedia.org/r/804422

Sorry @Sarai-WMDE, I missed the part where you asked to hold off on merging this until you've gotten approval from Alex. For now, I've moved this into design review, and once we hear back from Alex we can handle any changes needed: if there are any, it seems like they might go in Vector, anyway.

No problem at all, @AnneT!

I think it makes sense to ping @alexhollender_WMF here, so he has all the context. Hi Alex! 👋🏻

As the title of the task indicates: we removed the overrides applied to menu suggestions in TahS to make their font-size consistent with the standard Codex menu item.
You can access the updated version of the TypeaheadSearch component here.

As a consequence:

• Menu items will present a font-size that matches the base/body size. This means that menu titles/labels and descriptions will have a size of 14px in Vector 2022.
• As an outcome of applying consistent styling to all menu item instances, the line-height of the text within options was increased from auto to 1.6, which introduces a bit more white space between the options' title and description. Please note that (as mentioned above) this line-height might change again soon (to 1.4) and spacing would be reduced again, once Codex's typographic styles are ready. 👈🏻 We wanted to make you aware of this and hopefully get approval from you. We'll be happy to iterate and apply any needed adjustments, either in Codex or Vector.

@Sarai-WMDE thanks for the heads up — looks great.

After receiving Alex's thumbs-up (thanks so much for checking! 🙏🏻 ) the changes were verified in the latest versions of Chrome, Safari and Firefox on MacOS Monterey, where everything's looking good. The task can be signed-off from a design pov 👍🏻

Looks good to me!

Part of Codex alpha 8 release