Summary: | FORMATTING: Calc becomes slow after selecting cells and activating font chooser with previews enabled (summary: comment 29) | ||
---|---|---|---|
Product: | LibreOffice | Reporter: | Ryan Tandy <ryan> |
Component: | Spreadsheet | Assignee: | Not Assigned <libreoffice-bugs> |
Status: | NEW --- | QA Contact: | |
Severity: | normal | ||
Priority: | medium | CC: | barta, cno, fdbugs, gitzu, jmadero.dev, nopower |
Version: | 4.2.0.4 release | Keywords: | bisected, regression |
Hardware: | Other | ||
OS: | Linux (All) | ||
See Also: |
https://bugs.freedesktop.org/show_bug.cgi?id=80429 https://bugs.freedesktop.org/show_bug.cgi?id=80356 |
||
Whiteboard: | BSA bibisected | ||
i915 platform: | i915 features: | ||
Attachments: |
Test file
Video capture from 4.4.0.2 |
Description
Ryan Tandy
2014-04-11 22:22:49 UTC
I can confirm this bug also on LibreOffice 4.2.1.1 under Windows 8.1 and Ubuntu Gnome 13.04 If I may, I'm of the opinion that the problem permeates deeper than just fonts formatting because doing about anything that involves Select > All slows down Calc to the point where it is becoming questionable whether it's really ready for production. Since yagolf confirmed the bug (thanks!), changing from UNCONFIRMED to NEW. This needs to be re-verified with a more recent 4.2 build. Hi Kohei, The bug exists also in 4.2.4 RC2. Try daily build. Release build is not to be used for QA purposes. Even better would be the latest from the 4.2 branch. Confirmed with libreoffice-4.2 daily build: libreoffice-4-2~2014-05-08_00.21.04 Seems to be fixed in master daily build: master~2014-05-07_23.36.18 I will try to find the responsible commit. There is no bibisect repository for 4.3 yet, right? Huh. Weird. I did backport (what I think to be) the fix to 4.2 a week or so ago. As far as Calc is concerned, there isn't much difference between master and 4.2 branches. (In reply to comment #7) > I will try to find the responsible commit. There is no bibisect repository > for 4.3 yet, right? Actually I don't know enough about bibisect to answer this. Sorry. Joel and Bjoern may know. This commit is what I suspected to be the fix. http://cgit.freedesktop.org/libreoffice/core/commit/?h=libreoffice-4-2&id=9585ea8a3f10be60c42b671a07dfafd78f173e52 and it's already in the 4.2 branch. Tested again with: libreoffice-4-2~2014-05-12_10.48.12_LibreOfficeDev_4.2.5.0.0_Linux_x86-64_deb master~2014-05-11_23.54.14_LibreOfficeDev_4.3.0.0.alpha1_Linux_x86-64_deb Actually, I don't think it's fixed, either on libreoffice-4-2 or on master. I don't know why I wrote that before. Easy to reproduce: 1. put some text in the first cell and copy-paste that to some rows and columns (e.g. 50x50) 2. select all, and change the font 3. scroll up and down With style previews enabled, after the font has been changed, scrolling is slow and there's a visible delay while cells are redrawn. With style previews disabled, scrolling is fast. Reading #74555, I really do expect this to be a duplicate of it. It sounds exactly the same. But I checked carefully that I'm testing with a build that includes that change, and I'm definitely still experiencing the problem like I wrote in my previous comment. Ok. Then this is perhaps something else. (In reply to comment #13) > Ok. Then this is perhaps something else. Yes, in versiĆ³n 4.2.5.2 still persists. Another way to reproduce it: Edit->Select All Ctrl-M (Format->Clean Format) Then arrow keys move slowly I hope this help to Kohei Yoshida in some way: How recuperate speed after change font in all cells: Simply go to combo box to change font and next press escape (with nothing selected) Calc then return to full speed! Still in libreoficce 4.3.0 RC1. I suppose it's a flag not checked when select all is applicated, because change font without select cells fix the speed of calc. please take a look at Bug 80429 - Calc slows down when you select all cells and change font it looks a very similar issue which I reproduced under Win7x64 in 4.2.x but not in 4.3.0.1.0+ Build ID: 52613b9b632721ebb5167a4d9529c242a0b907da TinderBox: Win-x86@42, Branch:libreoffice-4-3, Time: 2014-06-23_07:41:56 @gitzu can you retest with a more recent 4.3 build? The original report was against Calc < 4.2.5.x I think this is just OK in 4.2.5 ad 4.3.0 See https://bugs.freedesktop.org/show_bug.cgi?id=80356#c2 Ok. let's mark this as RESOLVED WORKSFORME. feel free to reopen if somebody still reproduces this on recent 4.2.x or 4.3.x builds I want to try if really the bug was corrected in recent versions, but Windows installers in Nightly Builds don't install. Example: libo-43~2014-06-25_00.02.29_LibreOfficeDev_4.3.0.1.0_Win_x86.msi bad news... please open a new report about that 4.3.x install issue (In reply to comment #20) > feel free to reopen if somebody still reproduces this on recent 4.2.x or > 4.3.x builds Reopening. Reproduced with: Version: 4.2.6.0.0+ Build ID: 32eddb3f48fcea0a052401a8a5dc075c7847f1c5 TinderBox: Linux-rpm_deb-x86_64@46-TDF, Branch:libreoffice-4-2, Time: 2014-07-02_08:19:58 Version: 4.3.1.0.0+ Build ID: c482a2f87ef2a38d249c276924e071bbc714a07f TinderBox: Linux-rpm_deb-x86_64@46-TDF, Branch:libreoffice-4-3, Time: 2014-07-01_23:55:39 on Ubuntu 14.04 64-bit. (In reply to comment #19) > See https://bugs.freedesktop.org/show_bug.cgi?id=80356#c2 I agree with that comment. It is better than it was but there is still a noticeable loss of performance after opening the font chooser, when previews are enabled. I think bug 80356 is a duplicate of this one (or vice versa), and both are a continuation of the issue that was partly fixed in bug 74555 and still partly exists. confirm this bug. tested with LibreOffice_4.2.4_Win_x86.msi on windows xp, libreoffice-bin-4.2.3.3 and libreoffice-bin-4.2.6.3 on gentoo linux. disable "Show preview of fonts" solves problem. Needs independent confirmation from QA team - moving to UNCONFIRMED. Thanks all On Linux, Ubuntu, 32 bits 4.2.7.2, 4.3.3.2, 4.4.0.alpha1 font preview activated create a spreadsheet select A1:T20000 or A1:Y20000 open font drop down choose font close drop down, open and chose another front and some more I experience no delay.. On Windows 7, 32 bits 4.3.3.2 font preview activated create a spreadsheet select A1:T20000 *** FILL IT *** with aleatory content open font drop down choose font Navigate... I experience delay.. Created attachment 112513 [details]
Test file
There is still an issue present in at least 4.4.0.2 and 4.5 master which does appear to date back to the range mentioned in comment 0 (which in the 43all bibisect repo is source-hash-bcc51fb2ebdf77a1cc089857775fd742085b45b6). Steps to reproduce: 1. Load the file from attachment 112513 [details] 2. Ensure "Tools - Options - LibreOffice - View - Font Lists - Show preview of fonts" is enabled 3. Select All 4. Change to another font Expected result: - Interface should be as responsive as before (4) - e.g. scrolling, clicking between cells, selecting a cell range Actual result: - Interface is much slower than before. Saving and reloading the file resets the speed to normal Created attachment 112516 [details]
Video capture from 4.4.0.2
The attached video capture shows the observed effect. Note that after font preview is enabled and the font changed, the selection is much less responsive.
This effect is much more pronounced on a dbgutil build of 4.5 master (it hasn't become slower; this just seems to be a side effect of dbgutil)
commit bcc51fb2ebdf77a1cc089857775fd742085b45b6 Building from source confirms that the slowdown started at the below commit Adding Cc: to nopower@novell.com; Could you possibly take a look at this? Thanks Author: Noel Power <noel.power@suse.com> Date: Thu Aug 29 17:15:33 2013 +0100 add support for in-place style preview selecting a style in the styles dialog ( without double clicking ) will apply the style to the currently selected cell(s) You can with the keys navigate to other styles and they in turn will also be applied. Preview will end when you click back onto the document. *FIXME* - the styles dialog isn't really suitable for previewing, a new dialog ( possibly in the toolbar ) might be nicer ( see Excel ) *FIXME* - when there is a multiple selection the highlight colour (applied as a transparent overlay) is most annoying ( and is mixed with any background colour applied if part of a style ) see ( ScGridWindow::UpdateSelectionOverlay() ) However my puny attempts to make the selection use a transparent colour made all the borders of the selected cells dissappear. I guess maybe a box/border around each selected cell ( or group of cells ) would also work but I didn't try that Change-Id: I0950e79085ffb75f60ee961835665df0c230172f |
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.