-
Notifications
You must be signed in to change notification settings - Fork 10
plat-485 Fix Search Front End (theme to be updated later) #541
Conversation
…g to make the page functional again
{% elseif (field_article_image) %} | ||
<div class="search-result__image search-result--{{ nodetype }}__image"> | ||
{{ field_article_image }} | ||
</div> | ||
{% endif %} | ||
</div> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Heleen-cr could you add a new line at the end of the file? thanks!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
</div> | ||
</div> | ||
{% elseif (field_article_image) %} | ||
<div class="search-result__image search-result--{{ nodetype }}__image"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It'd be a little nicer to construct these classes up top in a class array: https://www.drupal.org/docs/8/theming-drupal-8/using-attributes-in-templates
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Create an array for each element? search-result, title, logo and img .. problem also is to add search-result--{{ nodetype }} in the array. Can we do that?
I would change search-result--{{ nodetype }}__image to search-result__title--nodetype
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I will do those changes in another PR.. merging this one now
Fixes https://jira.comicrelief.com/browse/PLAT-485
Changes proposed in this pull request