I think that most of the time, you'd be using the same scope, icon, etc for a set of regions, so it makes sense to put that in the constructor. You can still override that in set().
Notes:
New ST4 stuff not yet included.
View.get_regions() doesn't give any details, so the only way to test the args is by mocking View. I have not yet done this.
Rather than accepting a flags argument, maybe we should break this out by flag. That especially makes more sense when overriding defaults in set().
For #141.
I think that most of the time, you'd be using the same scope, icon, etc for a set of regions, so it makes sense to put that in the constructor. You can still override that in
set()
.Notes:
View.get_regions()
doesn't give any details, so the only way to test the args is by mocking View. I have not yet done this.flags
argument, maybe we should break this out by flag. That especially makes more sense when overriding defaults inset()
.