On Mon, Sep 20, 2010 at 10:10 AM, Adam Barth <[email protected]> wrote:

> On Mon, Sep 20, 2010 at 8:37 AM, Alexey Proskuryakov <[email protected]>
> wrote:
> > 16.09.2010, в 18:39, Darin Fisher написал(а):
> >> Push the publish button to review your comments :-)
> >
> > Alas, not any more!
> >
> > https://bugs.webkit.org/show_bug.cgi?id=46074
>
> Yeah.  The machinery is still there for the preview, I'm just not sure
> what the best UI is for triggering it.
>
> Adam



How about this?

If any annotations were made to the patch, then "the button" gets named
Preview.  Else, the button is named "Publish" and when clicked performs its
work in one shot.

Was there a strong outcry for removing the preview step?  I only found it
bothersome when I wanted to issue a quick r=me on a patch that didn't
require any additional changes.

-Darin
_______________________________________________
webkit-dev mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev

Reply via email to