Bug 814731

Summary: [RFE] Replace Toolbar Buttons with Menu Structure
Product: Red Hat Enterprise Linux 5 Reporter: Matt Reid <mreid>
Component: subscription-managerAssignee: Bryan Kearney <bkearney>
Status: CLOSED ERRATA QA Contact: Entitlement Bugs <entitlement-bugs>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 5.9CC: bgollahe, bkearney, fsharath, jmolet, jsefler
Target Milestone: betaKeywords: FutureFeature
Target Release: 5.9   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Enhancement
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2013-01-08 03:51:50 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 771748, 827225    
Attachments:
Description Flags
System drop down menu none

Description Matt Reid 2012-04-20 14:48:42 UTC 
Description of problem:
Feedback we've gotten from users indicates that they're not too thrilled with the toolbar buttons and their icons. I think having a menu makes sense, after all, this is a desktop application, and it would remove the ambiguity created by the buttons. 

I propose having 2 Menus, System and Help.

System:
Un/Register
Import Cert
Redeem Subscription
View System Facts
Configure Proxy
Configure Settings
Quit

Help:
About
Getting Started
(Maybe a direct link to online documentation?)

Wireframes can be seen at -
https://engineering.redhat.com/trac/subscription-manager/attachment/wiki/GUIRedesign/Menu.png

[1] - 
Shows new menus in subscription mananger. Toolbar icons removed, register button functionality absorbed into the button within My Installed Software, now acting as the one point of reference for them to get a system up and running. 
We can increase the visibility of messaging about what they must do, ie coloring the message saying they need to register. This messaging is now more to the point for the user, framed around actions, something like "You must register this system before subscribing."

When they click Register, it will follow the SLA flow we implemented in 6.3, with an additional screen. The new addition to the Register flow would allow users to choose what to register with, be it our servers, an on-premise solution, or by importing an existing cert.

As the system is not registered yet, we do not show any org or sla preference info.

[2] -
If the system is registered, but not consuming any entitlements, we can show the Org that its registered with, and potentially show the SLA preference. The button that said Register in the previous example is now a Subscribe button, and the message has been changed to "You have 1 product not covered with a subscription"

Clicking auto-subscribe will take them through our existing SLA flow.

[3] -
When the system is both registered and subscribed, then we can show the SLA Preference (if applicable), and the org registered to. Cert Status says that everything is good until the nearest end date.

[A] - 
Selecting Register now shows this screen before jumping into our current SLA flow, so we'd need to build out a bit more functionality, to ensure that people can register to a SAM instance, or import a local cert.

Additional info:
As subscription-manager gets more like a real desktop app, we can consider adding in right click functionality for subscribing, which a few people asked for at the focus group.

We may want to consider supporting activation keys from the GUI, instead of just with the CLI.
Comment 1 James Bowes 2012-05-24 17:05:48 UTC 
Mike's got this done
Comment 2 Matt Reid 2012-05-30 19:46:12 UTC 
Can I be a pain and recommend that we change Settings to Preferences in our beautiful, shiny new menu? We could keep the menu shortcut the same, P is already taken by Proxy, and there's an S in Preferences.

The Service Level and Release Version variables feel more like preferences than settings, and the window title is System Preferences. 

Also would keep it consistent with other GNOME apps, which have Preferences, not Settings.
Comment 3 J.C. Molet 2012-06-21 15:28:51 UTC 
Tossing back to dev for comment 2.
Comment 4 Bryan Kearney 2012-07-19 19:35:46 UTC 
Making the menu item Preferences, and made Configure Proxy have no accelerator keys.
Comment 5 John Sefler 2012-07-21 13:22:37 UTC 
(In reply to comment #4)
> Making the menu item Preferences, and made Configure Proxy have no
> accelerator keys.

Don't remove the accelerator for "Configure Proxy" making it inaccessible.
Make the accelerator for the new "Preference" menu item Ctrl+P   (not S)
Make the accelerator for the existing "Configure Proxy" menu item Ctrl+C or Ctrl+X

Tossing back to dev again.
Comment 6 Matt Reid 2012-07-23 18:10:08 UTC 
(In reply to comment #5)
> (In reply to comment #4)
> > Making the menu item Preferences, and made Configure Proxy have no
> > accelerator keys.
> 
> Don't remove the accelerator for "Configure Proxy" making it inaccessible.
> Make the accelerator for the new "Preference" menu item Ctrl+P   (not S)
> Make the accelerator for the existing "Configure Proxy" menu item Ctrl+C or
> Ctrl+X
> 
> Tossing back to dev again.

My preference would be to keep accelerators on all menu items. Ctrl+C gets my vote, as X would be our only accelerator that isn't the first letter in one of the words.
Comment 7 Bryan Kearney 2012-07-23 18:14:09 UTC 
i would NACK ctrl-c as that has other contations. Keep in mind, evern if htey do not have accelerators, you can still get to them with the arrow key pads. Note in other products that not every menu item has accelerators.
Comment 8 Bryan Kearney 2012-07-23 19:38:05 UTC 
Adrian commited a version with Ctrl-X.
Comment 10 Sharath Dwaral 2012-07-30 18:16:25 UTC 
Version:

# rpm -qa | egrep "subscription-manager|python-rhsm"
subscription-manager-gui-1.0.11-1.git.3.b3c4a8b.el5
subscription-manager-1.0.11-1.git.3.b3c4a8b.el5
subscription-manager-migration-data-1.11.2.2-1.git.2.c3c8e22.el5
python-rhsm-1.0.4-1.git.6.19a55da.el5
subscription-manager-firstboot-1.0.11-1.git.3.b3c4a8b.el5
subscription-manager-migration-1.0.11-1.git.3.b3c4a8b.el5

Verfication:

# subscription-manager-gui

Click on "System" when the GUI pops up The menu has 

1) Register                Ctrl+R
2) Import Cert             Ctrl+I
3) View system facts       Ctrl+F 
4) Configure Proxy         Ctrl+X
5) Preference              Ctrl+P  
6) Quit                    Ctrl+Q

(Could not get a screen-shot because when the the system menu is opened, the control remains within the window and I was not able to use an application/print-screen to get a screen-shot)

Moving bug to VERIFIED
Comment 11 Sharath Dwaral 2012-07-30 21:11:17 UTC 
Created attachment 601342 [details]
System drop down menu

Screen-shot of the above verified menu
Comment 13 errata-xmlrpc 2013-01-08 03:51:50 UTC 
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

http://rhn.redhat.com/errata/RHBA-2013-0033.html