Hi,<br><br><div class="gmail_quote">On 6 September 2011 04:36, Jon Stahl <span dir="ltr"><<a href="mailto:jonstahl@gmail.com">jonstahl@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">

<div><div></div><div class="h5"><br>
</div></div>This is really excellent.  Here are my initial bits of feedback, in no<br>
particular order:<br>
<br>
1) When saving a file, I'm returned to a blank screen.  I expected to<br>
be returned to the file I am editing.<br></blockquote><div><br></div><div>Yeah - it reload the whole page and doesn't remember which file you have open. I think that's fixable, though.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">


2) The "dirty" asterisk seems to get set when you simply load a file<br>
in the editor, before you even make any changes.  I expected it to<br>
only be set when I actually make a change to the file.<br></blockquote><div><br></div><div>It fetches the content and puts it in a hidden textarea when you select it. I think we could probably mark that dirty or not on first edit, it's just a bit more fiddly. I don't want to submit a changed field if it's not there.</div>

<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
3) I was surprised to not be able to edit existing third-party themes<br>
I'd loaded into my test site.  I'd think that tweaking an existing<br>
canned theme would be one of the most-requested use-cases for this<br>
capability.<br></blockquote><div><br></div><div>You should able to edit anything that's in the ZODB (but not on the filesystem). Can you give me a reproducible test case?</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">


4) Following on the previous point... I do think we should probably<br>
try to do full create/delete/rename/upload (maybe not copy/move) file<br>
support for at least images, CSS and JS.  Think if I want to use an<br>
existing theme, but add my own logo.  :-)<br></blockquote><div><br></div><div>Agree. I just don't want to build the Finder into this. ;-) I wonder if there's some kind of widget or UI we can reuse.</div><div> </div>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
5) If anyone ever gets around to properly tackling the "banjo" concept<br>
for point-and-click Diazo rule building, this control panel would be<br>
the obvious place to bolt it on. ;-)<br></blockquote><div><br></div><div>Indeed. I don't want to do that now, but this is one step in that direction.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">


6) I wanted to be able to rename my finished theme from "Jon Test"<br></blockquote><div><br></div><div>We could probably support this. Changing the title is easy (edit manifest.cfg). Changing the id a bit more fiddly.</div>

<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
7) I wasn't sure what commands are available in the editor; I<br>
discovered <a href="https://github.com/ajaxorg/ace/wiki/Default-Keyboard-Shortcuts" target="_blank">https://github.com/ajaxorg/ace/wiki/Default-Keyboard-Shortcuts</a>,<br>
but wished there was a link to it somewhere (or its contents were<br>
reproduced).<br></blockquote><div><br></div><div>Good point.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
More as I think of it.  I think folks are going to LOVE this.  I'd<br>
love to see this approach used for all our "TTW code editing" features<br>
in the future.</blockquote><div><br></div><div>Certainly it can be generalised.</div><div><br></div><div>Martin </div></div>