Bug 1431072
| Summary: | [Ansible Embedded] - Menu not highlighted when navigate to Automation -> Ansible | |||
|---|---|---|---|---|
| Product: | Red Hat CloudForms Management Engine | Reporter: | Pavol Kotvan <pakotvan> | |
| Component: | UI - OPS | Assignee: | Martin Hradil <mhradil> | |
| Status: | CLOSED CURRENTRELEASE | QA Contact: | Kedar Kulkarni <kkulkarn> | |
| Severity: | high | Docs Contact: | ||
| Priority: | high | |||
| Version: | 5.8.0 | CC: | hkataria, jhardy, kkulkarn, lcouzens, lgalis, mhradil, mpovolny, obarenbo, simaishi | |
| Target Milestone: | GA | Keywords: | TestOnly | |
| Target Release: | 5.9.0 | |||
| Hardware: | Unspecified | |||
| OS: | Unspecified | |||
| Whiteboard: | ui:ansible_embed | |||
| Fixed In Version: | 5.9.0.1 | Doc Type: | If docs needed, set a value | |
| Doc Text: | Story Points: | --- | ||
| Clone Of: | ||||
| : | 1439294 (view as bug list) | Environment: | ||
| Last Closed: | 2018-03-06 14:58:13 UTC | Type: | Bug | |
| Regression: | --- | Mount Type: | --- | |
| Documentation: | --- | CRM: | ||
| Verified Versions: | Category: | Bug | ||
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | ||
| Cloudforms Team: | Ansible | Target Upstream Version: | ||
| Embargoed: | ||||
| Bug Depends On: | ||||
| Bug Blocks: | 1439294 | |||
|
Comment 4
Martin Hradil
2017-03-31 12:57:40 UTC
New commit detected on ManageIQ/manageiq-ui-classic/master: https://github.com/ManageIQ/manageiq-ui-classic/commit/36991830d71726454d4fc22d35c76b3a78ed2b91 commit 36991830d71726454d4fc22d35c76b3a78ed2b91 Author: Martin Hradil <mhradil> AuthorDate: Mon Apr 3 11:50:36 2017 +0000 Commit: Martin Hradil <mhradil> CommitDate: Mon Apr 3 15:46:01 2017 +0000 Menu highlighting - use menu_section_id when available fall back to the old `@layout` method if not, preserving the original `primary_nav_class` behaviour for root menu sections. + a lot of to_sym conversions to catch inconsistencies https://bugzilla.redhat.com/show_bug.cgi?id=1431072 app/helpers/application_helper.rb | 33 ++++++++++++++++++---------- app/views/layouts/_vertical_navbar.html.haml | 10 ++++----- 2 files changed, 26 insertions(+), 17 deletions(-) Verified in 5.9.0.2 |