Merge lp:~tpeeters/ubuntu-ui-toolkit/foregroundIconColorFix into lp:ubuntu-ui-toolkit/staging

[Tim Peeters]

Fix the icon color in the AppHeader.

Fix the icon color in the AppHeader.

I did not spend time on adding a regression test, because this is the header that we are deprecating in 1.3. I verified that for the new PageHeader, the icon color was already correct.

[PS Jenkins bot (ps-jenkins)]

PASSED: Continuous integration, rev:1719
http://jenkins.qa.ubuntu.com/job/ubuntu-sdk-team-ubuntu-ui-toolkit-staging-ci/2503/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/ubuntu-sdk-team-ubuntu-ui-toolkit-staging-vivid-amd64-ci/1231
    SUCCESS: http://jenkins.qa.ubuntu.com/job/ubuntu-sdk-team-ubuntu-ui-toolkit-staging-vivid-armhf-ci/1233
        deb: http://jenkins.qa.ubuntu.com/job/ubuntu-sdk-team-ubuntu-ui-toolkit-staging-vivid-armhf-ci/1233/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/ubuntu-sdk-team-ubuntu-ui-toolkit-staging-vivid-i386-ci/1230

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/ubuntu-sdk-team-ubuntu-ui-toolkit-staging-ci/2503/rebuild

review: Approve (continuous-integration)

[Zsombor Egri (zsombi)]

Thanks for the fix.

review: Approve

=== modified file 'src/Ubuntu/Components/Themes/Ambiance/1.3/PageHeadStyle.qml'
--- src/Ubuntu/Components/Themes/Ambiance/1.3/PageHeadStyle.qml	2015-09-22 14:39:47 +0000
+++ src/Ubuntu/Components/Themes/Ambiance/1.3/PageHeadStyle.qml	2015-11-18 20:17:52 +0000
@@ -403,6 +403,16 @@
 
             actions: headerStyle.config.actions
             numberOfSlots: 3
+
+            delegate: AbstractButton {
+                style: IconButtonStyle { }
+                objectName: action.objectName + "_button"
+                height: parent ? parent.height : undefined
+                action: modelData
+                StyleHints {
+                    foregroundColor: headerStyle.buttonColor
+                }
+            }
         }
     }
 }