Project

General

Profile

Actions

Bug #21566

closed

top level vertical navigation menu is slow to respond

Added by Andrew Kofink over 6 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Web Interface
Target version:
Difficulty:
Triaged:
No
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.

Actions #1

Updated by Walden Raines over 6 years ago

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

Updated by Roxanne Hoover over 6 years 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.

Actions #3

Updated by Andrew Kofink over 6 years 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.

Actions #4

Updated by Marek Hulán over 6 years ago

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

Actions #5

Updated by Walden Raines over 6 years ago

  • Target version changed from 228 to 234
Actions #6

Updated by Walden Raines over 6 years ago

  • Target version changed from 234 to 237
Actions #7

Updated by Walden Raines over 6 years ago

  • Target version changed from 237 to 239
Actions #8

Updated by Walden Raines about 6 years ago

  • Target version changed from 239 to 243
Actions #9

Updated by Walden Raines about 6 years ago

  • Target version changed from 243 to 248
Actions #10

Updated by Walden Raines about 6 years ago

  • Target version changed from 248 to 250
Actions #11

Updated by Walden Raines about 6 years ago

  • Target version changed from 250 to 253
Actions #12

Updated by Andrew Kofink almost 6 years 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

Actions #13

Updated by Walden Raines almost 6 years ago

  • Target version deleted (253)
Actions #14

Updated by The Foreman Bot over 5 years ago

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

Updated by The Foreman Bot over 5 years ago

  • Status changed from New to Ready For Testing
Actions #16

Updated by Ohad Levy over 5 years ago

  • Target version set to 1.20.0
  • Fixed in Releases 1.20.0 added
Actions #17

Updated by Anonymous over 5 years ago

  • Status changed from Ready For Testing to Closed
Actions

Also available in: Atom PDF