NREL / floorspace.js

Other
66 stars 35 forks source link

Confusing filled area for some polygons #281

Closed shorowit closed 6 years ago

shorowit commented 6 years ago

Two examples below. The second one doesn't bother me as much, but the first one is quite confusing. FloorspaceJS.webm.txt FloorspaceJS2.webm.txt

macumber commented 6 years ago

@shorowit personally I don't find these confusing at all

@DavidGoldwasser @ljbrackney if you download these webm videos and watch them do you find the presentation of polygon areas before clicking to be confusing?

shorowit commented 6 years ago

This came up in my demo to DOE where I couldn't see the map to trace over it due to this issue. Here's a better example where I can't figure out where the next vertex is supposed to be because it's covered by the polygon:

FloorspaceJS3.webm.txt

ljbrackney commented 6 years ago

I think it might be confusing the first time someone was using the tool, but after you know what it's doing it's not too bad. The third example is problematic - need more transparency in the polygon.

shorowit commented 6 years ago

I’d be fine with increased transparency if that’s easier.

bgschiller commented 6 years ago

Ready for testing at https://325-pr-floorspace.js-NREL.surge.sh

shorowit commented 6 years ago

Much better, thanks!