Project

General

Profile

Bug #21566

top level vertical navigation menu is slow to respond

Added by Andrew Kofink about 1 year ago. Updated 2 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Web Interface
Target version:
Difficulty:
Triaged:
No
Bugzilla link:
Team Backlog:
Fixed in Releases:
Found in Releases:

Description

The old horizontal nav used to open the top level menus (i.e. Monitor, Content, Containers) immediately. The new vertical nav has a delay (maybe half a second?) before the menu opens. It should open immediately.

Associated revisions

Revision a88bc000 (diff)
Added by Gilad Lekner 2 months ago

Fixes #23445, #21566, #23448 - Refactor Layout to PF

History

#1 Updated by Walden Raines about 1 year ago

  • Target version set to 228
  • Assignee set to Amir Fefer
  • Category changed from Web UI to Web Interface
  • Project changed from Katello to Foreman

#2 Updated by Roxanne Hoover about 1 year ago

The delay is intentionally. If memory serves me correctly this was so that users who "wander" with their mouse weren't getting inundated with fly-outs. The small delay offers that relief.

#3 Updated by Andrew Kofink about 1 year ago

The delay is either too long, or the menu should respond immediately to clicks. As a user, it feels like I'm wasting time waiting on the menu to open. Also, the horizontal navigation has always opened immediately, so I would think that users are accustomed to that behavior. Is this delay a result of feedback from UX trials? I would be curious how many users out of the group commented on this aspect of the menu.

#4 Updated by Marek Hulán about 1 year ago

+1 to comment 3, if menu opens immediately after click, it would be a win :-)

#5 Updated by Walden Raines about 1 year ago

  • Target version changed from 228 to 234

#6 Updated by Walden Raines about 1 year ago

  • Target version changed from 234 to 237

#7 Updated by Walden Raines about 1 year ago

  • Target version changed from 237 to 239

#8 Updated by Walden Raines 11 months ago

  • Target version changed from 239 to 243

#9 Updated by Walden Raines 10 months ago

  • Target version changed from 243 to 248

#10 Updated by Walden Raines 10 months ago

  • Target version changed from 248 to 250

#11 Updated by Walden Raines 9 months ago

  • Target version changed from 250 to 253

#12 Updated by Andrew Kofink 6 months ago

So, I did some research, and patternfly doesn't allow overriding this delay, though there's an issue/feature request to add it [1]. I went ahead and submitted a PR to patternfly [2] to implement the feature, since the conversation on the feature request seemed stale.

[1] https://github.com/patternfly/patternfly/issues/744
[2] https://github.com/patternfly/patternfly/pull/1073

#13 Updated by Walden Raines 6 months ago

  • Target version deleted (253)

#14 Updated by The Foreman Bot 4 months ago

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

#15 Updated by The Foreman Bot 4 months ago

  • Status changed from New to Ready For Testing

#16 Updated by Ohad Levy 2 months ago

  • Target version set to 1.20.0
  • Fixed in Releases 1.20.0 added

#17 Updated by Anonymous 2 months ago

  • Status changed from Ready For Testing to Closed

Also available in: Atom PDF