> On 2012-05-29 17:29:08, Yuri Zelikov wrote:
> > Can we make this configurable by passing a flag to client in ClientFlags?
>
> Vicente J. Ruiz Jurado wrote:
> Yes, but, because I don't know well this part of the code, I'm not sure
> of the differences between be notified of the wave changes or not. It's a way
> to update faster the search digest and not to wait to the search query
> results? your proposal is to make this configurable in server.config also? I
> prefer to find a better fix.
What I mean is to have something like this:
public void activate() {
if (useDynamicDigest) {
// so the activation logic....
}
}
This way we let the admin to decide what he prefers - dynamic digest update or
no flickering in Chrome. Also, I think we should open an issue to actually fix
the flickering instead of avoiding to update the digest with user changes.
- Yuri
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/5257/#review8175
-----------------------------------------------------------
On 2012-05-29 10:48:22, Vicente J. Ruiz Jurado wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/5257/
> -----------------------------------------------------------
>
> (Updated 2012-05-29 10:48:22)
>
>
> Review request for wave and Yuri Zelikov.
>
>
> Summary
> -------
>
> In chrome the avatars flicker while editing (on each key press). Not
> registering the digest to wave changes fix the issue, and seems that the
> search panel updates correctly anyway (on title change, adding/removing
> participants, etc).
>
> I'm not an expert on this part of the code, so probably Yuri have to double
> check this patch.
>
> Related:
> https://kune.cc/?locale=es#kune.lists.1226.1257
>
>
> Diffs
> -----
>
> src/org/waveprotocol/box/webclient/search/SimpleSearch.java 5240554
>
> Diff: https://reviews.apache.org/r/5257/diff
>
>
> Testing
> -------
>
> Normal use of the server in chrome.
>
>
> Thanks,
>
> Vicente J.
>
>
