search

respec / solar-suitability-app

Solar Suitability Application
4 stars 3 forks source link

delete feature button should hide once features deleted #59

Closed andywalz closed 8 years ago

andywalz commented 8 years ago

is this hard to do? not burning priority...

andywalz commented 8 years ago

I fixed this but my solution may not be correct. I see edit toolbar code commented out but not sure why. Let's discuss this weekend.

cmartin616 commented 8 years ago

Hey,

Editor code is commented out because we aren't using a editing toolbar. I left it in case the powers that be demand we offer all the drawing tools rather than polygons only.

I haven't gotten a chance to check your solution but I would have the button disabled at first. After draw end, if number of graphics is greater than zero, enable button. After pressing delete graphics, disable button.

That should cover the cases, right?

Chris On Dec 16, 2015 7:47 AM, "Andy Walz" notifications@github.com wrote:

I fixed this but my solution may not be correct. I see edit toolbar code commented out but not sure why. Let's discuss this weekend.

— Reply to this email directly or view it on GitHub https://github.com/flatrockgeo/solar-suitability-app/issues/59#issuecomment-165151363 .