LGTM! Thanks for the quick fix. Feel free to remove the extra log statements as you mentioned. My only nitpick is that you could make a small utility to merge the messagebar and logmessages if the message is the same.

Question: I presume test.py can be deleted since the code it tests is no longer present; does anything else test-related need to be changed/removed?

make a small utility to merge the messagebar and logmessages if the message is the same

I'll include this in my push.

Good point, test.py can indeed be deleted.

Just pushed the changes we've been discussing (414ec89). Code was tested and one [unrelated] issue was seen and addressed, see below. This code should be ready to merge.

The issue that I encountered was that if one requested watching of a mix of watchable and unwatchable layers then no watchable selected layers after the first unwatchable one would be added. This was fixed by changing return to continue in the relevant tests (i.e. on encountering an unwatchable layer the code won't continue the current iteration of the loop but will continue with the next layer in loop).

Nice find, merged. I'll make a new release. :)

I'll make a new release.

Before you do I have a bunch of minor changes I'd like to push (mostly documentation changes IIRC). I'll push them shortly.

Pull request #10 includes the suggested changes I mentioned in my previous post.