Skip to content
This repository has been archived by the owner on Nov 10, 2017. It is now read-only.

Fix issue 16: Don't trigger delete requests when pressing ENTER elsewhere on the form #17

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

graememcc
Copy link
Contributor

This change also decouples "Add User" from the rest of the form - I don't think it's obvious that hitting enter there will also submit any modifications you have made to existing users.

@edmorley
Copy link
Contributor

Catlee, don't suppose you could take a look at this, so we can close this out (and get closer to having 0 open issues, so can switch them off)?

@catlee
Copy link
Contributor

catlee commented Apr 5, 2013

Should we close this out in favour of ewong's work in PR 39?

@edmorley
Copy link
Contributor

edmorley commented Apr 5, 2013

This is about users, whereas pull request 39 is about trees, so they're unrelated aiui.

@catlee
Copy link
Contributor

catlee commented Apr 5, 2013

Oh yes, you're right.

On Fri, Apr 5, 2013 at 10:42 AM, Ed Morley notifications@github.com wrote:

This is about users, whereas pull request 39 is about trees, so they're
unrelated aiui.


Reply to this email directly or view it on GitHubhttps://github.com//pull/17#issuecomment-15959513
.

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

Successfully merging this pull request may close these issues.

3 participants