<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Mar 9, 2015 at 9:05 PM, Dirk Hohndel <span dir="ltr"><<a href="mailto:dirk@hohndel.org" target="_blank">dirk@hohndel.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Yosef,<br>
<br>
please try to figure out how to answer inline. This is a technical mailing<br>
list where top posting is considered quite annoying. The occasional email<br>
from a phone may be acceptable, but in general we expect people to use in<br>
line responses...<br></blockquote><div>Sorry about that...</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span><br>
On Mon, Mar 09, 2015 at 07:01:09PM +0200, Yosef Hamza wrote:<br>
> If the user hasn't made any changes and now it displays "Add location"<br>
> "close".<br>
><br>
> Is there anything special that should happen when the choose "Add location"?<br>
<br>
</span>As I said, that exceeds the "easy fix" part of this. Yes, we should add<br>
the code that allows people to create a new location. But that's a bit<br>
more complicated. For the first patch I'll be happy with something that<br>
just has a close button there by default.</blockquote><div> </div><div>I don't quite get it, Should I create a new location with some default values</div><div>since the form is empty (user hasn't made any changes to it)?</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"> <br></blockquote><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span>
> If now why it has to change when the user make changes, we can keep "Add<br>
> location"<br>
> instead of "Save and close"?<br>
<br>
</span>I can't parse that sentence. Can you try again<br>
<br></blockquote><div>I meant we can just keep the action text to "Add location" instead of changing it to</div><div>"Save and close" and only change "Close" to "Discard and close" when the user</div><div>make some changes.<br></div><div><br></div><div>§Yousef</div><div><br></div><div><br></div></div><br></div></div>