Code review comment for lp:~gilir/xpad/xpad-panel-icon

Revision history for this message
Arthur Borsboom (arthurborsboom) wrote :

https://code.launchpad.net/~gilir/xpad/xpad-panel-icon/+merge/119710

* Reason is feature request: https://bugs.launchpad.net/xpad/+bug/907705
* Proposed branch: lp:~gilir/xpad/xpad-panel-icon

Expected result(s)
If a file called xpad-panel.svg is added to the current Icon Theme,
this icon should be shown in the systray. The regular icon of the
application, should not change. Other behaviour of the application
should not change.

Specific tests regarding the change: works
Regression tests:
- Create new note: works
- Delete note: works
- Save note: works
- Tray shows: works
- Tray hides: works
- Quit application: works
- Show all: works
- Close all: works
- Select specific note from tray: works
- Preferences, font change: works
- Preferences, save font change: works

Code quality: fine.
Review: ok

Ready to be merged.

With kind regards,
Arthur Borsboom
------------------------

« Back to merge proposal