----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/6152/#review9518 -----------------------------------------------------------
Ship it! LGTM Committed as r1366196, - Yuri Zelikov On July 26, 2012, 11:11 a.m., Vicente J. Ruiz Jurado wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/6152/ > ----------------------------------------------------------- > > (Updated July 26, 2012, 11:11 a.m.) > > > Review request for wave, Michael MacFadden, Yuri Zelikov, and Ali Lown. > > > Description > ------- > > This patch ask for a confirmation before delete a blip. You can avoid the > confirmation dialog pressing the shift key. > > This patch is motivated because some people delete blips accidentally and we > don't have currently a way to restore them. > > > Diffs > ----- > > src/org/waveprotocol/wave/client/wavepanel/impl/menu/MenuController.java > 70af3fa > > Diff: https://reviews.apache.org/r/6152/diff/ > > > Testing > ------- > > delete of blips using "ant waveharness-hosted" > > > Thanks, > > Vicente J. Ruiz Jurado > >
