<div dir="ltr">All right, I wrote all of these up, and I'll be making a fork to address these issues: 53, 104, and 115 through 121.  Assuming none of these are very complicated (except maybe 53), should I just lump these all together into one pull request?  Or make separate pull requests per issue?<div>

<br></div><div>Thanks,</div><div>Sergey</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Sat, Aug 31, 2013 at 7:39 PM,  <span dir="ltr"><<a href="mailto:justin@postgresql.org" target="_blank">justin@postgresql.org</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On 2013-09-01 07:55, Sergey Krilov wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hello,<br>
<br>
I came across BrowserQuest a few days ago and thought I might try<br>
contributing something to it.  I need some practice with JavaScript<br>
and also wanted to learn Node.js, and this seems like a great way to<br>
do it.<br>
</blockquote>
<br></div>
Welcome aboard Sergey. :)<br>
<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I was going to start with something simple, and thought issue #53 [1]<div class="im"><br>
might be a good candidate (show a more helpful error message when<br>
logging in with an incorrect password). However, after looking into it<br>
a bit more, I think the issue runs a bit deeper than just the<br>
unhelpful error message - the fact is, if the username/password are<br></div>
not correct, were still effectively launching a non-functional game<div class="im"><br>
with a broken UI, and no possibility of navigating back to the login<br>
form without refreshing the browser.  I think a proper solution here<br>
would be, in client/js/main.js, before calling app.tryStartingGame, to<br>
make a separate call to the server to check the username/password.<br>
 If that call returns that the username/password are not correct,<br>
then we show an error message right within the login form, without<br>
navigating away.  This should result in a more intuitive behavior for<br>
the user, but would require a bit more effort.<br>
</div></blockquote>
<br>
That definitely makes sense.  Code up the initial solution, to the<br>
point you're happy with it, then make a Pull Request of it. :)<br>
<br>
Someone will look over it reasonably soon + test it out (same day<br>
generally), then merge it if it's all good.<br>
<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">
Besides this, I noticed there were a handful of other issues with<br></div>
login and new character creation which werent documented yet.  Im<div class="im"><br>
writing these up at the bottom of this email.  I wanted to get<br></div>
everyones thoughts on all of these before I start coding away - let me<br>
know if anything that Im proposing sounds like it might be a bad idea.<div class="im"><br>
<br>
<br>
Also, do you prefer that I write these up as actual issues in the main<br>
repo?<br>
</div></blockquote>
<br>
Any time you spot a bug or other er... wrongness, please create an<br>
issue for it on the browserquest/BrowserQuest repo, one issue per<br>
seperate thing:<br>
<br>
  <a href="https://github.com/browserquest/BrowserQuest/" target="_blank">https://github.com/<u></u>browserquest/BrowserQuest/</a><br>
<br>
Yeah, I know it can be a bit tedious to do 1 each... but it makes<br>
discussing and coding for specific issue easier. :)<br>
<br>
<br>
<snip><div class="im"><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Issue A: Logging in with incorrect password is not intuitive.<br>
</blockquote>
<br></div>
Hmmm... any chance you can make the new ones of these into issues<br>
on the GitHub repo?  If you can't be bothered, I'll do it.  (I'm<br>
just attempting to be lazy here :>)<br>
<br>
Btw, if you want to prioritise... my vote is for "Issue E: 'a' and<br>
'f' keys do not work in login form", sooner rather than later.<br>
<br>
It's straight out annoying (every time I go to log in...) :)<br>
<br>
Regards and best wishes,<br>
<br>
Justin Clift<br>
<br>
______________________________<u></u>_________________<br>
Browserquest mailing list<br>
<a href="mailto:Browserquest@mozilla.org" target="_blank">Browserquest@mozilla.org</a><br>
<a href="https://mail.mozilla.org/listinfo/browserquest" target="_blank">https://mail.mozilla.org/<u></u>listinfo/browserquest</a><br>
</blockquote></div><br></div>