Project

General

Profile

Feature #32175

New UI jails management menu is lacking icons / restart and confirm

Added by Disk Didler over 3 years ago. Updated over 3 years ago.

Status:
Done
Priority:
Nice to have
Assignee:
Dennis Mullen
Category:
GUI (new)
Estimated time:
Severity:
Low
Reason for Closing:
Reason for Blocked:
Needs QA:
Yes
Needs Doc:
Yes
Needs Merging:
No
Needs Automation:
No
Support Suite Ticket:
n/a
Hardware Configuration:

Description

I do not like material design which results in unlabelled colourless icons, awful move for the industry.
HOWEVER,..
You guys have gone the other way, text only icons in the jail management menu, for "stop | update | delete"
Be nice to use similar (same?) icons from v9/v11 - a green start or red stop button, a delete trashcan, that kind of thing.
Also colours don't go astray to make it easier to identify at a glance.

Furthermore, on the v9/v11 system if I hit stop on a jail, it then pops up a box, basically confirming this. Be nice to have one here.

Tested on Nightly, 16 Apr, 2018

Associated revisions

Revision 1cea909f (diff)
Added by Dennis Mullen over 3 years ago

Add color indicator to jail status Ticket: #32175

Revision 509a13fd (diff)
Added by Dennis Mullen over 3 years ago

Finish up color-coding items on jail menu Ticket: #32175

Revision 79ad950f (diff)
Added by Dennis Mullen over 3 years ago

Flatten buttons, go with uncolored icons with dynamic start/stop Ticket: #32175

Revision 1109c262 (diff)
Added by Dennis Mullen over 3 years ago

Add confirm dialog to Stop button and menu item Ticket: #32175

Revision b583e537 (diff)
Added by Dennis Mullen over 3 years ago

Remove button labels, replace with tooltips Ticket: #32175

History

#1 Updated by Dru Lavigne over 3 years ago

  • Assignee changed from Release Council to Anonymous
  • Target version set to 11.2-BETA1

#2 Updated by Anonymous over 3 years ago

  • Assignee changed from Anonymous to Dennis Mullen

#3 Updated by Dennis Mullen over 3 years ago

  • Status changed from Unscreened to Screened

#4 Updated by Dennis Mullen over 3 years ago

  • Status changed from Screened to In Progress

#5 Updated by Disk Didler over 3 years ago

Is this available to test in nightly? I'm not seeing it.

Thanks

#6 Updated by Dennis Mullen over 3 years ago

Disk Didler wrote:

Is this available to test in nightly? I'm not seeing it.

Thanks

It's not in the nightly. I submitted a branch with some icons on the tabs, including a color change for start/stop but I don't think it will make it it into the build just yet, as the UI team is thinking thru overall design, and will want to answer other design questions before they can see if this fits.

The branch is: feature/jails-list-32175

#7 Updated by Dennis Mullen over 3 years ago

  • Severity set to Low

#8 Updated by Disk Didler over 3 years ago

What about the 'confirm to stop jail' notification? Did you drop that one in too?

Thanks.

#9 Updated by Dennis Mullen over 3 years ago

Disk Didler wrote:

What about the 'confirm to stop jail' notification? Did you drop that one in too?

Thanks.

Added the confirmation to jail stop this morning. This may go into the nightlies soon. If so, I will try to post it here.

#10 Updated by Dennis Mullen over 3 years ago

PR #704

#11 Updated by Dennis Mullen over 3 years ago

  • Status changed from In Progress to Done

#12 Updated by Dru Lavigne over 3 years ago

  • Target version changed from 11.2-BETA1 to Master - FreeNAS Nightlies
  • Needs Merging changed from Yes to No

Also available in: Atom PDF