Merge ~deadlight/maas:network-discovery-issues into ~ya-bo-ng/maas:vanilla

Proposed by Karl Williams
Status: Merged
Merged at revision: a3c3b322b568d04182e520ea788b325ddc761dbe
Proposed branch: ~deadlight/maas:network-discovery-issues
Merge into: ~ya-bo-ng/maas:vanilla
Diff against target: 63 lines (+9/-9)
2 files modified
src/maasserver/static/partials/dashboard.html (+4/-4)
src/maasserver/static/partials/nodes-list.html (+5/-5)
Reviewer Review Type Date Requested Status
Anthony Dillon Approve
Review via email: mp+337780@code.launchpad.net

Description of the change

Fixed expanding table issues on the dashboard

QA:
 - Go to the dashboard and click "open" on the rows and check that the form displays correctly.

To post a comment you must log in.
Revision history for this message
Anthony Dillon (ya-bo-ng) wrote :

Thanks +1

review: Approve
Revision history for this message
Karl Williams (deadlight) wrote :

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/src/maasserver/static/partials/dashboard.html b/src/maasserver/static/partials/dashboard.html
2index 9423c74..fbdeb51 100644
3--- a/src/maasserver/static/partials/dashboard.html
4+++ b/src/maasserver/static/partials/dashboard.html
5@@ -89,7 +89,7 @@
6 </button>
7 </div>
8 </td>
9- <td class="p-table-expanding__panel">
10+ <td class="p-table-expanding__panel col-12" data-ng-if="discovery.first_seen === selectedDevice">
11 <maas-obj-form data-ng-if="discovery.first_seen === selectedDevice"
12 obj="convertTo" manager="proxyManager" pre-process="preProcess"
13 after-save="afterSave" table-form="true" save-on-blur="false">
14@@ -143,9 +143,9 @@
15 disable-label="true" placeholder="Hostname (optional)"></maas-obj-field>
16 </div>
17 </div>
18- <div>
19- <label class="col-2">Type</label>
20- <div class="col-3">
21+ <div class="p-form__group u-clearfix">
22+ <label class="p-form__label col-2">Type</label>
23+ <div class="p-form__control col-3">
24 <select name="type" data-ng-model="convertTo.type">
25 <option value="" disabled>Choose type</option>
26 <option value="device">Device</option>
27diff --git a/src/maasserver/static/partials/nodes-list.html b/src/maasserver/static/partials/nodes-list.html
28index feeffa8..1e69db1 100644
29--- a/src/maasserver/static/partials/nodes-list.html
30+++ b/src/maasserver/static/partials/nodes-list.html
31@@ -876,7 +876,7 @@ sudo maas-rack register --url {$ tabs.controllers.registerUrl $} --secret {$ tab
32 data-ng-model="tabs.nodes.search" data-ng-change="updateFilters('nodes')"
33 data-ng-class="{ error: !tabs.nodes.searchValid }"
34 data-ng-disabled="tabs.nodes.actionOption" />
35- <button type="reset" class="p-search-box__reset" alt="reset"
36+ <button type="reset" class="p-search-box__reset" alt="reset"
37 data-ng-click="clearSearch('nodes')"
38 data-ng-show="tabs.nodes.search.length > 0"><i class="p-icon--close"></i></button>
39 <button type="submit" class="p-search-box__button" alt="search"><i class="p-icon--search"></i></button>
40@@ -893,9 +893,9 @@ sudo maas-rack register --url {$ tabs.controllers.registerUrl $} --secret {$ tab
41 <div class="p-search-box">
42 <!-- XXX rvba 2015-02-25 - Need to add e2e test. -->
43 <input
44- type="search" placeholder="Search devices"
45+ type="search" placeholder="Search devices"
46 class="p-search-boxh__input p-form-validation__input"
47- data-ng-model="tabs.devices.search"
48+ data-ng-model="tabs.devices.search"
49 data-ng-change="updateFilters('devices')"
50 data-ng-disabled="tabs.devices.actionOption" />
51 <button type="reset" class="p-search-box__reset" alt="reset"
52@@ -1055,9 +1055,9 @@ sudo maas-rack register --url {$ tabs.controllers.registerUrl $} --secret {$ tab
53 data-ng-class="{ 'is-error': !tabs.switches.searchValid }">
54 <!-- XXX rvba 2015-02-25 - Need to add e2e test. -->
55 <input
56- type="search" placeholder="Search switches"
57+ type="search" placeholder="Search switches"
58 class="p-search-box__input p-form-validation__input"
59- data-ng-model="tabs.switches.search"
60+ data-ng-model="tabs.switches.search"
61 data-ng-change="updateFilters('switches')"
62 data-ng-disabled="tabs.switches.actionOption" />
63 <button type="reset" class="p-search-box__reset" alt="reset"

Subscribers

People subscribed via source and target branches

to all changes: