Bug 2499

Summary: [Hob2] The content of the Proxy tab in the Settings dialog needs review by copywriting
Product: [Build System, Metadata & Runtime] Hob Reporter: Belen Barros Pena <belen.barros.pena>
Component: hobAssignee: Jessica <jessica.zhang>
Status: VERIFIED FIXED QA Contact:
Severity: normal    
Priority: Medium CC: jessica.zhang, jiajun.xu, poky.bs.watcher, poky.watcher, shane.wang
Version: 1.3   
Target Milestone: 1.3   
Hardware: x86   
OS: Multiple   
Whiteboard: a new patch to bitbake-dev is out for review based on the latest text change request
OS type for building Yocto: --- Type of Regression: ---
Verified: Documentation change: ---
Attachments:
Description Flags
Proxy tab screenshots
none
Proxy tab screenshots
none
Of course I forgot one tooltip
none
use-same-proxy-for-all-protocols.png none

Description Belen Barros Pena 2012-05-25 09:32:28 UTC
Hi Scott,

When you have a chance, could you review the content of the Proxy tab in the Settings dialog? 

Thanks!

Belen
Comment 1 Shane Wang 2012-05-29 14:23:49 UTC
Give it to Scott for him to review the content of the proxy settings.
Comment 2 Scott Rifenbark 2012-05-29 21:24:31 UTC
Please send me a screen shot of the stuff that needs reviewed.  I am unable to build and run Hob on my Ubuntu 10.04 system due to fatal Gtk+, PyGtk and PyGobject issues.  Josh Lock tells me that the fix in master for Hob does not support 10.04.  I can't run Hob on my 10.04 system.

Scott
Comment 3 Belen Barros Pena 2012-05-30 08:06:05 UTC
Created attachment 553 [details]
Proxy tab screenshots

Attached.
Comment 4 Belen Barros Pena 2012-05-30 08:14:25 UTC
Created attachment 554 [details]
Proxy tab screenshots

New version of proxy.zip including all tooltips... I think. I hope I haven't forgotten anything.

Cheers

Belen
Comment 5 Belen Barros Pena 2012-05-30 08:18:46 UTC
Created attachment 555 [details]
Of course I forgot one tooltip

And hopefully the last tooltip of the proxy tab.
Comment 6 Scott Rifenbark 2012-05-30 14:44:23 UTC
I would say 'Set the proxies used when fetching source code' instead of 'Set the proxies that will be used during fetching source code.'  Everything else looks fine.

i am marking this resolved as the bug is to simply review the screen.

Scott
Comment 7 Belen Barros Pena 2012-05-30 14:51:06 UTC
Hi Scott, 

You mention only one screen, so I am not sure if you have seen the other 6 screenshots in the first attachment (https://bugzilla.yoctoproject.org/attachment.cgi?id=554)

Cheers

Belen

(In reply to comment #6)
> I would say 'Set the proxies used when fetching source code' instead of 'Set
> the proxies that will be used during fetching source code.'  Everything else
> looks fine.
> 
> i am marking this resolved as the bug is to simply review the screen.
> 
> Scott
Comment 8 Scott Rifenbark 2012-05-30 15:04:51 UTC
When i clicked on the attachment earlier I was just presented with one screen.  My bad.  Here are full comments accordingly:

1.png: This is the comment I made earlier.  I would say 'Set the proxies used when fetching source code' instead of 'Set the proxies that will be used during fetching source code.'  Everything else looks fine

2.png:  For tool tip say "Set the proxies used when fetching source code.  A blank field uses a direct internet connection."

3.png: this is fine

4.png: I would say "Check this box to use a direct internet connection with no proxy"

5.png: I would say "Check this box to manually set up a specific proxy."

6.png: i am assuming the way this one works is the user defines just the HTTP proxy (the first of the available five).  Then by checking the box the user is saying to use that one for all five.  If this is the case, I would say "Check this box to use a single proxy for all five proxies.  For example, define the HTTP proxy and the check the box to use that proxy for all five proxies."  I am not quite sure how the actual code works here.  If the user can define the FTp proxy only and check the box and have it work then the wording needs to be different.  Let me know on the exact usage if it is other than I have descrived here.
Comment 9 Belen Barros Pena 2012-05-30 15:14:38 UTC
Created attachment 558 [details]
use-same-proxy-for-all-protocols.png

Sorry, Scott: I should have explained how it works.

If you select the "Manual proxy configuration" radio button, all the fields underneath (Use the same proxy for all protocols, HTTP proxy, HTTPS proxy, FTP proxy, GIT proxy and CVS proxy) become active. If you then select 'Use the same proxy for all protocols', the HTTP proxy field remains active, but all other fields become inactive. My understanding is that the HTTP proxy will be used for all other protocols (HTTPS, FTP, GIT and CVS). Maybe Shane can confirm.

I've attached a screenshot (use-same-proxy-for-all-protocols.png): I hope it helps.  

Belen
Comment 10 Scott Rifenbark 2012-05-30 15:33:54 UTC
Belen, 

Thanks for the added explanation for 6.png.  Based on your information I would say this for the tooltip:

"Check this box to use the HTTP proxy for all five proxies."  

Once the user checks the box, it will be evident that they can only define the HTTP proxy.
Comment 11 Belen Barros Pena 2012-05-30 16:07:59 UTC
So, as per Scott's review, below are the content changes that we need to make to the proxy tab:

1. At the top of the screen, replace 'Set the proxies that will be used during fetching source code' with 'Set the proxies used when fetching source code'

2. The persistent tooltip should say: 'Set the proxies used when fetching source code. A blank field uses a direct internet connection.'

3. The 'Direct internet connection' tooltip should say: 'Check this box to use a direct internet connection with no proxy'

4. The 'Manual proxy configuration' tooltip should say: 'Check this box to manually set up a specific proxy'

5. The 'Use the same proxy for all protocols' tooltip should say: 'Check this box to use the HTTP proxy for all five proxies'

Scott, I am assuming the 'Use authentication' tooltip shown in https://bugzilla.yoctoproject.org/attachment.cgi?id=555 is fine.

Shane: could you assign the bug to someone who could make the above changes?

Thanks!

Belen
Comment 12 Scott Rifenbark 2012-05-30 16:16:54 UTC
Belen, 

The Use Authentication tooltip is fine. 

Scott
Comment 13 Belen Barros Pena 2012-07-09 16:30:14 UTC
Shane: could you please assign this bug to someone who can implement the content changes in the Proxy tab? Valentin is working on 2162 (Settings-related), so he might be able to do it.

Thanks!

Belen
Comment 15 Jessica 2012-09-17 20:44:39 UTC
patch merged into bitbake
Comment 16 Belen Barros Pena 2012-09-21 09:50:52 UTC
Verified