Bug Hunting Session
Bug 115637 - Conditional Formatting: OK, Cancel button not working (and UI objects misplaced) with Non-English UI
Summary: Conditional Formatting: OK, Cancel button not working (and UI objects misplac...
Status: NEW
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Calc (show other bugs)
(earliest affected)
Hardware: All Linux (All)
: high major
Assignee: Not Assigned
Keywords: bibisected, bisected, regression
Depends on:
Blocks: Dialog Conditional-Formatting-Editing
  Show dependency treegraph
Reported: 2018-02-12 01:53 UTC by Kevin Suo
Modified: 2018-11-29 09:20 UTC (History)
3 users (show)

See Also:
Crash report or crash signature:

screenshot (39.02 KB, image/png)
2018-02-12 01:53 UTC, Kevin Suo

Note You need to log in before you can comment on or make changes to this bug.
Description Kevin Suo 2018-02-12 01:53:28 UTC
Conditional Formatting not working with Simplified Chinese UI.  The OK, Cancel buttons are not working on click. 

I also noticed that the UI elements are misplaced, which may be a different issue.

Steps to Reproduce:
1.Open Calc with Simplified Chinese UI. (I guess this may happen with other non-english UIs)
2. Select some cell ranges, then go to "Format > Conditional Formatting > Condition > (any of the sub-menu)
3. Set (or without setting) some conditions, then try to click OK (确认) or Cancel (取消)

Actual Results:  
No reaction with clicking on OK or Cancel.

Expected Results:
OK: Apply and close window;
Cancel: Close window without applying conditions.

Reproducible: Always

User Profile Reset: Yes

Additional Info:
Build ID:60bfb1526849283ce2491346ed2aa51c465abfe6
CPU 线程:4; 操作系统:Linux 4.14; UI 渲染:默认; VCL: gtk2; 
区域语言:zh-CN (zh_CN.UTF-8); Calc: group

Fedora 27, GNOME Shell 3.26.2.

User-Agent: Mozilla/5.0 (X11; Fedora; Linux x86_64; rv:58.0) Gecko/20100101 Firefox/58.0
Comment 1 Kevin Suo 2018-02-12 01:53:57 UTC
Created attachment 139804 [details]
Comment 2 Kevin Suo 2018-02-12 01:56:38 UTC
Set priority to HIGH MAJOR as the core functionality is broken.

Adding Volga to cc list, if you have time, please help to test and confirm, thanks.
Comment 3 Kevin Suo 2018-02-12 02:07:40 UTC
Of course this is a regress.

Bug also exists in version release.

Bibisecting is not possible due to the fact that none of the bibisect repo provides other langpacks.
Comment 4 Kevin Suo 2018-02-12 02:16:49 UTC
Bug also exits in the KeyID "language".

XZmEs‖Build ID: 4082a18406c18af7b4fcef7bd501c3679c3be56b
Dk4vc‖CPU threads: 2; 5DyEd‖OS: Linux 4.14; Yqrwo‖UI render: ZitRE‖default; VCL: gtk3; 
Ru2AA‖Locale: zh-CN (zh_CN.UTF-8); Calc: group threaded
Comment 5 Kevin Suo 2018-02-12 02:25:47 UTC
Bug already exists on the build I have at
6eeac3539ea4cac32d126c5e24141f262eb5a4d9 libreoffice-6-0-branch-point
Comment 6 Kevin Suo 2018-02-12 04:27:08 UTC
(In reply to Kevin Suo from comment #3)

Noted the for "" release, I mean: 
Build ID:
CPU 线程:2; 操作系统:Linux 4.14; UI 渲染:默认; VCL: gtk3; 
区域语言:zh-CN (zh_CN.UTF-8); Calc: group

This is a build by Fedora. 

In the version I download from TDF, the bug does not exist in and
Comment 7 Kevin Suo 2018-02-12 05:09:09 UTC
libreoffice-6-0-branch-point is at the tag of libreoffice-
Comment 8 Kevin Suo 2018-02-12 08:01:05 UTC
Regression introduced by the following commit:

author	Bernhard Widl <bernhard.widl@cib.de> 2017-10-20 14:34:53 +0200
committer Samuel Mehrbrodt <Samuel.Mehrbrodt@cib.de> 2017-10-26 08:43:10 +0200
commit	cb151051d79a7ec176e769a33a56e7d906fc1425 (patch)
tree	fc6c2890b0cce32924934572569ce10a8ccc418b
parent	08a690cf66d4e798896b61aa5a7a81e81885cd0c (diff)
tdf#113444 removed capping of default font size

Adding Bernhard Widl to the cc list: would you please take a look?


Blow are the bibisect log:

64c0802644ae8f66c9b2479e29dab0b1103ad302 is the first bad commit
commit 64c0802644ae8f66c9b2479e29dab0b1103ad302
Author: Jenkins Build User <tdf@pollux.tdf>
Date:   Sun Oct 29 11:54:01 2017 +0100

    source sha:cb151051d79a7ec176e769a33a56e7d906fc1425
    source sha:cb151051d79a7ec176e769a33a56e7d906fc1425

:040000 040000 207569338305669788fdd2bba19f8ff03fb2e47a c8ed6c7bf6d5de72a427a61d1f08ad00ee2badb7 M	instdir

[suokunlong@x230i bibisect-linux-64-6.0]$ git bisect log
# bad: [cddc6665c4e33f5a18e7e1a02cd2799d95653f0a] source sha:47cc374c0659fd3db74a1b184c870eaa56bc385b
# good: [b9d78ce81dc3481fccb0cb75d76fcb6ac939cbd5] source sha:9feb7f7039a3b59974cbf266922177e961a52dd1
git bisect start 'master' 'oldest'
# good: [eb6680d85befc418b36f2cb00402f56a2472758f] source sha:c54850b23a8240a41755af171a6d3f990ee69f84
git bisect good eb6680d85befc418b36f2cb00402f56a2472758f
# good: [3fa1ae98ece9b3537df34742d3383aac834caf76] source sha:87c671d188f4a0f5dcc7944b450cc58e84217d81
git bisect good 3fa1ae98ece9b3537df34742d3383aac834caf76
# bad: [9bc1a6bd3e4aafadd3a19ac4983952db039716f9] source sha:bff8d843bd4e5dcca5dc1a60c2c7852b1b72a00b
git bisect bad 9bc1a6bd3e4aafadd3a19ac4983952db039716f9
# bad: [a953b1c0692258b21302b14bc5d7e624c24ba880] source sha:db378d64a9ca8675066fda31f3bd77dc37c52398
git bisect bad a953b1c0692258b21302b14bc5d7e624c24ba880
# bad: [7c316d96affb607e199132c13bd140e29187ed59] source sha:7e87403953dfd54bf1e904ccc7436c6f327a9069
git bisect bad 7c316d96affb607e199132c13bd140e29187ed59
# good: [b1e211a114e722288c7c2f8c592e0f0f8647a04e] source sha:57c512c4a3cffd069b30596642864368356c4038
git bisect good b1e211a114e722288c7c2f8c592e0f0f8647a04e
# good: [9142b3920dcda25dd5649ac952ca7c676226bd5d] source sha:4d65adb2fa02645e3c9e946a0b896af51ef27a7d
git bisect good 9142b3920dcda25dd5649ac952ca7c676226bd5d
# good: [fbbdd8c3d775574f145f1a7c9d97ffa59adc3221] source sha:d271ec87b8ec3264095a5265a7b0f1c073bc211b
git bisect good fbbdd8c3d775574f145f1a7c9d97ffa59adc3221
# good: [d98a3f9c99d7689fa7197488fe9cdf41dd91f277] source sha:dd0bc7323014c528fefb516cacae0591f270862a
git bisect good d98a3f9c99d7689fa7197488fe9cdf41dd91f277
# good: [33eda07a4761b5c81110518cc058d0de51fd8c33] source sha:08a690cf66d4e798896b61aa5a7a81e81885cd0c
git bisect good 33eda07a4761b5c81110518cc058d0de51fd8c33
# bad: [8977043bc38727f337dba48114079c8606a91549] source sha:aded98ce3d7a4189a4fa30989505da9427abc83b
git bisect bad 8977043bc38727f337dba48114079c8606a91549
# bad: [5eea56c750ffdca0c47adea10d8db86b4a8b10f5] source sha:b4e235a925b5c73a2220f57cdb8b85379fb3bac1
git bisect bad 5eea56c750ffdca0c47adea10d8db86b4a8b10f5
# bad: [64c0802644ae8f66c9b2479e29dab0b1103ad302] source sha:cb151051d79a7ec176e769a33a56e7d906fc1425
git bisect bad 64c0802644ae8f66c9b2479e29dab0b1103ad302
# first bad commit: [64c0802644ae8f66c9b2479e29dab0b1103ad302] source sha:cb151051d79a7ec176e769a33a56e7d906fc1425
Comment 9 Kevin Suo 2018-02-12 08:11:42 UTC
Just for the record, from bibisecting this bug, I now know how to bibisect locale-dependent bugs using the bibisect repos.

The TDF bibisect repos at [1] do not include any lang-packs. 

To bibisect bugs which work good in english UI, but bad in non-english UI (like CJKs): one should copy-paste the following files, from a release build which has your language, to the bibisect build locations:

1. program/resource/<your_lang_tag>
2. share/registry/Langpack-<your_lang_tag>.xcd
3. share/registry/res/fcfg_langpack_<your_lang_tag>.xcd
4. share/registry/res/registry_<your_lang_tag>.xcd

The suggested language resources you should copy is the one for the same major branch. For example, for the bibisect-linux-64-6.0 repo, I copied from release.
Comment 10 Xisco Faulí 2018-02-26 13:06:36 UTC
@kevin, thanks for adding it to the wiki -> https://wiki.documentfoundation.org/QA/Bibisect#Bibisecting_language-specific_bugs
Comment 11 Lun 2018-11-29 09:20:05 UTC
Bug not reproducible in version (x64)

Additional Info:
Version: (x64)
Build ID: 0f25a3c36f27fd51453b9a9115f236b83c143684
CPU threads: 8; OS: Windows 10.0; UI render: GL; VCL: win; 
TinderBox: Win-x86_64@42, Branch:master, Time: 2018-11-27_20:06:55
Locale: zh-TW (zh_TW); UI-Language: en-US
Calc: threaded

OS:Windows 10 Pro. 64-bit x64