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

fix character size measurement

Browse source
This commit is contained in:
Fabian Jakobs 2010-12-15 15:41:08 +01:00
commit 633293ee5e
1 changed files with 2 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

5
lib/ace/layer/text.js
View file

@ -92,6 +92,7 @@ var Text = function(parentEl) {
}, },
this.$measureSizes = function() { this.$measureSizes = function() {
var n = 1000;
if (!this.$measureNode) { if (!this.$measureNode) {
var measureNode = this.$measureNode = document.createElement("div"); var measureNode = this.$measureNode = document.createElement("div");
var style = measureNode.style; var style = measureNode.style;
@ -107,12 +108,11 @@ var Text = function(parentEl) {
// in FF 3.6 monospace fonts can have a fixed sub pixel width. // in FF 3.6 monospace fonts can have a fixed sub pixel width.
// that's why we have to measure many characters // that's why we have to measure many characters
// Note: characterWidth can be a float! // Note: characterWidth can be a float!
var n = 1000;
measureNode.innerHTML = lang.stringRepeat("Xy", n); measureNode.innerHTML = lang.stringRepeat("Xy", n);
document.body.insertBefore(measureNode, document.body.firstChild); document.body.insertBefore(measureNode, document.body.firstChild);
} }
var style = this.$measureNode; var style = this.$measureNode.style;
for (var prop in this.$fontStyles) { for (var prop in this.$fontStyles) {
var value = dom.computedStyle(this.element, prop); var value = dom.computedStyle(this.element, prop);
style[prop] = value; style[prop] = value;
@ -122,7 +122,6 @@ var Text = function(parentEl) {
height: this.$measureNode.offsetHeight, height: this.$measureNode.offsetHeight,
width: this.$measureNode.offsetWidth / (n * 2) width: this.$measureNode.offsetWidth / (n * 2)
}; };
return size; return size;
}; };