<div dir="ltr">I was looking at the new change screen on <a href="https://review.openstack.org">https://review.openstack.org</a> today[1] and it seems to do something vaguely similar.<div><br></div><div><div>Rather than saying "James polley made 4 inline comments", the contents of the comments are shown, along with a link to the file so you can see the context.</div><div><br></div><div>Have you seen this? It seems fairly similar to what you're wanting. </div><div><br></div><div>Have</div><div>[1] To activate it, go to <a href="https://review.openstack.org/#/settings/preferences">https://review.openstack.org/#/settings/preferences</a> and set "Change view" to "New Screen", then look at a change screen (such as <a href="https://review.openstack.org/#/c/127283/">https://review.openstack.org/#/c/127283/</a>)</div></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Dec 16, 2014 at 4:45 PM, Jeremy Stanley <span dir="ltr"><<a href="mailto:fungi@yuggoth.org" target="_blank">fungi@yuggoth.org</a>></span> wrote:<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 2014-12-16 17:19:55 +0200 (+0200), Radoslav Gerganov wrote:<br>
> We don't need GoogleAppEngine if we decide that this is useful. We<br>
> simply need to put the html page which renders the view on<br>
> <a href="https://review.openstack.org" target="_blank">https://review.openstack.org</a>. It is all javascript which talks<br>
> asynchronously to the Gerrit backend.<br>
><br>
> I am using GAE to simply illustrate the idea without having to<br>
> spin up an entire Gerrit server.<br>
<br>
</span>That makes a lot more sense--thanks for the clarification!<br>
<span class=""><br>
> I guess I can also submit a patch to the infra project and see how<br>
> this works on <a href="https://review-dev.openstack.org" target="_blank">https://review-dev.openstack.org</a> if you want.<br>
<br>
</span>If there's a general desire from the developer community for it,<br>
then that's probably the next step. However, ultimately this seems<br>
like something better suited as an upstream feature request for<br>
Gerrit (there may even already be thread-oriented improvements in<br>
the works for the new change screen--I haven't kept up with their<br>
progress lately).<br>
<span class="HOEnZb"><font color="#888888">--<br>
Jeremy Stanley<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
_______________________________________________<br>
OpenStack-dev mailing list<br>
<a href="mailto:OpenStack-dev@lists.openstack.org">OpenStack-dev@lists.openstack.org</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</div></div></blockquote></div></div>