Consider updating Classy's search result markup

+<h3{{ title_attributes.addClass('search-result__title') }}> <a href="{{ url }}">{{ title }}</a> </h3> {{ title_suffix }} <div class="search-result__snippet-info"> {% if snippet %} <p{{ content_attributes.addClass('search-result__snippet') }}>{{ snippet }}</p> {% endif %} {% if info %} <p class="search-result__info">{{ info }}</p> {% endif %} </div>

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

First commit to issue fork.
Merge request !8192Updated search result markup class names as per BEM → (Open) created by Unnamed author
Status changed to Needs review 6 months ago3:41am 27 May 2024
Comment 6 months ago →
🇮🇳India gauravvvv Delhi, India
Pipeline finished with Success
6 months ago
Total: 514s
#182940
Status changed to Needs work 6 months ago2:57pm 27 May 2024
Comment 6 months ago →
🇺🇸United States smustgrave
First item in the tasks is to discuss but don't see that. This could have some BC concerns that need to be considered.
Comment 2 months ago →
saurav-drupal-dev
Hi @smustgrave

for the discussion please share ur thought what approach we should be considering?
Comment about 2 months ago →
🇮🇳India nayana_mvr
As per comment in the parent ticket https://www.drupal.org/project/drupal/issues/2489394#comment-9956761 →

it's confusing that we have search-result__info, search-result__snippet, and search-result__snippet-info, all as different things.

So, do we need to change all the three class names or can we just change .search-result__snippet-info to something which doesn't contain the words info or snippet. I think .search-result__content used in the MR is a good example or else please suggest new class names.
After the current MR changes, this is how it looks like:-

Consider updating Classy's search result markup

Problem/Motivation

Beta phase evaluation

Proposed resolution

Remaining tasks

User interface changes

API changes

Merge Requests

!8192Consider updating Classy's search result markup
Open

Comments & Activities

Consider updating Classy's search result markup

Problem/Motivation

Beta phase evaluation

Proposed resolution

Remaining tasks

User interface changes

API changes

Merge Requests

!8192Consider updating Classy's search result markupOpen

Comments & Activities

!8192Consider updating Classy's search result markup
Open