Browse Source

fixes #5972, Table now focused eagerly when interacting with mouse with a header cell. Previously the table was focused in at an invalid moment causing variables to be in wrong order and for the table in two patches.

svn changeset:15937/svn branch:6.5
tags/6.7.0.beta1
Matti Tahvonen 13 years ago
parent
commit
918cfdb7ff
1 changed files with 8 additions and 2 deletions
  1. 8
    2
      src/com/vaadin/terminal/gwt/client/ui/VScrollTable.java

+ 8
- 2
src/com/vaadin/terminal/gwt/client/ui/VScrollTable.java View File

@@ -1862,10 +1862,16 @@ public class VScrollTable extends FlowPanel implements Table, ScrollHandler,
onResizeEvent(event);
}
} else {
handleCaptionEvent(event);
if (DOM.eventGetType(event) == Event.ONMOUSEUP) {
/*
* Ensure focus before handling caption event. Otherwise
* variables changed from caption event may be before
* variables from other components that fire variables when
* they lose focus.
*/
if (DOM.eventGetType(event) == Event.ONMOUSEDOWN) {
scrollBodyPanel.setFocus(true);
}
handleCaptionEvent(event);
event.stopPropagation();
event.preventDefault();
}

Loading…
Cancel
Save