Skip to content

Commit 8946d23

Browse files
authored
Merge pull request #11394 from SAYAN02-DEV/fix-hide-carousel-on-tagged-subjects
fix: Hide default carousel on subject pages with associated Tags
2 parents f7d7196 + c764ce4 commit 8946d23

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

openlibrary/templates/subjects.html

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -55,8 +55,9 @@ <h3>$_("Disambiguations")</h3>
5555
<div class="contentBody">
5656
$if has_tag:
5757
$:format(sanitize(page.tag.body))
58-
59-
$:macros.QueryCarousel(query=page.solr_query, sort='trending,trending_score_hourly_sum', user_lang_only=True, fallback=True)
58+
$else:
59+
$# Only show default carousel if subject doesn't have an associated Tag
60+
$:macros.QueryCarousel(query=page.solr_query, sort='trending,trending_score_hourly_sum', user_lang_only=True, fallback=True)
6061
$:macros.PublishingHistory(page.publishing_history)
6162

6263
<div class="clearfix"></div>

0 commit comments

Comments
 (0)