> On June 25, 2012, 6:54 p.m., Yuri Zelikov wrote: > > src/org/waveprotocol/wave/client/wavepanel/view/dom/full/ContinuationIndicator.css, > > line 62 > > <https://reviews.apache.org/r/5426/diff/1/?file=112259#file112259line62> > > > > I think that displaying only upper part of the "unknown" participant > > image doesn't look good. It looks a bit weird to me. > > Vicente J. Ruiz Jurado wrote: > Can you test the new patch? Also you can test it in action in > http://kune.cc. > > Jeremy Naegel wrote: > I've tested it on kune.cc and I think it makes the blue line inline reply > more visible and clearer to understand for newcomers. > > Could it display (scaled down) the actual user avatar instead of the > unknown one? IMHO that would make its function even clearer. > What about adding the "Click here to reply" text in the blip preview? It > would make it match the reply box at the bottom of waves.
Thanks for the feedback, I agree that both suggestions can improve more the use. We can try to implement it in the future. - Vicente J. ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5426/#review8555 ----------------------------------------------------------- On June 25, 2012, 7:36 p.m., Vicente J. Ruiz Jurado wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/5426/ > ----------------------------------------------------------- > > (Updated June 25, 2012, 7:36 p.m.) > > > Review request for wave, Michael MacFadden, Yuri Zelikov, and Ali Lown. > > > Description > ------- > > Few people are using the inline reply (the on over blue border) compared to > the use of the "Reply" button. > > This small patch improves the visibility and usability of the inline reply > and I think that encourage people to use it. > > > Diffs > ----- > > > src/org/waveprotocol/wave/client/wavepanel/view/dom/full/ContinuationIndicator.css > c2bc9a9 > > Diff: https://reviews.apache.org/r/5426/diff/ > > > Testing > ------- > > Tested with "ant waveharness-hosted". > > > Thanks, > > Vicente J. Ruiz Jurado > >
