diff --git a/web/styles/search.css b/web/styles/search.css index 1e7ebb0598..1f1eb4786c 100644 --- a/web/styles/search.css +++ b/web/styles/search.css @@ -391,6 +391,10 @@ align-items: center; } + .search_list_item .description { + margin: 2px 0; + } + .search_list_item .pill-container { margin: 2px 0; /* This contains only one pill, which handles its own border */ diff --git a/web/templates/search_list_item.hbs b/web/templates/search_list_item.hbs index 7e262087f6..d599f4eabb 100644 --- a/web/templates/search_list_item.hbs +++ b/web/templates/search_list_item.hbs @@ -1,14 +1,12 @@
{{#each pills}} - - {{#if (eq this.type "search_user")}} - {{> search_user_pill this}} - {{else if (eq this.operator "search")}} - {{{this.description_html}}} - {{else}} - {{> input_pill this}} - {{/if}} - + {{#if (eq this.operator "search")}} +
{{{this.description_html}}}
+ {{else if (eq this.type "search_user")}} + {{> search_user_pill this}} + {{else}} + {{> input_pill this}} + {{/if}} {{/each}} {{#if description_html}}
{{{description_html}}}
{{/if}}
diff --git a/web/tests/search.test.cjs b/web/tests/search.test.cjs index c7856cda5b..80326e298f 100644 --- a/web/tests/search.test.cjs +++ b/web/tests/search.test.cjs @@ -115,12 +115,12 @@ run_test("initialize", ({override, override_rewire, mock_template}) => { /* Test highlighter */ let description_html = "Search for ver"; - let expected_value = `
\n \n ${description_html}\n \n \n
\n`; + let expected_value = `
\n
Search for ver
\n \n
\n`; assert.equal(opts.item_html(source[0]), expected_value); const search_string = "channel: Verona"; description_html = "Stream Verona"; - expected_value = `
\n \n
\n \n \n ${search_string}\n \n
\n \n
\n
\n
\n
${description_html}
\n
\n`; + expected_value = `
\n
\n \n \n ${search_string}\n \n
\n \n
\n
\n
\n
${description_html}
\n
\n`; assert.equal(opts.item_html(source[1]), expected_value); /* Test sorter */ @@ -170,7 +170,7 @@ run_test("initialize", ({override, override_rewire, mock_template}) => { /* Test highlighter */ const description_html = "Search for zo"; - let expected_value = `
\n \n ${description_html}\n \n \n
\n`; + let expected_value = `
\n
${description_html}
\n \n
\n`; assert.equal(opts.item_html(source[0]), expected_value); people.add_active_user({ @@ -179,13 +179,13 @@ run_test("initialize", ({override, override_rewire, mock_template}) => { full_name: "Zoe", }); override(realm, "realm_enable_guest_user_indicator", true); - expected_value = `
\n \n
\n sender:\n \n
\n \n
\n \n Zoe\n
\n \n
\n
\n
\n
\n \n
\n`; + expected_value = `
\n
\n sender:\n \n
\n \n
\n \n Zoe\n
\n \n
\n
\n
\n
\n \n
\n`; assert.equal(opts.item_html(source[1]), expected_value); - expected_value = `
\n \n
\n dm:\n \n
\n \n
\n \n Zoe\n
\n \n
\n
\n
\n
\n \n
\n`; + expected_value = `
\n
\n dm:\n \n
\n \n
\n \n Zoe\n
\n \n
\n
\n
\n
\n \n
\n`; assert.equal(opts.item_html(source[2]), expected_value); - expected_value = `
\n \n
\n dm-including:\n \n
\n \n
\n \n Zoe\n
\n \n
\n
\n
\n
\n \n
\n`; + expected_value = `
\n
\n dm-including:\n \n
\n \n
\n \n Zoe\n
\n \n
\n
\n
\n
\n \n
\n`; assert.equal(opts.item_html(source[3]), expected_value); /* Test sorter */