After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 773864 - Flashing format combo value on mouse-wheel scroll
Flashing format combo value on mouse-wheel scroll
Status: RESOLVED FIXED
Product: evolution
Classification: Applications
Component: Composer
3.22.x (obsolete)
Other Linux
: Normal normal
: ---
Assigned To: Tomas Popela
Evolution QA team
Depends on:
Blocks:
 
 
Reported: 2016-11-02 22:40 UTC by Milan Crha
Modified: 2016-11-04 08:00 UTC
See Also:
GNOME target: ---
GNOME version: ---


Description Milan Crha 2016-11-02 22:40:34 UTC 
I just noticed that when I scroll with a mouse wheel and the text contains a paragraph style changes, then these are flashing in the combo box. That indicates a performance issue, unnecessary UI updates when the background code tries to do something after scroll (I guess it's spell-checking, but it's only a wild guess). The signals about "cursor position changes" shouldn't be propagated in the situations which are not initiated by the user (aka when the cursor returns back to the "saved position" after some processing.
Comment 1 Tomas Popela 2016-11-03 10:05:22 UTC 
I don't think that it will be the spell-checking as it is already doing something to prevent that - https://git.gnome.org/browse/evolution/tree/src/modules/webkit-editor/web-extension/e-editor-dom-functions.c#n869 . But there is also a chance that it is not working properly.
Comment 2 Tomas Popela 2016-11-04 08:00:59 UTC 
Fixed with the following commits:

    Bug 773864 - Flashing format combo value on mouse-wheel scroll
    
    Restore the selection position on the end of the spell-check refresh
    before unblocking the SelectionChanged callbacks as otherwise the last
    selection from doing the refresh will be used as well.
    
commit 28f1790 in the master branch for Evolution 3.23.2+
commit c38801a in the gnome-3-22 for Evolution 3.22.2+