manjaro / manjaro-settings-manager

This repo has been archived. Our code is now hosted at
https://gitlab.manjaro.org/
GNU General Public License v3.0
18 stars 19 forks source link

Notifier consolidation #137

Closed vfjpl closed 6 years ago

vfjpl commented 6 years ago

When I was looking at where notifier saves it's setting I wrote some consolidations code and was thinking about writing some module so there would be only one place with configuration. But then I had some different things to do. This patches consolidates code between notifier and notifier_kde.

philmmanjaro commented 6 years ago

@Kirek: code seems fine from my end. I found some time to review it. Maybe we need to apply it to see if there are any regressions ...

Kirek commented 6 years ago

Seems fine. thanks.