Merge ~andreserl/maas:lp1684131 into maas:master

Proposed by Andres Rodriguez
Status: Merged
Approved by: Andres Rodriguez
Approved revision: a1b320503042948f24c11f32ac8bb9d8de2a0d06
Merge reported by: MAAS Lander
Merged at revision: not available
Proposed branch: ~andreserl/maas:lp1684131
Merge into: maas:master
Diff against target: 25 lines (+9/-7)
1 file modified
src/maasserver/static/partials/ipranges.html (+9/-7)
Reviewer Review Type Date Requested Status
Newell Jensen (community) Approve
Review via email: mp+326972@code.launchpad.net

Commit message

LP: #1684131 Fix lack of spacing between 'Reserve range' and 'Reserve dynamic range' buttons

To post a comment you must log in.
Revision history for this message
Newell Jensen (newell-jensen) wrote :

LGTM

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/ipranges.html b/src/maasserver/static/partials/ipranges.html
2index f1ac580..2de32ef 100755
3--- a/src/maasserver/static/partials/ipranges.html
4+++ b/src/maasserver/static/partials/ipranges.html
5@@ -121,11 +121,13 @@
6 </div>
7 </main>
8 </div>
9- <button class="button--secondary button--inline"
10- data-ng-disabled="newRange"
11- data-ng-click="addRange('reserved')">Reserve range</a>
12- <button class="button--secondary button--inline"
13- data-ng-if="isSuperUser()"
14- data-ng-disabled="newRange"
15- data-ng-click="addRange('dynamic')">Reserve dynamic range</a>
16+ <div>
17+ <button class="button--secondary button--inline"
18+ data-ng-disabled="newRange"
19+ data-ng-click="addRange('reserved')">Reserve range</button>
20+ <button class="button--secondary button--inline"
21+ data-ng-if="isSuperUser()"
22+ data-ng-disabled="newRange"
23+ data-ng-click="addRange('dynamic')">Reserve dynamic range</button>
24+ </div>
25 </div>

Subscribers

People subscribed via source and target branches