Bug 1009774 - [ALL_LANG][Admin Portal] Host tab -> Network Interfaces subtab ->Setup Host Networks -> remove 'menu for' header
Summary: [ALL_LANG][Admin Portal] Host tab -> Network Interfaces subtab ->Setup Host N...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: oVirt
Classification: Retired
Component: ovirt-engine-webadmin
Version: 3.4
Hardware: Unspecified
OS: Unspecified
low
low
Target Milestone: ---
: 3.4.0
Assignee: Alona Kaplan
QA Contact: Yuko Katabami
URL:
Whiteboard: network
: 1063896 (view as bug list)
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-09-19 06:58 UTC by Yuko Katabami
Modified: 2014-07-30 06:55 UTC (History)
17 users (show)

Fixed In Version: av3
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2014-03-31 12:25:03 UTC
oVirt Team: ---


Attachments (Terms of Use)
Menu for bond0 -- unlocalized (260.55 KB, image/png)
2013-09-19 06:58 UTC, Yuko Katabami
no flags Details
code: NetworkItemPanel.java (37.74 KB, image/png)
2013-09-19 22:04 UTC, Einav Cohen
no flags Details


Links
System ID Private Priority Status Summary Last Updated
oVirt gerrit 24982 0 None None None Never
oVirt gerrit 25381 0 None MERGED webadmin: removing menu header from the menu in setup network dialog Never

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


Note You need to log in before you can comment on or make changes to this bug.