[MGNLUI-2814] ENTER keyboard shortcut causes error in Move Dialog Created: 22/Apr/14 Updated: 13/May/14 Resolved: 30/Apr/14 |
|
| Status: | Closed |
| Project: | Magnolia UI |
| Component/s: | None |
| Affects Version/s: | 5.2.4 |
| Fix Version/s: | 5.2.5 |
| Type: | Bug | Priority: | Major |
| Reporter: | Christopher Zimmermann | Assignee: | Peili Liang |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||
| 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: | |||||||||
| Description |
|
To reproduce:
However if you press the "MOVE INSIDE" button, the item is moved as expected. The problem relates to the fact that the ENTER keyboard attempts to trigger the action named "commit" which most dialogs have. (See BaseDialogPresenter#CommitDialogShortcutListener). Perhaps ENTER key should not be supported here. ERROR a.ui.contentapp.movedialog.MoveDialogPresenterImpl: An error occurred while executing an action. |
| Comments |
| Comment by Cheng Hu [ 29/Apr/14 ] |
|
The behavior we implement for the ENTER key is that if the Move Inside button is available, it will be as if Move Inside button is pressed. If the Move Inside button is not available, then it will be as if nothing happened and there will be no errors. |