[2.4, UI] Storage firmware version is mispelled & should be part of Name | Model | Serial switcher

Bug #1757155 reported by Andres Rodriguez
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
MAAS
Fix Released
Medium
Lee Trager

Bug Description

The storage firmware version is mispelled, and reads 'fireware version'.

That said, to not occupy to much space, it should be part of the switcher 'Name | Model | Serial | Firmware'

Related branches

Revision history for this message
Andres Rodriguez (andreserl) wrote :
Changed in maas:
milestone: none → 2.4.0beta1
importance: Undecided → High
status: New → Triaged
assignee: nobody → Lee Trager (ltrager)
Changed in maas:
importance: High → Medium
Revision history for this message
Lee Trager (ltrager) wrote :

Thanks for finding the spelling mistake. Name, model, and serial are all identifiers for the storage device. The firmware version is information about the device. Adding firmware to the first column will also increase the size of the first column giving it alot of white space. What if I shorten it to just firmware?

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

I think just the word "Firmware" will be fine.

Revision history for this message
Andres Rodriguez (andreserl) wrote : Re: [Bug 1757155] Re: [2.4, UI] Storage firmware version is mispelled & should be part of Name | Model | Serial switcher

Adding firmware as part of the switches is less space than having its own
column for it.

On Fri, Mar 23, 2018 at 9:41 AM Blake Rouse <email address hidden>
wrote:

> I think just the word "Firmware" will be fine.
>
> --
> You received this bug notification because you are subscribed to the bug
> report.
> https://bugs.launchpad.net/bugs/1757155
>
> Title:
> [2.4, UI] Storage firmware version is mispelled & should be part of
> Name | Model | Serial switcher
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/maas/+bug/1757155/+subscriptions
>
> Launchpad-Notification-Type: bug
> Launchpad-Bug: product=maas; milestone=2.4.0beta1; status=Triaged;
> importance=Medium; <email address hidden>;
> Launchpad-Bug-Information-Type: Public
> Launchpad-Bug-Private: no
> Launchpad-Bug-Security-Vulnerability: no
> Launchpad-Bug-Commenters: andreserl blake-rouse ltrager
> Launchpad-Bug-Reporter: Andres Rodriguez (andreserl)
> Launchpad-Bug-Modifier: Blake Rouse (blake-rouse)
> Launchpad-Message-Rationale: Subscriber
> Launchpad-Message-For: andreserl
>
--
Andres Rodriguez (RoAkSoAx)
Ubuntu Server Developer
MSc. Telecom & Networking
Systems Engineer

Changed in maas:
milestone: 2.4.0beta1 → 2.4.0beta2
Lee Trager (ltrager)
Changed in maas:
status: Triaged → In Progress
Revision history for this message
Lee Trager (ltrager) wrote :

Discussed on IRC, just fixing spelling for now. Much of the width is taken up by the title so adding it to the switch won't save that much space. The user also needs to be able to see which disk the firmware is related to. If firmware is in the switcher the user will be forces to toggle back and forth to get the name/model/serial and the firmware version.

Changed in maas:
status: In Progress → Fix Committed
Revision history for this message
Andres Rodriguez (andreserl) wrote :

I'm re-opening this. Please move the 'Firmware' column to be a switcher instead, because that colum makes the table too large.

Changed in maas:
status: Fix Committed → Confirmed
Revision history for this message
Andres Rodriguez (andreserl) wrote :

Makes the 'Action' column disappear.

Revision history for this message
Lee Trager (ltrager) wrote :

Putting 'Firmware' in the switcher causes the switcher to wrap even on high resolution screens. The scroll bar is only shown on lower resolutions(< 1440px width) which is why I didn't catch it initially. I'll bring this up in the SU with Karl.

Changed in maas:
status: Confirmed → In Progress
Changed in maas:
milestone: 2.4.0beta2 → 2.4.0rc1
Changed in maas:
status: In Progress → Fix Committed
Changed in maas:
milestone: 2.4.0rc1 → 2.4.0beta3
Changed in maas:
status: Fix Committed → Fix Released
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.