Code review comment for lp:~muktupavels/compiz/remove-mate-integration
Revision history for this message
| Alberts Muktupāvels (muktupavels) wrote | # |
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
67d34a1
(Get the code!)
I think there could be multiple options:
- duplicate also gsettings backend as gsettings-mate, but that does not sound like good thing to just add MATE integration.
- Link MATE integration with gsettings backend and choose integration based on current desktop environment.
- Change integration setting in configuration from boolean to string or enum - None, GNOME and MATE. And if backend supports integration (ini backend does not) then it could return list with valid integrations that user can choose from in ccsm. Of course default provided by config.
Third option in my opinion could be best thing and second might work, but does not sound that good. Anyway MATE configuration currently has integration = false and I am not going to do this work... If anyone decides to do this work this can be reverted if needed.