MAAS

Merge lp:~ricgard/maas/table-owner-hover into lp:~maas-committers/maas/trunk

  1. table-owner-hover
  2. Merge into trunk
Proposed by Richard McCartney
Status: Merged
Approved by: Blake Rouse
Approved revision: no longer in the source branch.
Merged at revision: 3885
Proposed branch: lp:~ricgard/maas/table-owner-hover
Merge into: lp:~maas-committers/maas/trunk
Diff against target: 12 lines (+1/-1)
1 file modified
src/maasserver/static/partials/nodes-list.html (+1/-1)
To merge this branch: bzr merge lp:~ricgard/maas/table-owner-hover
Reviewer Review Type Date Requested Status
Blake Rouse (community) Approve
Review via email: mp+258977@code.launchpad.net

Commit message

Fixes owner column missing hover state on the node listing table.

Description of the change

Resolves markup issue on node listing. Re-adds missing table cell class to show the correct hover style on the listing table.

To post a comment you must log in.
Revision history for this message
Blake Rouse (blake-rouse) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/maasserver/static/partials/nodes-list.html'
2--- src/maasserver/static/partials/nodes-list.html 2015-05-12 13:06:15 +0000
3+++ src/maasserver/static/partials/nodes-list.html 2015-05-13 09:29:33 +0000
4@@ -554,7 +554,7 @@
5 <span title="{$ node.power_state $}" class="powerstates power-{$ node.power_state $}"></span>
6 </td>
7 <td class="table-listing__cell status">{$ node.status $}</td>
8- <td>{$ node.owner $}</td>
9+ <td class="table-listing__cell">{$ node.owner $}</td>
10 <td class="align-right table-listing__cell">{$ node.cpu_count $}</td>
11 <td class="align-right table-listing__cell">{$ node.memory $}</td>
12 <td class="align-right table-listing__cell">{$ node.disks $}</td>