Closed
Bug 538999
Opened 15 years ago
Closed 14 years ago
Formfill does not zoom out on load of dynamically loaded page content
Categories
(Firefox for Android Graveyard :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: aakashd, Assigned: vingtetun)
Details
(Keywords: polish, Whiteboard: formfill)
Attachments
(1 file, 1 obsolete file)
3.15 KB,
patch
|
mfinkle
:
review+
|
Details | Diff | Splinter Review |
Build Id: Mozilla/5.0 (X11; U; Linux armv7l; Nokia N900; en-US; rv:1.9.2pre) Gecko/20100111 Firefox/3.6pre Fennec/1.1a1pre and Mozilla/5.0 (X11; U; Linux armv6l; Nokia N8xx; en-US; rv:1.9.2) Gecko/20100111 Firefox/3.6pre Fennec/1.1a1pre Steps to Reproduce: 1. Go to www.aircanada.com 2. Click on "Canada - English" 3. Click on "Flight Status" 4. Tap on the flight number field next to "AC" and the date field. 5. Type in "757" as a search term. 6. Click on "Search" Actual Results: The page loads the content, but the formfill bar is still visible and the page is zoomed in. Expected Results: The page should show the entire page or wherever the content has been updated.
Reporter | ||
Updated•15 years ago
|
Whiteboard: formfill
Assignee | ||
Comment 1•15 years ago
|
||
What is changing here is the content of an iframe in the webpage. I'm starting to thing of correcting this one and bug 538430 in the same row by adding a custom web progress listener to the ownerDocument of the current target element.
Assignee | ||
Comment 2•14 years ago
|
||
(In reply to comment #1) > What is changing here is the content of an iframe in the webpage. > > I'm starting to thing of correcting this one and bug 538430 in the same row by > adding a custom web progress listener to the ownerDocument of the current > target element. Finally I have a plan relying on http://mxr.mozilla.org/mozilla-central/source/content/html/content/public/nsIFormSubmitObserver.idl for those. I will post a working patch today or tomorrow.
Assignee | ||
Comment 3•14 years ago
|
||
This patch correct this bug and bug bug 538430.
Attachment #428332 -
Flags: review?(mark.finkle)
Comment 4•14 years ago
|
||
Comment on attachment 428332 [details] [diff] [review] Patch >diff -r d61fdff66712 chrome/content/browser.js >+const gFormSubmitObserver = { >+ notify: function notify(aFormElement, aWindow, aActionURI, aCancelSubmit) { >+ let doc = aWindow.content.top.document; >+ let tab = Browser.getTabForDocument(doc); >+ if (tab) >+ tab.browser.hasChanged = true; >+ }, Maybe we could just call FormHelper.close() here? >- if (location != this.browser.lastSpec) { >+ if (location != this.browser.lastSpec || this.browser.hasChanged) { Do we really want to fire all this code?
Comment 5•14 years ago
|
||
I guess we want to zoom out, but do we want to close notifications and scroll to top?
Assignee | ||
Comment 6•14 years ago
|
||
(In reply to comment #5) > I guess we want to zoom out, but do we want to close notifications and scroll > to top? I'm not really sure for the notifications part, but this didn't remove the notification prompt for remembering password as example. For the scroll to top part, I'd say yes, and as an example just tried to log in into twitter by using the form assistant(and futhermore, once we'll sync the browser and the content part we really want to) So, i agree that tab.browser.hasChanged is not very elegant, but I've wanted to avoid duplicating code.
Reporter | ||
Updated•14 years ago
|
Component: General → Panning/Zooming
Reporter | ||
Updated•14 years ago
|
Assignee: 21 → nobody
Component: Panning/Zooming → General
Assignee | ||
Comment 7•14 years ago
|
||
Just a little changed to the patch to remove the hasChanged flag.
Assignee: nobody → 21
Attachment #428332 -
Attachment is obsolete: true
Attachment #442685 -
Flags: review?(mark.finkle)
Attachment #428332 -
Flags: review?(mark.finkle)
Updated•14 years ago
|
Attachment #442685 -
Flags: review?(mark.finkle) → review+
Comment 8•14 years ago
|
||
Comment on attachment 442685 [details] [diff] [review] Patch v0.2 >diff -r 64c363ad5a4d chrome/content/browser.js > >+const gFormSubmitObserver = { var FormSubmitObserver + { We need to move away from Firefoxes old naming conventions and use the new ones. Thanks for removing the flag! r+ with nit fixed
Assignee | ||
Comment 9•14 years ago
|
||
http://hg.mozilla.org/mobile-browser/rev/e59e43d3d2d9
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•