Project

General

Profile

Bug #26052

Vertical navigation missing IDs

Added by Walden Raines 10 months ago. Updated 7 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Web Interface
Target version:
-
Difficulty:
Triaged:
No
Bugzilla link:

Description

Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1647482

Description of problem:
Missing HTML IDs (at least) in Vertical Navigation. Regression against 6.4.

Version-Release number of selected component (if applicable):
Sat 6.5 snap 1

How reproducible:
Deterministic

Steps to Reproduce:
1. Login the Satellite
2. On the homepage, inspect Vertical Navigation's HTML

Actual results:
The tags don't have ID specified

Expected results:
The tags should have ID specified (preferrably the same as in Sat 6.4)


Related issues

Related to Foreman - Task #23445: Change vertical menu to react implementationClosed
Related to foreman-tasks - Bug #26827: Tasks menu has divider with no nameClosed

Associated revisions

Revision 68bda9e1 (diff)
Added by Gilad Lekner 7 months ago

fixes #26052 - navigation missing IDs

Revision bfce9f2d (diff)
Added by Tomer Brisker 7 months ago

Refs #26052 - skip menu items with no name

Not all menu items have a name, but patternfly-react requires names and
various attributes (such as id) assume the name is present.
This filters out any items with no name.
The selector has also been refactored a bit, to use `.map` instead of
`.forEach` and extracted the children logic to as seperate function.

History

#1 Updated by Walden Raines 10 months ago

  • Assignee changed from Ohad Levy to Gilad Lekner

#2 Updated by The Foreman Bot 9 months ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/6551 added

#3 Updated by Tomer Brisker 9 months ago

  • Related to Task #23445: Change vertical menu to react implementation added

#4 Updated by Tomer Brisker 7 months ago

  • Fixed in Releases 1.23.0 added

#5 Updated by Anonymous 7 months ago

  • Status changed from Ready For Testing to Closed

#6 Updated by The Foreman Bot 7 months ago

  • Pull request https://github.com/theforeman/foreman/pull/6770 added

#7 Updated by Tomer Brisker 7 months ago

  • Related to Bug #26827: Tasks menu has divider with no name added

Also available in: Atom PDF