<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, Jul 18, 2016 at 1:56 PM, Gregory Szorc <span dir="ltr"><<a href="mailto:gps@mozilla.com" target="_blank">gps@mozilla.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote"><span class="">On Thu, Jul 14, 2016 at 1:54 AM, Gijs Kruitbosch <span dir="ltr"><<a href="mailto:gijskruitbosch@gmail.com" target="_blank">gijskruitbosch@gmail.com</a>></span> wrote:</span><br><span class=""><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
***<br>
Just generally, it feels like mozreview is full of loading indicators. Everywhere. Even things that should just be instantaneous, like if you click "Finish review..." first you get a "Loading..." blob, before the dialog shows up. When the dialog shows up, then *that* has *another* loading blob while it's collecting all the issues. The dialog should just show up immediately - all the data it initially shows is already in the page that's "below" the dialog, so I have no idea what it's loading that it needed to wait for... I could sympathise with it having to fetch issues from the network (the second "loading" blob in the dialog) but even there it feels like it should have an optimistic cache of the data locally so that you don't feel like you're. waiting. all. the. time. Then if you click "Close" the dialog doesn't dismiss immediately, you get some kind of slow fade-from-black animation. If you actually submit a review, that gets you another "Loading..." blob but the button to submit it isn't immediately disabled, so you can click it twice, which leads to "interesting" "HTTP 0" (wat) errors. Generally it feels like all the animation durations should be at least halved if not gotten rid of entirely</blockquote><div><br></div></span><div>I wonder if this is a consequence of you being on a different continent from the server in California...<br></div></div></div></div></blockquote><div><br></div><div>It's not. I work from the Mountain View office, I have a fast computer, and I plug into Ethernet. The spinners in MozReview are really bad. Even small patches go through a few seconds of spinners. For big ones, I might wait 30 seconds or more. Why can't the default view be assembled and cached on the server, with XHRs only used when you ask for more context? The main reason I prefer Splinter is that it's quick and simple. I think most people would be a lot happier if MozReview was looked like Splinter but with smarter diffing and the option to get more context.<br><br></div><div>-Bill<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div></div><div> </div></div></div></div>
<br>_______________________________________________<br>
firefox-dev mailing list<br>
<a href="mailto:firefox-dev@mozilla.org">firefox-dev@mozilla.org</a><br>
<a href="https://mail.mozilla.org/listinfo/firefox-dev" rel="noreferrer" target="_blank">https://mail.mozilla.org/listinfo/firefox-dev</a><br>
<br></blockquote></div><br></div></div>