Bug #21566
closedtop level vertical navigation menu is slow to respond
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.
Updated by Walden Raines about 7 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
Updated by Roxanne Hoover about 7 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.
Updated by Andrew Kofink about 7 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.
Updated by Marek Hulán about 7 years ago
+1 to comment 3, if menu opens immediately after click, it would be a win :-)
Updated by Walden Raines about 7 years ago
- Target version changed from 228 to 234
Updated by Walden Raines about 7 years ago
- Target version changed from 234 to 237
Updated by Walden Raines about 7 years ago
- Target version changed from 237 to 239
Updated by Walden Raines almost 7 years ago
- Target version changed from 239 to 243
Updated by Walden Raines almost 7 years ago
- Target version changed from 243 to 248
Updated by Walden Raines almost 7 years ago
- Target version changed from 248 to 250
Updated by Walden Raines over 6 years ago
- Target version changed from 250 to 253
Updated by Andrew Kofink over 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
Updated by The Foreman Bot over 6 years ago
- Pull request https://github.com/theforeman/foreman/pull/5898 added
Updated by The Foreman Bot over 6 years ago
- Status changed from New to Ready For Testing
Updated by Ohad Levy about 6 years ago
- Target version set to 1.20.0
- Fixed in Releases 1.20.0 added
Updated by Anonymous about 6 years ago
- Status changed from Ready For Testing to Closed
Applied in changeset a88bc000064e68502ed81ef08d5663d2037ec3c7.