Issue 88861 - Cannot remove direct char format when mixed char formats
Summary: Cannot remove direct char format when mixed char formats
Status: CLOSED FIXED
Alias: None
Product: Writer
Classification: Application
Component: formatting (show other issues)
Version: OOo 2.4.0
Hardware: All All
: P3 Trivial (vote)
Target Milestone: ---
Assignee: michael.ruess
QA Contact: issues@sw
URL:
Keywords:
: 89409 (view as issue list)
Depends on:
Blocks: 88258
  Show dependency tree
 
Reported: 2008-04-29 16:11 UTC by cbrunet
Modified: 2013-08-07 14:43 UTC (History)
1 user (show)

See Also:
Issue Type: DEFECT
Latest Confirmation in: ---
Developer Difficulty: ---


Attachments

Note You need to log in before you can comment on or make changes to this issue.
Description cbrunet 2008-04-29 16:11:25 UTC
1- Select three words on a text, and press the italic button.
2- Select the word in the middle, and press the bold button.
3- Select all three words again, and press the italic button.

Expected: the italic char format to be removed
Actual result: the char attributes aren't modified.

Reproduced on both WinXP and Ubuntu Linux. I think this is a regression from 2.3.

Workaround: press Ctrl-Shift-Space to remove all char formats
Comment 1 frank.meies 2008-04-30 09:53:13 UTC
fme: Analysis: Regression from cws swwarnings (warning free code). The parameter
nStart of GetAttr() has been renamed to nStt because a class member was shadowed
by this. Unfortunately a resync introduced a new usage of nStart from a
different cws, causing this error. This bug occurs if the attribute state is
requested (e.g., for the toolbar) and there are some 'nested' formattings.
Comment 2 frank.meies 2008-04-30 11:04:05 UTC
.
Comment 3 frank.meies 2008-04-30 14:49:39 UTC
fme: fixed in cws sw241bf02, ndtxt.hxx, txtedt.cxx, thints.cxx
Comment 4 frank.meies 2008-04-30 16:10:00 UTC
Ready for QA.
Comment 5 michael.ruess 2008-05-06 07:56:47 UTC
Verified fix in CWS sw241bf02.
Comment 6 michael.ruess 2008-05-14 07:26:01 UTC
*** Issue 89409 has been marked as a duplicate of this issue. ***
Comment 7 michael.ruess 2008-06-09 14:16:47 UTC
Checked fix in DEV300m17.