commit 773de8228a04d6c141853934f0d14bb20bf82f0b
author Dmitrii Filippov <dmfilippov@google.com> Thu Apr 08 10:45:56 2021 +0200
committer Dmitrii Filippov <dmfilippov@google.com> Thu Apr 08 10:45:56 2021 +0200
tree 34eec899ce46ef162d5f60720f59151276e1c831
parent 4f04362b0a4279a29fce89ae009df8ca640a98e1

Always show owners below file name when "Show all owners" is checked

Change-Id: Ib9899bedfc99a8a1d1e8cac9307a7d60120c5bd3

ui/suggest-owners.js

diff --git a/ui/suggest-owners.js b/ui/suggest-owners.js
index 9d3f183..c3ea6cf 100644
--- a/ui/suggest-owners.js
+++ b/ui/suggest-owners.js
@@ -16,7 +16,6 @@
  */
 import {CodeOwnersModelMixin} from './code-owners-model-mixin.js';
 import {
-  BestSuggestionsLimit,
   SuggestionsState,
   SuggestionsType,
 } from './code-owners-model.js';
@@ -310,7 +309,7 @@
                 </div>
               </template>
               <template is="dom-if" if="[[!suggestion.owners.owned_by_all_users]]">
-                <template is="dom-if" if="[[_breakBeforeOwners(suggestion.owners.code_owners, _showAllOwners)]]">
+                <template is="dom-if" if="[[_showAllOwners]]">
                   <div class="flex-break"></div>
                 </template>
                 <ul class="suggested-owners">
@@ -675,11 +674,6 @@
     this.model.setSelectedSuggestionType(showAll ?
       SuggestionsType.ALL_SUGGESTIONS : SuggestionsType.BEST_SUGGESTIONS);
   }
-
-  _breakBeforeOwners(codeOwners, showAllOwners) {
-    if (!codeOwners || !showAllOwners) return false;
-    return codeOwners.length > BestSuggestionsLimit;
-  }
 }
 
 customElements.define(SuggestOwners.is, SuggestOwners);