angular-ui/ui-grid

update on blur not working when tabbing or pressing enter

Open

#5,970 建立於 2017年1月26日

在 GitHub 查看
 (2 留言) (0 反應) (0 負責人)JavaScript (5,395 star) (2,496 fork)batch import
good first issue

描述

  • This is a bug report, not a question on how to use the grid.
    Use Stack Overflow or Gitter for questions.
  • You have searched the open issues to see if this bug has been filed before. We don't want duplicate issues.
  • You have reviewed the tutorials and documentation to ensure you are using the grid correctly. ui-grid.info
  • You are using the latest version of the grid. Older versions are not patched or supported.
  • You have provided steps to recreate your bug. A plunkr is even better.

It doesn't seem possible to get update on blur working when leaving a cell using the tab or enter keys. This plunker shows the problem:

http://plnkr.co/edit/8tA4aCuaJCb0TAE4kUiu

The plunker is tutorial 301 with editableCellTemplate added on the address.city and name columns. The editableCellTemplate is the standard cellEditor with update on blur added.

The plunker also watches the model for changes. The age field has no editableCellTemplate and typing into it shows that the model changes with each keystroke. The other two fields do not update on each keystroke but they also don't update if you leave the cell by tabbing or pressing enter. They do update if you click outside of the cell.

This problem has been reported before in #3181 which was closed. Also see PR #4698.

貢獻者指南

技術棧
javascript
領域
frontend
議題類型
bug
難度面向新貢獻者的預計實作難度，1 表示很小改動，5 表示專家級工作。
3
預計時間有經驗貢獻者完成調查、實作、測試並準備 pull request 的粗略時間範圍。
1-2 days
活動狀態議題目前的可參與程度：新鮮、活躍、陳舊、阻塞或等待維護者輸入。
stale
清晰度議題是否清楚說明預期改動、驗收標準和下一步。
clear
前置要求
AngularJS basicsui grid cell editingJavaScript events
新手友善度1-100 的估計分數，表示該議題對首次貢獻者的友善程度。
65
研究方向
The bug is in the update on blur behavior of cell editing when leaving via tab or enter keys. Refer to the provided plunkr reproduction and the previously closed issue #3181 and PR #4698. Investigate the editable cell template and event handling in the grid source code to identify why keyboard triggered blurs don't fire the update. The fix likely involves ensuring blur events from tab and enter are captured and processed similarly to mouse clicks outside the cell.