Closed
Bug 1197166
Opened 9 years ago
Closed 9 years ago
Facebook status update field not growing as you type in Firefox Android
Categories
(Web Compatibility :: Mobile, defect)
Web Compatibility
Mobile
Tracking
(Not tracked)
RESOLVED
WORKSFORME
People
(Reporter: hsteen, Unassigned)
References
()
Details
* Use Facebook
* Type a status update that's more than one line
Expected: box will grow to expand the text if it's > one line
Actual: shows only one line, needs some really tricky scrolling to edit the text
Reporter | ||
Comment 1•9 years ago
|
||
w.style.height = Math.max(0 + y.metrics.measure().height, y.originalHeight) + 'px';
y.metrics.measure() inserts the status update text in a PRE tag and measures its height. This way:
k.prototype.measure = function() {
'use strict';
var l = this._node,
m = this._shadow,
n = this._offset,
o = i.IS_APPLE_WEBKIT_IOS ? j : 0;
m.style.width = l.offsetWidth - n.width - o + 'px';
m.innerText = l.value + '\n';
return {
width: m.clientWidth + n.width,
height: m.clientHeight + n.height
};
};
The problem is that it uses element.innerText to "update" the PRE element. Firefox doesn't support the IEism innerText (yet) so it fails. Using textContent property instead makes it work fine.
Harald, can you take this further?
Flags: needinfo?(hkirschner)
Reporter | ||
Updated•9 years ago
|
Comment 2•9 years ago
|
||
Confirmed bug. In both Firefox & Chrome, the user begins with a single line for status update input. IN Chrome, the box expands downward for each subsequent line of text entered. In Firefox, the box does not expand and only the currently-active line of text is visible.
@Harald, can you please provide your thoughts on this?
Reporter | ||
Comment 3•9 years ago
|
||
The question wasn't for me, but these are my thoughts :)
IMO Mozilla should implement innerText per bug 264412. However, a workaround on Facebook's side is absolutely trivial and the problem is a real usability issue - it's really not a nice experience to enter long comments on Facebook at the moment. Hence we should reach out and recommend a small fix on their side. It will make their JS more standards-compliant :)
Reporter | ||
Comment 4•9 years ago
|
||
The innerText fix in Nightly solves this problem :-D
I'll close this as outreach is no longer necessary.
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → WORKSFORME
Reporter | ||
Updated•9 years ago
|
Flags: needinfo?(hkirschner)
Assignee | ||
Updated•6 years ago
|
Product: Tech Evangelism → Web Compatibility
You need to log in
before you can comment on or make changes to this bug.
Description
•