Merge ~andreserl/maas:lp1773387_lp1775099_2.4 into maas:2.4

Proposed by Andres Rodriguez
Status: Merged
Approved by: Andres Rodriguez
Approved revision: 3669c96d481310a840e76b551f8392d3d866018e
Merge reported by: MAAS Lander
Merged at revision: not available
Proposed branch: ~andreserl/maas:lp1773387_lp1775099_2.4
Merge into: maas:2.4
Diff against target: 79 lines (+26/-26)
2 files modified
src/maasserver/static/partials/pod-details.html (+25/-25)
src/maasserver/templates/maasserver/index.html (+1/-1)
Reviewer Review Type Date Requested Status
Andres Rodriguez (community) Approve
Review via email: mp+347530@code.launchpad.net

Commit message

Backport 9bdbbc8 and b2f0e80 - LP: #1773387 and LP: #1775099.

LP: #1773387 - Improve styling of Pod Configuration form.
LP: #1775099 - Apply escapejs template filter to site_name in data-ng-init.

To post a comment you must log in.
Revision history for this message
Andres Rodriguez (andreserl) wrote :

selfie!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/src/maasserver/static/partials/pod-details.html b/src/maasserver/static/partials/pod-details.html
2index a6a7167..4e07590 100644
3--- a/src/maasserver/static/partials/pod-details.html
4+++ b/src/maasserver/static/partials/pod-details.html
5@@ -212,36 +212,36 @@
6 </div>
7 </section>
8 <section class="p-strip" data-ng-if="pod && section.area === 'configuration'">
9- <div class="row">
10- <div class="col-10">
11- <h2>Pod configuration</h2>
12- </div>
13- <div class="col-2">
14- <button class="p-button--neutral u-float--right"
15- data-ng-if="canEdit() && !editing"
16- data-ng-click="editPodConfiguration()">Edit</button>
17+ <div class="row">
18+ <div class="col-10">
19+ <h2>Pod configuration</h2>
20+ </div>
21+ <div class="col-2">
22+ <button class="p-button--neutral u-float--right"
23+ data-ng-if="canEdit() && !editing"
24+ data-ng-click="editPodConfiguration()">Edit</button>
25+ </div>
26 </div>
27- </div>
28- <div class="row">
29+ <div class="row">
30 <maas-obj-form obj="pod" manager="podManager" manager-method="updateItem"
31 table-form="true" save-on-blur="false" after-save="exitEditPodConfiguration" data-ng-disabled="!editing">
32 <div class="col-6">
33- <div class="col-6">
34- <dl>
35- <dt>Type</dt>
36- <dd>{$ getPodTypeTitle() $}</dd>
37- </dl>
38- <maas-obj-field type="options" key="zone" label="Zone"
39- subtle="false" placeholder="Choose a zone"
40- options="zone.id as zone.name for zone in zones"></maas-obj-field>
41- <maas-obj-field type="options" key="default_pool"
42- label="Resource pool" subtle="false"
43- placeholder="Choose a pool"
44- options="pool.id as pool.name for pool in pools"></maas-obj-field>
45- <maas-obj-field type="tags" key="tags" label="Tags"
46- label-width="2" input-width="5"
47- disable-label="false" placeholder="Add a tag"></maas-obj-field>
48+ <div class="p-form__group">
49+ <label for="type" class="p-form__label col-2">Type</label>
50+ <div class="p-form__control u-no-margin--top col-5">
51+ <input type="text" id="type" disabled="disabled" value="{$ getPodTypeTitle() $}" />
52+ </div>
53 </div>
54+ <maas-obj-field type="options" key="zone" label="Zone"
55+ subtle="false" placeholder="Choose a zone"
56+ options="zone.id as zone.name for zone in zones"></maas-obj-field>
57+ <maas-obj-field type="options" key="default_pool"
58+ label="Resource pool" subtle="false"
59+ placeholder="Choose a pool"
60+ options="pool.id as pool.name for pool in pools"></maas-obj-field>
61+ <maas-obj-field type="tags" key="tags" label="Tags"
62+ label-width="2" input-width="5"
63+ disable-label="false" placeholder="Add a tag"></maas-obj-field>
64 </div>
65 <div class="col-6">
66 <maas-pod-parameters hide-type="true"></maas-pod-parameters>
67diff --git a/src/maasserver/templates/maasserver/index.html b/src/maasserver/templates/maasserver/index.html
68index f88d019..7c68d81 100755
69--- a/src/maasserver/templates/maasserver/index.html
70+++ b/src/maasserver/templates/maasserver/index.html
71@@ -18,7 +18,7 @@
72
73 {% include "maasserver/css-conf.html" %}
74 </head>
75-<body class="has-sticky-footer" data-ng-init="site = '{{ global_options.site_name }}'" data-maas-version-reloader window-width>
76+<body class="has-sticky-footer" data-ng-init="site = '{{ global_options.site_name|escapejs }}'" data-maas-version-reloader window-width>
77 <header id="navigation" class="p-navigation" data-dropdown-toggle="mobile-nav-toggle" data-dropdown-menu="mobile-nav-menu">
78 <div class="p-navigation__banner">
79 <div class="p-navigation__logo">

Subscribers

People subscribed via source and target branches