<div dir="ltr">If the user hasn't made any changes and now it displays "Add location" "close".<br><br>Is there anything special that should happen when the choose "Add location"?<br><br>If now why it has to change when the user make changes, we can keep "Add location"<br>instead of "Save and close"?<br><br>Let me know what you think.<br><br>§Yousef<br><br></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Mar 9, 2015 at 5:57 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"><span class="">On Mon, Mar 09, 2015 at 07:45:31AM +0200, Yosef Hamza wrote:<br>
> The fix is according to the comment on the ticket.<br>
<br>
</span>- your patch was attached as base64 encoded attachment. Nothing horribly<br>
  wrong with that, but it's inconvenient as it makes it hard to comment on<br>
  the code in a response.<br>
- changing the names (and the order) of the buttons isn't exactly a fix<br>
  for the issue -- what needs to happen is that the message bar is shown<br>
  with actions that make sense:<br>
  -- Add location / close (if the user hasn't made any changes)<br>
  -- Save and close / Discard and close (if the user has made changes or<br>
     selected to add a site)<br>
<br>
For an "easy fix" ticket I'd be happy with a solution that ignores the<br>
"add" part of this. So make the message bar show "Close" by default and<br>
Save / Discard once the user starts editing.<br>
<span class="HOEnZb"><font color="#888888"><br>
/D<br>
</font></span></blockquote></div><br></div>