Closed
Bug 877344
Opened 11 years ago
Closed 10 years ago
facebook.com - Comments field displays all previous entries
Categories
(Web Compatibility :: Mobile, defect, P2)
Tracking
(Not tracked)
RESOLVED
WORKSFORME
People
(Reporter: krudnitski, Assigned: colby)
References
(Blocks 1 open bug, )
Details
(Whiteboard: [country-all] [sitewait])
Attachments
(1 file)
(deleted),
image/png
|
Details |
Nexus 4. Go to the mobile facebook site. Click on 'comment' on someone's post. Tons of previous comments are automatically listed in there. Not very cool.
Comment 1•11 years ago
|
||
is this a regression?
Updated•11 years ago
|
Flags: needinfo?(krudnitski)
Reporter | ||
Comment 2•11 years ago
|
||
yes.
granted, the comments field never has behaved completely properly (just about to log another bug on it), but it has never done this before. Although I would guess that I started seeing it a couple of weeks ago.
Flags: needinfo?(krudnitski)
Updated•11 years ago
|
Keywords: regressionwindow-wanted
Comment 3•11 years ago
|
||
Is this a TE bug by any chance?
Comment 4•11 years ago
|
||
(In reply to Jason Smith [:jsmith] from comment #3)
> Is this a TE bug by any chance?
I strongly suspect that this is an invalid bug and works as expected, but Karen says it is a regression, which is why are looking for a regression range.
Reporter | ||
Comment 5•11 years ago
|
||
checked again today. It happens on nightly and aurora, but not on beta.
Comment 6•11 years ago
|
||
(In reply to Karen Rudnitski [:kar] from comment #5)
> checked again today. It happens on nightly and aurora, but not on beta.
Perhaps this is related to bug 566746?
Gavin or Mark, have you guys seen any similar complaints on desktop?
Comment 7•11 years ago
|
||
If this is a regression, it would be great to try to get a reduced testcase.
Cc'ing Martijn, since he has an arsenal of form autocomplete testcases :)
Comment 8•11 years ago
|
||
I can reproduce this on trunk and on Firefox beta.
I don't think this is in Fennec, there is no autocomplete="off" attribute on the relevant text input or the form.
When spoofing as iPhone 3, then autocomplete is turned off.
Also checked with Google Chrome, they also get a proper version of the page served with autocomplete off.
So I think this is tech evangelism.
Updated•11 years ago
|
tracking-fennec: ? → ---
Component: General → Mobile
Keywords: regressionwindow-wanted
Product: Firefox for Android → Tech Evangelism
Version: Firefox 24 → Trunk
Comment 9•11 years ago
|
||
(In reply to :Margaret Leibovic from comment #6)
> (In reply to Karen Rudnitski [:kar] from comment #5)
> > checked again today. It happens on nightly and aurora, but not on beta.
>
> Perhaps this is related to bug 566746?
>
> Gavin or Mark, have you guys seen any similar complaints on desktop?
Nope, I've never heard anything like this.
Updated•11 years ago
|
Summary: Comments field displays all previous entries → facebook.com - Comments field displays all previous entries
Comment 10•11 years ago
|
||
Isn't this just about firefox remembering past comments you typed in some form elements and offering them for auto-completion? This feature is typically used for INPUT fields (not, however, for TEXTAREAs where you typically want to type very different things from time to time).
Online, comment fields happen to mostly be coded as TEXTAREAs, so this distinction served us fine. Along comes Facebook and makes an INPUT field a comment field, and our logic is suddently not quite perfect anymore..
This is not really tech evangelism. Although, if Firefox respects autocomplete=off for this feature, we could ask Facebook very, very nicely to add autocomplete=off to their comment INPUT..
Updated•11 years ago
|
Component: Mobile → Preinstalled B2G Apps
Updated•11 years ago
|
Blocks: b2g-facebook
Comment 11•11 years ago
|
||
Android bug, not a FxOS bug - back over to Mobile
No longer blocks: b2g-facebook
Component: Preinstalled B2G Apps → Mobile
Comment 12•11 years ago
|
||
Harald, this looks to be Firefox specific content served by Facebook. See comment 8. Can you please ask Facebook to turn autocomplete off for Firefox as well?
Flags: needinfo?(hkirschner)
Comment 13•11 years ago
|
||
(do we know that Firefox respects autocomplete=off here? Just checking..)
Comment 15•11 years ago
|
||
yup
http://mxr.mozilla.org/mozilla-central/source/toolkit/components/satchel/nsFormFillController.cpp#862
http://mxr.mozilla.org/mozilla-central/source/content/base/src/nsContentUtils.cpp#682
I'll be off for any value other than empty or "on"
Flags: needinfo?(blassey.bugs)
Updated•11 years ago
|
Whiteboard: [country-all] [contactready]
Comment 16•11 years ago
|
||
We'll meet with them next week, I'll bring this up then.
Flags: needinfo?(hkirschner)
Comment 17•11 years ago
|
||
Harald, done? We'll change [contactready] to [sitewait] if we're waiting for a response :)
Updated•11 years ago
|
Status: NEW → ASSIGNED
Updated•10 years ago
|
Whiteboard: [country-all] [contactready] → [country-all] [sitewait]
Comment 21•10 years ago
|
||
Clearing NI - Colby is no longer at Fb
Closing this bug due to a lack of activity. If needed, feel free to reopen this bug.
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Flags: needinfo?(colby)
Resolution: --- → WORKSFORME
Updated•10 years ago
|
Depends on: b2g-facebook
Updated•10 years ago
|
Priority: -- → P2
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
•