When clicking an inactive feature in direct_select, select it

[trygveaa]

With this, you don't have to click twice to select another feature when
you are in direct_select.

The mode will still be changed to simple_select. So if you want to
activate direct_select for another feature, you have to click it twice,
but that's still one less click than before.

[usb248]

Good ! But cursor pointer (hover) on inactive feature is missing @trygveaa

[trygveaa]

@usb248: Ah, right. I fixed it now.

[adrianababakanian]

Thanks for opening @trygveaa -- this is working well. I'd advise a small update to the test case before merging.

[adrianababakanian]

Since a selection change is occurring, it is worth checking that the correct event is fired.

Suggested change

	afterNextRender(() => {
	click(map, makeMouseEvent(clickAt[0], clickAt[1]));
	afterNextRender(() => {
	t.equal(Draw.getSelectedIds().indexOf(polygonId) !== -1, true, 'polygon is now selected');
	cleanUp(() => st.end());
	});
	});
	afterNextRender(() => {
	click(map, makeMouseEvent(clickAt[0], clickAt[1]));
	map.fire.resetHistory();
	afterNextRender(() => {
	const args = getFireArgs().filter(arg => arg[0] === 'draw.selectionchange');
	t.equal(args.length, 1, 'should have one and only one selectionchange event');
	t.equal(Draw.getSelectedIds().indexOf(polygonId) !== -1, true, 'polygon is now selected');
	cleanUp(() => st.end());
	});
	});

[trygveaa]

Good idea, applied.