AMT power type config in the UI contains a "Power MAC" instead of "Power address" field

Bug #1436290 reported by Raphaël Badin
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
MAAS
Fix Released
High
Ricardo Bánffy

Bug Description

Additionally I noticed that when you hover over the various field names on the add node page, you get a weird hand cursor instead of the default one.

Tags: ui

Related branches

Changed in maas:
milestone: none → next
Revision history for this message
Andres Rodriguez (andreserl) wrote :

shouldn't AMT contain both, Power MAC and Power Address? Power Mac is the optoinal

Changed in maas:
assignee: nobody → Ricardo Bánffy (rbanffy)
Changed in maas:
status: Triaged → Fix Committed
Revision history for this message
Carla Berkers (carlaberkers) wrote :

We have a MAC address and an IP address, so I think using the term "Power address" is ambiguous.

Can we use "Power MAC" and "Power IP" instead to make sure it is clear what input is required from the user?

Also I see some capitalisation inconsistancies slipping in, can we make sure not to use title case (example in trunk, please use "MAC address" instead of "MAC Address")

Revision history for this message
Ricardo Bánffy (rbanffy) wrote :

There is no big issue with changing the labels. The only thing I would recommend is to remain consistent across all power controller types. Do you need them changed?

Revision history for this message
Carla Berkers (carlaberkers) wrote :

Agreed, labeling should be consistent across power controller types. If its not to much effort and you can change them as part of this bug you will have made one ux designer very happy :-)

If you can't do it as part of this bug let me know, I'll file a separate bug for it. Thanks! C

Revision history for this message
Blake Rouse (blake-rouse) wrote :

Carla,

This bug has already been fixed. Please file it as a separate bug.

Revision history for this message
Carla Berkers (carlaberkers) wrote :

added a new bug: https://bugs.launchpad.net/maas/+bug/1438885.

Thanks all!

Changed in maas:
status: Fix Committed → Fix Released
milestone: next → none
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.