Bug 1009774

Summary: [ALL_LANG][Admin Portal] Host tab -> Network Interfaces subtab ->Setup Host Networks -> remove 'menu for' header
Product: [Retired] oVirt Reporter: Yuko Katabami <ykatabam>
Component: ovirt-engine-webadminAssignee: Alona Kaplan <alkaplan>
Status: CLOSED CURRENTRELEASE QA Contact: Yuko Katabami <ykatabam>
Severity: low Docs Contact:
Priority: low    
Version: 3.4CC: acathrow, alkaplan, ecohen, eng-i18n-bugs, gcheresh, gklein, iheim, knesenko, lijli, lvernia, masayag, mgoldboi, mpavlik, nyechiel, qe-i18n-bugs, Rhev-m-bugs, yeylon
Target Milestone: ---Keywords: CodeChange, i18n, Triaged
Target Release: 3.4.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard: network
Fixed In Version: av3 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2014-03-31 12:25:03 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:
Attachments:
Description Flags
Menu for bond0 -- unlocalized
none
code: NetworkItemPanel.java none

Description Yuko Katabami 2013-09-19 06:58:43 UTC
Created attachment 799755 [details]
Menu for bond0 -- unlocalized

Description of problem:
"Menu for bond0" is unlocalized and is not pushed to Zanata.

Version-Release number of selected component (if applicable):3.3.0-0.22


How reproducible:100%


Steps to Reproduce:
1. Login to Admin Portal in any given (non en-US) locale
2. Click on Host tab and select a host in the list.
3. Click on Network Interfaces subtab in the details pane.
4. Click on "Setup Host Networks" action button.
5. Right click one of the eth will show ""Menu for bond0"
6. Also after bonding two nics, when you right click, it also shows ""Menu for bond0"

Actual results:
"Menu for bond0" is not localized


Expected results:
"Menu for bond0" should be localized

Additional info:

Comment 1 Einav Cohen 2013-09-19 22:04:07 UTC
Created attachment 800183 [details]
code: NetworkItemPanel.java

Comment 2 Einav Cohen 2013-09-19 22:09:03 UTC
confirming that the "Menu for [name]" string is in hard-coded English in NetworkItemPanel.java (see attachment 800183 [details]).

need to localize into a string in the form of "Menu for {0}" (where {0} is the placeholder for the network/bond name) within the web-admin's ApplicationMessages or similar.

Comment 3 Lior Vernia 2013-09-22 13:31:20 UTC
Einav, is this menu header actually useful? Might this be a good opportunity to get rid of it?

Comment 4 Einav Cohen 2013-09-22 18:37:36 UTC
(In reply to Lior Vernia from comment #3)
> Einav, is this menu header actually useful? Might this be a good opportunity
> to get rid of it?

If the users thread [1] concludes that - feel free to close this BZ once the menu's removal is merged upstream. thanks.

Comment 5 Einav Cohen 2013-09-22 18:38:45 UTC
(In reply to Einav Cohen from comment #4)
> (In reply to Lior Vernia from comment #3)
> > Einav, is this menu header actually useful? Might this be a good opportunity
> > to get rid of it?
> 
> If the users thread [1] concludes that - feel free to close this
> BZ once the menu's removal is merged upstream. thanks.

[1] http://lists.ovirt.org/pipermail/users/2013-September/016518.html

Comment 7 Nir Yechiel 2014-02-24 12:33:32 UTC
*** Bug 1063896 has been marked as a duplicate of this bug. ***

Comment 8 Lior Vernia 2014-02-27 13:08:01 UTC
As the thread on the mailing list didn't seem to produce much interest, and due to numerous complaints about the ugliness of the empty menu, I feel it's safe to remove the title of the menu and not show it in case there are no valid actions to perform.

Comment 9 Alona Kaplan 2014-03-10 13:44:11 UTC
Moved by mistake to on_qa.

Comment 10 GenadiC 2014-03-18 10:04:49 UTC
Verified in av3

Comment 11 Sandro Bonazzola 2014-03-31 12:25:03 UTC
this is an automated message: moving to Closed CURRENT RELEASE since oVirt 3.4.0 has been released