----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5180/#review9983 -----------------------------------------------------------
I investigated the issue https://issues.apache.org/jira/browse/WAVE-362 and found that it was caused by this patch. I think it would be better to revert the changes for now and resubmit later after fixing. - Yuri Zelikov On May 21, 2012, 6:21 p.m., Vicente J. Ruiz Jurado wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/5180/ > ----------------------------------------------------------- > > (Updated May 21, 2012, 6:21 p.m.) > > > Review request for wave, Michael MacFadden, Yuri Zelikov, and Ali Lown. > > > Description > ------- > > With this patch blip menu items reacts to mouse clicks instead of mouse down > events (similar behaviour than the toolbar buttons). > > (This feedback comes from a usability expert) > > > Diffs > ----- > > src/org/waveprotocol/wave/client/wavepanel/impl/menu/MenuController.java > 99b8fac > > Diff: https://reviews.apache.org/r/5180/diff/ > > > Testing > ------- > > Some tests with "ant waveharness-hosted" > > > Thanks, > > Vicente J. Ruiz Jurado > >
