search

respec / solar-suitability-app

Solar Suitability Application
4 stars 3 forks source link

Reset report (site details) if new query is run #63

Closed cmartin616 closed 8 years ago

cmartin616 commented 8 years ago

Have to determine the best way to check for a new query -or- add a button to reset the report. Thoughts? For example, site title is 'Chris' House' but then I move onto 'Andy's House' - do I want it to show the stored values 'Chris' House' or 'Site Title'?

a) Last Lat/Lng for a click stored. If Lat/Long when opening report =/= last Lat/Lng, reset details. May be an issue if they just adjust their query point to a different point on the roof.

b) Always leave details unless they reset it.

andywalz commented 8 years ago

I like option b or maybe auto reset if distance between last/current point is greater than 1/4 mi

Sent from my iPhone

On Dec 19, 2015, at 11:41 AM, Chris Martin notifications@github.com wrote:

Have to determine the best way to check for a new query -or- add a button to reset the report. Thoughts? For example, site title is 'Chris' House' but then I move onto 'Andy's House' - do I want it to show the stored values 'Chris' House' or 'Site Title'?

a) Last Lat/Lng for a click stored. If Lat/Long when opening report =/= last Lat/Lng, reset details. May be an issue if they just adjust their query point to a different point on the roof.

b) Always leave details unless they reset it.

— Reply to this email directly or view it on GitHub.

andywalz commented 8 years ago

Maybe a reset button in edit under form fields or little icon next to customize button a good idea too. Whatever you think.

Sent from my iPhone

On Dec 19, 2015, at 11:41 AM, Chris Martin notifications@github.com wrote:

Have to determine the best way to check for a new query -or- add a button to reset the report. Thoughts? For example, site title is 'Chris' House' but then I move onto 'Andy's House' - do I want it to show the stored values 'Chris' House' or 'Site Title'?

a) Last Lat/Lng for a click stored. If Lat/Long when opening report =/= last Lat/Lng, reset details. May be an issue if they just adjust their query point to a different point on the roof.

b) Always leave details unless they reset it.

— Reply to this email directly or view it on GitHub.

cmartin616 commented 8 years ago

I think I might just do nothing with this. What do you think? It would be a huge check to see if anything has changed to show a reset button. And I'm not sure where we should show it anyway.

andywalz commented 8 years ago

Agreed. Let’s do nothing for now, we may be trying to solve a problem that doesn’t exist.

On Dec 20, 2015, at 6:31 PM, Chris Martin notifications@github.com wrote:

I think I might just do nothing with this. What do you think? It would be a huge check to see if anything has changed to show a reset button. And I'm not sure where we should show it anyway.

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