Skip to content

Commit 405f74f

Browse files
author
Chris Hart
committed
Remove accidentally commented-out Group logic
Fixes: CNVS-35246 I introduced this bug when fixing CNVS-34740 by commenting out some logic so I could see the search input while working on the layout. The commented-out logic was committed; this commit reinstates it. Test plan: - Go to [any course] > People > + Group Set - Cancel out of the modal - The search input for Search Groups should now only show if `groupsAreSearchable` is true Change-Id: I63ddb564b44fc7f22cdda4388e1a2fff7cfa6543 Reviewed-on: https://gerrit.instructure.com/103321 Tested-by: Jenkins Reviewed-by: Colleen Palmer <colleen@instructure.com> Product-Review: Colleen Palmer <colleen@instructure.com> QA-Review: Chris Hart <chart@instructure.com> QA-Review: Dan Sasaki
1 parent f919b5d commit 405f74f

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

app/views/jst/groups/manage/groupCategory.handlebars

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
<span class="accessibility-warning" tabindex="0"> Warning: for improved accessibility assigning students to groups, please use the Move To Dialog option found in the menu.</span>
22

33
<div class="group-category-summary">
4-
{{!#if groupsAreSearchable}}
4+
{{#if groupsAreSearchable}}
55
<div class="group-category-summary__search">
66
<input
77
type="text"
@@ -11,7 +11,7 @@
1111
role="search"
1212
>
1313
</div>
14-
{{!/if}}
14+
{{/if}}
1515
<div class="group-category-summary__detail" data-view="groupCategoryDetail"></div>
1616
</div>
1717

0 commit comments

Comments
 (0)