Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Reset button national dashboard #137

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Reset button national dashboard #137

wants to merge 4 commits into from

Conversation

kynan
Copy link
Member

@kynan kynan commented Oct 13, 2014

Add small reset button on the national dashboard that simply resets back the the national view

@kynan
Copy link
Member

kynan commented Oct 13, 2014

I'm happy with this. @dgorissen?

@@ -115,13 +128,6 @@ angular.module('taarifaWaterpointsApp')
##############
### LAYERS ###
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Niggle, this section "header" no longer really reflects the contents. Worth rewording/rejigging slightly?

@dgorissen
Copy link
Member Author

Added a couple comments, else good for me

@kynan
Copy link
Member

kynan commented Oct 13, 2014

@dgorissen Thanks for the thorough check, had missed those points.

@kynan
Copy link
Member

kynan commented Nov 18, 2014

Seems @machakux is no longer available, so it'll be down to one of us to get this in shape.

@kynan
Copy link
Member

kynan commented Sep 21, 2015

@dgorissen Is this still relevant?

@dgorissen
Copy link
Member Author

This would be useful but again, depends on where things will go strategically.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants