jyapayne/ace
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Merge pull request #2237 from ajaxorg/focus

Browse source 
fix focus related issues
This commit is contained in:
Lennart Kats 2014-11-17 10:34:12 +01:00
commit d097d41144
3 changed files with 25 additions and 11 deletions
Download patch file Download diff file Expand all files Collapse all files

9
lib/ace/keyboard/textinput.js
View file

@ -75,7 +75,14 @@ var TextInput = function(parentNode, host) {
host.onFocus(e);
resetSelection();
});
this.focus = function() { text.focus(); };
this.focus = function() {
text.style.position = "fixed";
text.style.top = "-10000000px";
text.focus();
setTimeout(function() {
text.style.position = "";
}, 0);
};
this.blur = function() { text.blur(); };
this.isFocused = function() {
return isFocused;

10
lib/ace/mouse/mouse_handler.js
View file

@ -47,10 +47,12 @@ var MouseHandler = function(editor) {
new DefaultGutterHandler(this);
new DragdropHandler(this);
var focusEditor = function(e) {
if (!editor.isFocused() && editor.textInput)
editor.textInput.moveToMouse(e);
editor.focus()
var focusEditor = function(e) {
// because we have to call event.preventDefault() any window on ie and iframes
// on other browsers do not get focus, so we have to call window.focus() here
if (!document.hasFocus || !document.hasFocus())
window.focus();
editor.focus();
};
var mouseTarget = editor.renderer.getMouseEventTarget();

17
lib/ace/virtual_renderer.js
View file

@ -375,6 +375,9 @@ var VirtualRenderer = function(container, theme) {
if (this.resizing)
this.resizing = 0;
// reset cached values on scrollbars, needs to be removed when switching to non-native scrollbars
// see https://github.com/ajaxorg/ace/issues/2195
this.scrollBarV.scrollLeft = this.scrollBarV.scrollTop = null;
};
this.$updateCachedSize = function(force, gutterWidth, width, height) {
@ -634,9 +637,12 @@ var VirtualRenderer = function(container, theme) {
var posLeft = this.$cursorLayer.$pixelPos.left;
posTop -= config.offset;
var style = this.textarea.style;
var h = this.lineHeight;
if (posTop < 0 || posTop > config.height - h)
if (posTop < 0 || posTop > config.height - h) {
style.top = style.left = "0";
return;
}
var w = this.characterWidth;
if (this.$composition) {
@ -649,11 +655,10 @@ var VirtualRenderer = function(container, theme) {
posLeft = this.$size.scrollerWidth - w;
posLeft += this.gutterWidth;
this.textarea.style.height = h + "px";
this.textarea.style.width = w + "px";
this.textarea.style.left = Math.min(posLeft, this.$size.scrollerWidth - w) + "px";
this.textarea.style.top = Math.min(posTop, this.$size.height - h) + "px";
style.height = h + "px";
style.width = w + "px";
style.left = Math.min(posLeft, this.$size.scrollerWidth - w) + "px";
style.top = Math.min(posTop, this.$size.height - h) + "px";
};
/**