Closed
Bug 1001234
Opened 9 years ago
Closed 9 years ago
disable arrow panel animation on linux
Categories
(Toolkit :: XUL Widgets, defect)
Toolkit
XUL Widgets
Tracking
()
VERIFIED
FIXED
mozilla32
People
(Reporter: Gavin, Assigned: enndeakin)
References
(Blocks 1 open bug)
Details
(Whiteboard: p=3 s=it-32c-31a-30b.1 [qa!])
Attachments
(1 file, 2 obsolete files)
8.26 KB,
patch
|
dao
:
review+
Sylvestre
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
See bug 994562 comment 13 for the rationale and instructions on how to do this.
Updated•9 years ago
|
Flags: firefox-backlog?
Reporter | ||
Updated•9 years ago
|
Flags: firefox-backlog? → firefox-backlog+
Whiteboard: p=3
Updated•9 years ago
|
Component: General → XUL Widgets
Updated•9 years ago
|
Assignee | ||
Comment 1•9 years ago
|
||
Does the patch in bug 994562 fix the problem?
Assignee | ||
Updated•9 years ago
|
Assignee: nobody → enndeakin
Reporter | ||
Updated•9 years ago
|
Whiteboard: p=3 → p=3 s=it-32c-31a-30b.1 [qa+]
Assignee | ||
Comment 3•9 years ago
|
||
Attachment #8415646 -
Flags: review?(dao)
Comment 4•9 years ago
|
||
Comment on attachment 8415646 [details] [diff] [review] Disable animation on Linux > <binding id="arrowpanel" extends="chrome://global/content/bindings/popup.xml#panel"> >- <content flip="both" side="top" position="bottomcenter topleft" consumeoutsideclicks="false"> >+ <content flip="both" side="top" position="bottomcenter topleft" consumeoutsideclicks="false" >+#ifdef MOZ_WIDGET_GTK >+ animate="false" >+#endif >+ > Will this override the attribute in case it was set manually on a popup? I'd prefer if the attribute didn't have to be altered depending on the platform and was just ignored on Linux. Also, bug 994562 seems like a reasonable compromise to me (it's still a bit ugly with the compositor disabled, but I don't think we should overoptiize for that case).
Attachment #8415646 -
Flags: review?(dao)
Assignee | ||
Comment 5•9 years ago
|
||
Attachment #8415646 -
Attachment is obsolete: true
Assignee | ||
Comment 6•9 years ago
|
||
Attachment #8416596 -
Attachment is obsolete: true
Assignee | ||
Updated•9 years ago
|
Attachment #8416598 -
Flags: review?(dao)
Comment 7•9 years ago
|
||
Comment on attachment 8416598 [details] [diff] [review] Disable animation on Linux, v2.1 >--- a/toolkit/content/jar.mn >+++ b/toolkit/content/jar.mn >@@ -69,17 +69,17 @@ toolkit.jar: > *+ content/global/bindings/findbar.xml (widgets/findbar.xml) > content/global/bindings/general.xml (widgets/general.xml) > content/global/bindings/groupbox.xml (widgets/groupbox.xml) > *+ content/global/bindings/listbox.xml (widgets/listbox.xml) > content/global/bindings/menu.xml (widgets/menu.xml) > content/global/bindings/menulist.xml (widgets/menulist.xml) > content/global/bindings/notification.xml (widgets/notification.xml) > content/global/bindings/numberbox.xml (widgets/numberbox.xml) >- content/global/bindings/popup.xml (widgets/popup.xml) >+* content/global/bindings/popup.xml (widgets/popup.xml) This shouldn't be needed anymore, you're not touching popup.xml in this patch.
Attachment #8416598 -
Flags: review?(dao) → review+
Updated•9 years ago
|
QA Contact: catalin.varga
Assignee | ||
Comment 8•9 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/77aff162aa0c
Comment 9•9 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/77aff162aa0c
Status: NEW → RESOLVED
Closed: 9 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla32
Comment 10•9 years ago
|
||
Verified as fixed using the following environment: Environment: FF 32 Build id:20140507030202 OS: Ubuntu 13.04 x64 The arrow panel animation is disabled, however after this fix I found the following 2 bugs: bug 1007166 and bug 1007188 that appear to be related with this fix.
Status: RESOLVED → VERIFIED
Whiteboard: p=3 s=it-32c-31a-30b.1 [qa+] → p=3 s=it-32c-31a-30b.1 [qa!]
Assignee | ||
Comment 12•9 years ago
|
||
Comment on attachment 8416598 [details] [diff] [review] Disable animation on Linux, v2.1 [Approval Request Comment] Bug caused by (feature/regressing bug #): 610545 User impact if declined: doorhangers don't work for many linux users Testing completed (on m-c, etc.): yes Risk to taking this patch (and alternatives if risky): none String or IDL/UUID changes made by this patch: none
Attachment #8416598 -
Flags: approval-mozilla-aurora?
Flags: needinfo?(enndeakin)
Updated•9 years ago
|
status-firefox31:
--- → affected
status-firefox32:
--- → fixed
Updated•9 years ago
|
Attachment #8416598 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Assignee | ||
Comment 13•9 years ago
|
||
https://hg.mozilla.org/releases/mozilla-aurora/rev/2c1d788a824b
You need to log in
before you can comment on or make changes to this bug.
Description
•