Bug 123278

Summary: Color of Asian phonetic guide is not from the base text but from the text before that
Product: LibreOffice Reporter: himajin100000
Component: WriterAssignee: Not Assigned <libreoffice-bugs>
Status: NEW ---    
Severity: normal CC: buzea.bogdan, marklh9, raal
Priority: medium Keywords: bibisected, bisected, regression
Version: 6.1.0.0.alpha0+   
Hardware: All   
OS: All   
Whiteboard:
Crash report or crash signature: Regression By:
Bug Depends on:    
Bug Blocks: 50607    
Attachments: file for test

Description himajin100000 2019-02-09 02:02:49 UTC
Description:
see steps to reproduce

Steps to Reproduce:
1. Run Writer
2. type "AB"
3. Select "A"
4. context menu-> character->character
5. "Font Color" in Font "Effects" Tab
6. Select "Red"
7. OK
8. Select "B"
9. main menu-> format->Asian phonetic guide
10.Set "b" as ruby text for "B"
11. OK


Actual Results:
"b" is shown in red color

Expected Results:
"b" is shown in default black color


Reproducible: Always


User Profile Reset: No



Additional Info:
Version: 6.3.0.0.alpha0+ (x64)
Build ID: 82fb7f9f90a3ba9d53b3412124e513aa78ddca10
CPU threads: 4; OS: Windows 10.0; UI render: default; VCL: win; 
TinderBox: Win-x86_64@42, Branch:master, Time: 2019-02-05_23:41:03
Locale: ja-JP (ja_JP); UI-Language: en-US
Calc: threaded
Comment 1 raal 2019-02-09 19:56:22 UTC
Confirm with Version: 6.3.0.0.alpha0+
Build ID: f8c4459bbda62cb905d0c007cac044f3410ab76b
CPU threads: 4; OS: Linux 4.15; UI render: default; VCL: gtk3; 


regression, works in 6.0.7
Comment 2 raal 2019-02-09 20:04:20 UTC
This seems to have begun at the below commit.
Adding Cc: to Mark Hung ; Could you possibly take a look at this one?
Thanks
 3e278e12f3500738012cc2f579c68156f3306541 is the first bad commit
commit 3e278e12f3500738012cc2f579c68156f3306541
Author: Jenkins Build User <tdf@pollux.tdf>
Date:   Sun Jan 21 09:05:06 2018 +0100

    source d1bc14b318c9a412a761d243085da0895a1aed4a

author	Mark Hung <marklh9@gmail.com>	2018-01-20 23:35:44 +0800
committer	Mark Hung <marklh9@gmail.com>	2018-01-21 08:43:14 +0100
commit d1bc14b318c9a412a761d243085da0895a1aed4a (patch)
tree ea4dbeaf9c053974e690dc8b856fb4b66638ffe5
parent 121f6f3c79ea2dceb7cc3d61a56f5a56a1cb0d0d (diff)
tdf#35301 Formatting ruby text on right side.
Comment 3 raal 2019-02-09 20:04:46 UTC
Created attachment 149057 [details]
file for test
Comment 4 himajin100000 2019-08-24 08:38:14 UTC
Strangely, if the file given by raal is opened by MS Word, 'b' is shown in black color.
Comment 5 himajin100000 2019-08-26 19:19:21 UTC
https://opengrok.libreoffice.org/xref/core/sw/source/core/text/itrform2.cxx?r=56e95f8e#1259

Just out of curiosity:

1. Is there any way to get the next portion to this SwRubyPortion?
2. From which portion does this rInf come from? rInf.GetFont() is passed as the  second paramter of SwRubyPortion's constructor.

I don't know why the commit regressed Writer either.
Comment 6 QA Administrators 2021-08-26 03:47:18 UTC Comment hidden (obsolete)
Comment 7 BogdanB 2023-05-18 14:58:09 UTC
Also in 
Version: 7.6.0.0.alpha1+ (X86_64) / LibreOffice Community
Build ID: 3584fd0d110b4138ec32cf55ddd07e3fb8d81333
CPU threads: 16; OS: Linux 5.19; UI render: default; VCL: gtk3
Locale: ro-RO (ro_RO.UTF-8); UI: en-US
Calc: threaded