From f7d10b417e93a650bc459bb986fe35e8a6354e5f Mon Sep 17 00:00:00 2001
From: Emanuel Schuetze <emanuel@intevation.de>
Date: Fri, 7 Oct 2016 11:42:33 +0200
Subject: [PATCH] Template fixes

- motion list: show category and tags dropdown for manager only
- manage projector: fix submenu buttons and css classes
---
 openslides/core/static/css/app.css            |  9 ----
 .../templates/core/manage-projectors.html     | 41 ++++++++++---------
 .../static/templates/motions/motion-list.html | 34 +++++++++++----
 3 files changed, 46 insertions(+), 38 deletions(-)

diff --git a/openslides/core/static/css/app.css b/openslides/core/static/css/app.css
index 1d2a61d9f..02ce19333 100644
--- a/openslides/core/static/css/app.css
+++ b/openslides/core/static/css/app.css
@@ -270,15 +270,6 @@ img {
     float: right;
 }
 
-.col1 .header .submenu > div {
-    display: inline-block;
-    float: left;
-    margin-left: 5px;
-}
-.col1 .header .submenu > div.button {
-    float: none;
-    margin-left: 0;
-}
 .col1 .meta .title {
     width: 100%;
     cursor: pointer;
diff --git a/openslides/core/static/templates/core/manage-projectors.html b/openslides/core/static/templates/core/manage-projectors.html
index c2a3d76ef..a5e6c0b59 100644
--- a/openslides/core/static/templates/core/manage-projectors.html
+++ b/openslides/core/static/templates/core/manage-projectors.html
@@ -1,34 +1,35 @@
 <div class="header">
   <div class="title">
     <div class="submenu">
-      <div>
-        <button class="btn btn-primary" ng-bootbox-prompt="{{ 'Please enter a name for the new projector' | translate }}"
-          ng-bootbox-prompt-action="createProjector(result)">
-          <i class="fa fa-plus"></i>
-          <translate>New</translate>
-        </button>
-      </div>
-      <div class="dropdown">
-        <button class="btn btn-default dropdown-toggle" id="menuListofSpeakers" data-toggle="dropdown"
+      <!-- New -->
+      <button class="btn btn-primary" ng-bootbox-prompt="{{ 'Please enter a name for the new projector' | translate }}"
+        ng-bootbox-prompt-action="createProjector(result)">
+        <i class="fa fa-plus"></i>
+        <translate>New</translate>
+      </button>
+      <!-- Reference for current list of speakers -->
+      <div class="btn-group" uib-dropdown>
+        <button class="btn btn-default" id="menuListofSpeakers" uib-dropdown-toggle
             uib-tooltip="{{ 'Select the projector to which the current list of speakers refers to.' | translate }}"
             aria-haspopup="true" aria-expanded="true">
+          <i class="fa fa-microphone"></i>
           <translate>Reference for current list of speakers</translate>
           <span class="caret"></span>
         </button>
-        <ul class="dropdown-menu dropdown-entries" aria-labelledby="menuListOfSpeakers">
-          <li ng-repeat="projector in projectors"
-            ng-click="setListOfSpeakers(projector)">
-            <i class="fa fa-check" ng-if="projector.id == currentListOfSpeakers"></i>
-            {{ projector.name }}
+        <ul class="dropdown-menu" uib-dropdown-menu aria-labelledby="menuListOfSpeakers">
+          <li ng-repeat="projector in projectors">
+            <a href ng-click="setListOfSpeakers(projector)">
+              <i class="fa fa-check" ng-if="projector.id == currentListOfSpeakers"></i>
+              {{ projector.name }}
+            </a>
           </li>
         </ul>
       </div>
-      <div>
-        <button class="btn" ng-click="identifyProjectors()" ng-class="identifyPromise ? 'btn-primary' : 'btn-default'">
-          <i class="fa fa-binoculars"></i>
-          <translate>Identify</translate>
-        </button>
-      </div>
+      <!-- Identity -->
+      <button class="btn" ng-click="identifyProjectors()" ng-class="identifyPromise ? 'btn-primary' : 'btn-default'">
+        <i class="fa fa-binoculars"></i>
+        <translate>Identify</translate>
+      </button>
     </div>
     <h1 translate>Manage projectors</h1>
   </div>
diff --git a/openslides/motions/static/templates/motions/motion-list.html b/openslides/motions/static/templates/motions/motion-list.html
index 02b69adf4..ed4e64e12 100644
--- a/openslides/motions/static/templates/motions/motion-list.html
+++ b/openslides/motions/static/templates/motions/motion-list.html
@@ -324,7 +324,7 @@
               <span class="label" ng-class="'label-'+motion.state.css_class">
                 {{ motion.state.name | translate }}
               </span>
-              <span ng-class="{'hiddenDiv': !motion.stateHover}" uib-dropdown>
+              <span os-perms="motions.can_manage" ng-class="{'hiddenDiv': !motion.stateHover}" uib-dropdown>
                 <i class="fa fa-cog pointer" uib-dropdown-toggle id="state-dropdown{{ motion.id }}"></i>
                 <ul uib-dropdown-menu aria-labelledby="state-dropdown{{ motion.id }}">
                   <li ng-repeat="state in motion.state.getNextStates()">
@@ -376,11 +376,11 @@
       <div class="col-xs-4 content">
         <div style="width: 60%;" class="optional">
           <small>
-            <div ng-mouseover="motion.categoryHover=true"
-              ng-mouseleave="motion.categoryHover=false"
-              ng-show="categories.length > 0">
-              <!-- Category dropdown -->
-              <span uib-dropdown>
+            <!-- Category dropdown for manage user -->
+            <div os-perms="motions.can_manage" ng-show="categories.length > 0"
+                ng-mouseover="motion.categoryHover=true"
+                ng-mouseleave="motion.categoryHover=false">
+              <span uib-dropdown >
                 <span id="dropdown-category{{ motion.id }}" class="pointer"
                   uib-dropdown-toggle uib-tooltip="{{ 'Set a category' | translate }}"
                   tooltip-class="nobr">
@@ -405,9 +405,16 @@
                 </ul>
               </span>
             </div>
-            <div ng-mouseover="motion.tagHover=true"
-              ng-mouseleave="motion.tagHover=false"
-              ng-show="tags.length > 0">
+            <!-- Category string for normal user -->
+            <div os-perms="!motions.can_manage" ng-show="motion.category != null">
+              <i class="fa fa-sitemap spacer-right"></i>
+              {{ motion.category.name }}
+            </div>
+
+            <!-- Tag dropdown for manage user -->
+            <div os-perms="motions.can_manage" ng-show="tags.length > 0"
+                ng-mouseover="motion.tagHover=true"
+                ng-mouseleave="motion.tagHover=false">
               <span uib-dropdown>
                 <span id="dropdown-tags{{ motion.id }}" class="pointer"
                   uib-dropdown-toggle uib-tooltip="{{ 'Add a tag' | translate }}"
@@ -434,6 +441,15 @@
                 </ul>
               </span>
             </div>
+            <!-- Tag string for normal user -->
+            <div os-perms="!motions.can_manage" ng-show="motion.tags.length > 0">
+              <i class="fa fa-tags spacer-right"></i>
+              <span ng-repeat="tag in motion.tags">
+                {{ tag.name }}<span ng-if="!$last">,</span>
+              </span>
+            </div>
+
+            <!-- Origin -->
             <div ng-if="motion.origin">
               <i class="fa fa-share spacer-right" uib-tooltip="{{ 'Origin' | translate }}"></i>
                 {{ motion.origin | limitTo:25 }}{{ motion.origin.length > 25 ? '...' : '' }}