Closed
Bug 904111
Opened 12 years ago
Closed 12 years ago
[MP] Defect - Flyout errors interfering with update button functionality
Categories
(Firefox for Metro Graveyard :: Browser, defect, P2)
Tracking
(Not tracked)
VERIFIED
FIXED
Firefox 26
People
(Reporter: jimm, Assigned: jimm)
References
Details
(Whiteboard: [preview] feature=defect c=tbd u=tbd p=1)
Attachments
(2 files, 1 obsolete file)
1.18 KB,
patch
|
TimAbraldes
:
review+
|
Details | Diff | Splinter Review |
961 bytes,
patch
|
Details | Diff | Splinter Review |
No description provided.
![]() |
Assignee | |
Updated•12 years ago
|
Summary: Flyout errors interfering with update button functionlaity → Flyout errors interfering with update button functionality
![]() |
Assignee | |
Comment 1•12 years ago
|
||
TypeError: FlyoutPanelsUI.AboutFlyout is undefined
chrome://browser/content/browser.xul
1
Updated•12 years ago
|
Blocks: metrov1defect&change
Summary: Flyout errors interfering with update button functionality → [MP] Defect - Flyout errors interfering with update button functionality
Whiteboard: [preview] → [preview] feature=defect c=tbd u=tbd p=0
![]() |
Assignee | |
Comment 2•12 years ago
|
||
Looks like it should be AboutFlyoutPanel. (bug 897830)
No longer blocks: metrov1defect&change
Summary: [MP] Defect - Flyout errors interfering with update button functionality → Flyout errors interfering with update button functionality
Whiteboard: [preview] feature=defect c=tbd u=tbd p=0 → [preview]
![]() |
Assignee | |
Updated•12 years ago
|
Assignee: nobody → jmathies
Updated•12 years ago
|
Blocks: metrov1it13
Status: NEW → ASSIGNED
Priority: -- → P2
QA Contact: jbecerra
Summary: Flyout errors interfering with update button functionality → [MP] Defect - Flyout errors interfering with update button functionality
Whiteboard: [preview] → [preview] feature=defect c=tbd u=tbd p=0
Updated•12 years ago
|
Whiteboard: [preview] feature=defect c=tbd u=tbd p=0 → [preview] feature=defect c=tbd u=tbd p=1
![]() |
Assignee | |
Comment 3•12 years ago
|
||
Attachment #789071 -
Flags: review?(tabraldes)
Comment 4•12 years ago
|
||
Comment on attachment 789071 [details] [diff] [review]
fix
Review of attachment 789071 [details] [diff] [review]:
-----------------------------------------------------------------
::: dom/browser-element/BrowserElementPanning.js
@@ -463,5 @@
> - return metroUtils && metroUtils.immersive;
> - } catch (e) {}
> - return false;
> - },
> -
qref'ed this out of the wrong patch
![]() |
Assignee | |
Comment 5•12 years ago
|
||
ahh poop.
![]() |
Assignee | |
Comment 6•12 years ago
|
||
Attachment #789071 -
Attachment is obsolete: true
Attachment #789071 -
Flags: review?(tabraldes)
Attachment #789083 -
Flags: review?(tabraldes)
Comment 7•12 years ago
|
||
Comment on attachment 789071 [details] [diff] [review]
fix
Review of attachment 789071 [details] [diff] [review]:
-----------------------------------------------------------------
r=me if you fix the issue brian mentioned. Also double-check the indentation on that line in browser.xul. Splinter is showing the line as having no indentation at all.
::: browser/metro/base/content/browser.xul
@@ +383,5 @@
> #ifdef MOZ_UPDATER
> <deck id="updateDeck" orient="vertical">
> <hbox id="updateButtonBox" align="center">
> <button id="updateButton" align="start"
> + oncommand="FlyoutPanelsUI.AboutFlyoutPanel.appUpdater.buttonOnCommand();"/>
nit: What happened to the indentation here? Is splinter just showing this weirdly?
Attachment #789071 -
Attachment is obsolete: false
Comment 8•12 years ago
|
||
Comment on attachment 789071 [details] [diff] [review]
fix
Oops, reviewed obsolete patch
Attachment #789071 -
Attachment is obsolete: true
Comment 9•12 years ago
|
||
Comment on attachment 789083 [details] [diff] [review]
fix
Review of attachment 789083 [details] [diff] [review]:
-----------------------------------------------------------------
Just double-check the indentation plz
Attachment #789083 -
Flags: review?(tabraldes) → review+
Updated•12 years ago
|
Blocks: MetroPreviewRelease
![]() |
Assignee | |
Comment 10•12 years ago
|
||
![]() |
Assignee | |
Updated•12 years ago
|
Keywords: checkin-needed
![]() |
Assignee | |
Comment 11•12 years ago
|
||
Keywords: checkin-needed
Comment 12•12 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 26
Comment 13•12 years ago
|
||
Could anyone please give any guidelines that can help the QA in verifying this issue? Thanks!
Flags: needinfo?
Updated•12 years ago
|
Flags: needinfo? → needinfo?(jmathies)
![]() |
Assignee | |
Comment 14•12 years ago
|
||
(In reply to Manuela Muntean [:Manuela] [QA] from comment #13)
> Could anyone please give any guidelines that can help the QA in verifying
> this issue? Thanks!
If you can test updates, the ability to click the restart button would verify this.
Flags: needinfo?(jmathies)
Comment 15•12 years ago
|
||
> If you can test updates, the ability to click the restart button would
> verify this.
Verified as fixed, with the latest Nightly (build ID: 20130915030208), for iteration #14, following the above indication. The updates and the restart button both work.
Status: RESOLVED → VERIFIED
Updated•11 years ago
|
OS: Windows 8 Metro → Windows 8.1
You need to log in
before you can comment on or make changes to this bug.
Description
•