From 597bd472eaef9d07cfb5092a7cc3d4684fec75a9 Mon Sep 17 00:00:00 2001
From: Jens Langhammer <jens.langhammer@beryju.org>
Date: Sun, 4 Apr 2021 19:03:25 +0200
Subject: [PATCH] web/admin: fix inconsistent button levels for events lists

Signed-off-by: Jens Langhammer <jens.langhammer@beryju.org>
---
 web/src/pages/events/RuleListPage.ts                       | 5 +++--
 web/src/pages/events/TransportListPage.ts                  | 2 +-
 web/src/pages/flows/FlowViewPage.ts                        | 2 +-
 web/src/pages/property-mappings/PropertyMappingListPage.ts | 3 ++-
 web/src/pages/users/UserListPage.ts                        | 2 +-
 5 files changed, 8 insertions(+), 6 deletions(-)

diff --git a/web/src/pages/events/RuleListPage.ts b/web/src/pages/events/RuleListPage.ts
index 4c11bef8e..4a2741de8 100644
--- a/web/src/pages/events/RuleListPage.ts
+++ b/web/src/pages/events/RuleListPage.ts
@@ -66,7 +66,7 @@ export class RuleListPage extends TablePage<NotificationRule> {
                 </span>
                 <ak-event-rule-form slot="form" .rule=${item}>
                 </ak-event-rule-form>
-                <button slot="trigger" class="pf-c-button pf-m-primary">
+                <button slot="trigger" class="pf-c-button pf-m-secondary">
                     ${t`Edit`}
                 </button>
             </ak-forms-modal>
@@ -108,7 +108,8 @@ export class RuleListPage extends TablePage<NotificationRule> {
         return html`
         <td role="cell" colspan="4">
             <div class="pf-c-table__expandable-row-content">
-                <p>${t`These policies control upon which events this rule triggers.`}</p>
+                <p>${t`These policies control upon which events this rule triggers. Bindings to
+                groups/users are checked against the user of the event.`}</p>
                 <ak-bound-policies-list .target=${item.pk}>
                 </ak-bound-policies-list>
             </div>
diff --git a/web/src/pages/events/TransportListPage.ts b/web/src/pages/events/TransportListPage.ts
index 76870fc8e..3bb67d29b 100644
--- a/web/src/pages/events/TransportListPage.ts
+++ b/web/src/pages/events/TransportListPage.ts
@@ -70,7 +70,7 @@ export class TransportListPage extends TablePage<NotificationTransport> {
                 </span>
                 <ak-event-transport-form slot="form" .transport=${item}>
                 </ak-event-transport-form>
-                <button slot="trigger" class="pf-c-button pf-m-primary">
+                <button slot="trigger" class="pf-c-button pf-m-secondary">
                     ${t`Edit`}
                 </button>
             </ak-forms-modal>
diff --git a/web/src/pages/flows/FlowViewPage.ts b/web/src/pages/flows/FlowViewPage.ts
index f0e87c94a..6680c15be 100644
--- a/web/src/pages/flows/FlowViewPage.ts
+++ b/web/src/pages/flows/FlowViewPage.ts
@@ -61,7 +61,7 @@ export class FlowViewPage extends LitElement {
             <ak-tabs>
                 <div slot="page-1" data-tab-title="${t`Flow Overview`}" class="pf-c-page__main-section pf-m-no-padding-mobile">
                     <div class="pf-l-gallery pf-m-gutter">
-                        <div class="pf-c-card pf-l-gallery__item" style="grid-column-end: span 3;grid-row-end: span 2;">
+                        <div class="pf-c-card pf-l-gallery__item" style="grid-column-end: span 4;grid-row-end: span 2;">
                             <div class="pf-c-card">
                                 <div class="pf-c-card__body">
                                     <ak-flow-diagram flowSlug=${this.flow.slug}>
diff --git a/web/src/pages/property-mappings/PropertyMappingListPage.ts b/web/src/pages/property-mappings/PropertyMappingListPage.ts
index 7e9c58de6..34dacc49d 100644
--- a/web/src/pages/property-mappings/PropertyMappingListPage.ts
+++ b/web/src/pages/property-mappings/PropertyMappingListPage.ts
@@ -145,7 +145,8 @@ export class PropertyMappingListPage extends TablePage<PropertyMapping> {
     }
 
     renderToolbarAfter(): TemplateResult {
-        return html`<div class="pf-c-toolbar__group pf-m-filter-group">
+        return html`&nbsp;
+        <div class="pf-c-toolbar__group pf-m-filter-group">
             <div class="pf-c-toolbar__item pf-m-search-filter">
                 <div class="pf-c-input-group">
                     <div class="pf-c-check">
diff --git a/web/src/pages/users/UserListPage.ts b/web/src/pages/users/UserListPage.ts
index e0928701d..a3cca4705 100644
--- a/web/src/pages/users/UserListPage.ts
+++ b/web/src/pages/users/UserListPage.ts
@@ -75,7 +75,7 @@ export class UserListPage extends TablePage<User> {
                 </button>
             </ak-forms-modal>
             <ak-dropdown class="pf-c-dropdown">
-                <button class="pf-c-dropdown__toggle pf-m-primary" type="button">
+                <button class="pf-c-dropdown__toggle pf-m-secondary" type="button">
                     <span class="pf-c-dropdown__toggle-text">${item.isActive ? t`Disable` : t`Enable`}</span>
                     <i class="fas fa-caret-down pf-c-dropdown__toggle-icon" aria-hidden="true"></i>
                 </button>