Bug 123278 - Color of Asian phonetic guide is not from the base text but from the text before that
Summary: Color of Asian phonetic guide is not from the base text but from the text bef...
Status: NEW
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Writer (show other bugs)
Version:
(earliest affected)
6.1.0.0.alpha0+
Hardware: All All
: medium normal
Assignee: Not Assigned
URL:
Whiteboard:
Keywords: bibisected, bisected, regression
Depends on:
Blocks: Ruby
  Show dependency tree / graph
 
Reported: 2019-02-09 02:02 UTC by himajin100000
Modified: 2023-05-18 14:58 UTC (History)
3 users (show)

See Also:
Crash report or crash signature:

Attachments
file for test (8.10 KB, application/vnd.oasis.opendocument.text)
2019-02-09 20:04 UTC, raal 		Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
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