diff options
| author | Matti Tahvonen <matti.tahvonen@itmill.com> | 2008-09-26 06:45:05 +0000 |
|---|---|---|
| committer | Matti Tahvonen <matti.tahvonen@itmill.com> | 2008-09-26 06:45:05 +0000 |
| commit | 609881fcb0994a786bd7285bc0cbfebb73b13ede (patch) | |
| tree | a4d0723b9cd084733972cb338efa2ce57dd68129 | |
| parent | 262c6deadf66d1faec0d40c98203b64ba27835b5 (diff) | |
| download | vaadin-framework-609881fcb0994a786bd7285bc0cbfebb73b13ede.tar.gz vaadin-framework-609881fcb0994a786bd7285bc0cbfebb73b13ede.zip | |
tuning tabIndex behavior
svn changeset:5519/svn branch:trunk
| -rw-r--r-- | src/com/itmill/toolkit/terminal/gwt/client/ui/IView.java | 8 | ||||
| -rw-r--r-- | src/com/itmill/toolkit/terminal/gwt/server/ApplicationServlet.java | 9 |
2 files changed, 10 insertions, 7 deletions
diff --git a/src/com/itmill/toolkit/terminal/gwt/client/ui/IView.java b/src/com/itmill/toolkit/terminal/gwt/client/ui/IView.java index 3611e6e0d5..3255f8223c 100644 --- a/src/com/itmill/toolkit/terminal/gwt/client/ui/IView.java +++ b/src/com/itmill/toolkit/terminal/gwt/client/ui/IView.java @@ -63,7 +63,9 @@ public class IView extends SimplePanel implements Paintable, DOM.sinkEvents(getElement(), Event.ONKEYDOWN); - DOM.setElementProperty(getElement(), "tabIndex", "0"); + // iview is focused when created so element needs tabIndex + // 1 due 0 is at the end of natural tabbing order + DOM.setElementProperty(getElement(), "tabIndex", "1"); RootPanel.get(elementId).add(this); @@ -326,8 +328,8 @@ public class IView extends SimplePanel implements Paintable, } private static native void focusElement(Element e) /*-{ - e.focus(); - }-*/; + e.focus(); + }-*/; public String onWindowClosing() { return null; diff --git a/src/com/itmill/toolkit/terminal/gwt/server/ApplicationServlet.java b/src/com/itmill/toolkit/terminal/gwt/server/ApplicationServlet.java index 30fd38ca73..13b6d7988d 100644 --- a/src/com/itmill/toolkit/terminal/gwt/server/ApplicationServlet.java +++ b/src/com/itmill/toolkit/terminal/gwt/server/ApplicationServlet.java @@ -842,7 +842,7 @@ public class ApplicationServlet extends HttpServlet { // there are multiple toolkit portlets visible at the same time. // TODO remove this when hosted mode on linux gets newer gecko - page.write("<iframe id=\"__gwt_historyFrame\" " + page.write("<iframe tabIndex=\"-1\" id=\"__gwt_historyFrame\" " + "style=\"width:0;height:0;border:0;overflow:" + "hidden\" src=\"javascript:false\"></iframe>\n"); page.write("<script language='javascript' src='" + staticFilePath @@ -895,9 +895,10 @@ public class ApplicationServlet extends HttpServlet { + "if(!itmill) { var itmill = {}} \n" + "itmill.toolkitConfigurations = {};\n" + "itmill.themesLoaded = {};\n"); - page.write("document.write('<iframe id=\"__gwt_historyFrame\" " - + "style=\"width:0;height:0;border:0;overflow:" - + "hidden\" src=\"javascript:false\"></iframe>');\n"); + page + .write("document.write('<iframe tabIndex=\"-1\" id=\"__gwt_historyFrame\" " + + "style=\"width:0;height:0;border:0;overflow:" + + "hidden\" src=\"javascript:false\"></iframe>');\n"); page.write("document.write(\"<script language='javascript' src='" + staticFilePath + "/" + WIDGETSET_DIRECTORY_PATH + widgetset + "/" + widgetset |