Bug 332869

Summary: System tray animates when opened from a different tab
Product: [Plasma] plasmashell Reporter: David Edmundson <kde>
Component: System TrayAssignee: David Edmundson <kde>
Status: RESOLVED FIXED    
Severity: normal CC: bhush94, notmart
Priority: NOR    
Version: master   
Target Milestone: 1.0   
Platform: unspecified   
OS: Linux   
Latest Commit: Version Fixed In:
Bug Depends on: 332884    
Bug Blocks:    
Attachments: Image dump from apitrace

Description David Edmundson 2014-03-31 16:01:02 UTC
If I click on the battery applet in the system tray when the expanded view appears I can see it animate a fade from "Status and Notifications"
Comment 1 David Edmundson 2014-03-31 16:27:16 UTC
It looks like there is code trying to solve this.
There is a timer  which disables animation for the first 500ms for the page stack. This is somewhat hacky and doesn't always work

More importantly this doesn't include the header.

I'll assign this to me.
Comment 2 David Edmundson 2014-03-31 16:27:43 UTC
Created attachment 85872 [details]
Image dump from apitrace
Comment 3 David Edmundson 2014-03-31 20:51:15 UTC
I fixed the header (will push after split settles down, please poke me \)
 
Page stack still has a bug in it, opened with a test case.
Comment 4 Bhushan Shah 2014-04-03 12:16:08 UTC
*poke*
Comment 5 David Edmundson 2014-04-03 12:56:20 UTC
Oh. I pushed my part of the patch. I can't close this whilst there's a blocking bug.
Comment 6 Marco Martin 2014-04-03 16:08:17 UTC
Git commit 2f10450c179bc13e224784099911b723ed02b1ef by Marco Martin.
Committed on 03/04/2014 at 16:07.
Pushed by mart into branch 'master'.

make the animate property do something
Related: bug 332884

M  +3    -3    src/declarativeimports/plasmacomponents/qml/PageStack.qml

http://commits.kde.org/plasma-framework/2f10450c179bc13e224784099911b723ed02b1ef
Comment 7 Marco Martin 2014-04-07 16:50:51 UTC
Not reproduceable anymore after last commit