move header bar and control manager to c++
ClosedPublic

Authored by astippich on May 9 2018, 7:20 PM.

Details

Summary

next step of moving more classes to c++

Test Plan

headerbar still shows correct data, player controls still work

Diff Detail

Repository
R255 Elisa
Branch
move_control_manager
Lint
No Linters Available
Unit
No Unit Test Coverage
astippich requested review of this revision.May 9 2018, 7:20 PM
astippich created this revision.
mgallien added inline comments.May 16 2018, 6:48 PM
src/qml/ElisaMainWindow.qml
139–144

Please keep that. As far as I know, it is needed to open Elisa from mpris2 protocol.

mgallien requested changes to this revision.May 16 2018, 6:49 PM

Looks good except one issue. Please have a look at it.

This revision now requires changes to proceed.May 16 2018, 6:49 PM
astippich updated this revision to Diff 34319.May 16 2018, 7:34 PM
  • bring back raise player
mgallien accepted this revision.May 16 2018, 7:48 PM
This revision is now accepted and ready to land.May 16 2018, 7:48 PM
astippich updated this revision to Diff 34324.May 16 2018, 8:13 PM
  • rebase on master
This revision was automatically updated to reflect the committed changes.