Merge "Prepare for rule (only) editing"

polygerrit-ui/app/elements/admin/gr-access-section/gr-access-section.html

@@ -61,12 +61,13 @@ limitations under the License.
       .editContainer {
         display: none;
       }
-      .deleted #deletedContainer,
+      /* TODO @beckysiegel add back when editing allowed */
+      /* .deleted #deletedContainer,
       #mainContainer,
       .editing #addPermission,
       .editing #updateBtns  {
         display: block;
-      }
+      } */
       .editingRef .editContainer {
         display: flex;
       }
@@ -115,7 +116,7 @@ limitations under the License.
           </template>
           <div id="addPermission">
             Add permission:
-            <select id="permissionSelect">
+            <select id="permissionSelect"> -->
               <!-- called with a third parameter so that permissions update
                   after a new section is added. -->
               <template
@@ -125,7 +126,8 @@ limitations under the License.
               </template>
             </select>
             <gr-button id="addBtn" on-tap="_handleAddPermission">Add</gr-button>
-          </div><!-- end addPermission -->
+          </div>
+          <!-- end addPermission -->
         </div><!-- end sectionContent -->
       </div><!-- end mainContainer -->
       <div id="deletedContainer">

polygerrit-ui/app/elements/admin/gr-permission/gr-permission.html

@@ -46,9 +46,10 @@ limitations under the License.
         border: 1px solid #d1d2d3;
         border-bottom: 0;
       }
-      .editing .rules {
+      /* TODO @beckysiegel add back */
+      /* .editing .rules {
         border-bottom: 1px solid #d1d2d3;
-      }
+      } */
       .title {
         margin-bottom: .3em;
       }
@@ -56,13 +57,14 @@ limitations under the License.
       #removeBtn {
         display: none;
       }
-      .editing #removeBtn {
+      /* TODO @beckysiegel add back */
+      /* .editing #removeBtn {
         display: block;
       }
       .editing #addRule {
         display: block;
         padding: .7em;
-      }
+      } */
       #deletedContainer,
       .deleted #mainContainer {
         display: none;
@@ -105,7 +107,8 @@ limitations under the License.
                 placeholder="Add group"
                 on-commit="_handleAddRuleItem">
             </gr-autocomplete>
-          </div> <!-- end addRule -->
+          </div>
+          <!-- end addRule -->
         </div> <!-- end rules -->
       </div><!-- end mainContainer -->
       <div id="deletedContainer">

polygerrit-ui/app/elements/admin/gr-rule-editor/gr-rule-editor.html

@@ -35,9 +35,10 @@ limitations under the License.
       .buttons {
         display: none;
       }
-      .editing .buttons {
+      /* TODO @beckysiegel add back */
+      /* .editing .buttons {
         display: flex;
-      }
+      } */
       #options {
         align-items: baseline;
         display: flex;