[MGNLUI-5084] Only first subapp keeps keyboard focus (new framework) Created: 18/Mar/19  Updated: 17/Mar/20  Resolved: 04/Feb/20

Status: Closed
Project: Magnolia UI
Component/s: None
Affects Version/s: 6.1
Fix Version/s: None

Type: Bug Priority: Neutral
Reporter: Adam Siska Assignee: Unassigned
Resolution: Obsolete Votes: 1
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Issue Links:
Relates
relates to MGNLUI-4974 Implement keyboard navigation in tabu... Closed
relates to MGNLUI-4976 Provide default action support in Gri... Closed
dependency
depends upon MGNLUI-5065 Keyboard navigation doesn't work in t... Closed
Template:
Acceptance criteria:
Empty
Task DoD:
[ ]* Doc/release notes changes? Comment present?
[ ]* Downstream builds green?
[ ]* Solution information and context easily available?
[ ]* Tests
[ ]* FixVersion filled and not yet released
[ ]  Architecture Decision Record (ADR)
Bug DoR:
[ ]* Steps to reproduce, expected, and actual results filled
[ ]* Affected version filled
Date of First Response:
Epic Link: Keyboard shortcuts
Story Points: 3

 Description   

When ShortcutListeners attached on grids within Tabsheet, only grid in first tab can trigger action. No action is triggered when keys are touched on grids in other tabs.

 

Steps to reproduce:

  • open Notifications/Tasks app
  • select second tab
  • hit enter on item

Expected: detail view opens

Actual behaviour: nothing happens

 

Some investigation:

(vaadin-server-8.5.2-sources.jar) com.vaadin.event.ActionManager::handleAction() gets action parameter with wrong (always the first) grid in it. Therefore ownActions.contains(action) (on line 239) returns false for grids other than the first in tabsheet as ownActions is set correctly.



 Comments   
Comment by Roman Kovařík [ 03/Jul/19 ]

Code field doesn't accept ENTER key which might be related.

Comment by Simon Lutz [ 04/Feb/20 ]

Fixed by MGNLUI-4974

Generated at Mon Feb 12 09:23:03 CET 2024 using Jira 9.4.2#940002-sha1:46d1a51de284217efdcb32434eab47a99af2938b.