Issue 72738 - The state of the attribute is changed after reopen the setting dialog.
Summary: The state of the attribute is changed after reopen the setting dialog.
Status: CLOSED FIXED
Alias: None
Product: Impress
Classification: Application
Component: ui (show other issues)
Version: OOo 2.1
Hardware: All Windows, all
: P3 Trivial (vote)
Target Milestone: OOo 2.2
Assignee: christian.guenther
QA Contact: issues@graphics
URL:
Keywords:
Depends on:
Blocks: 72764
  Show dependency tree
 
Reported: 2006-12-19 07:47 UTC by liujiaxiang
Modified: 2007-01-25 13:46 UTC (History)
2 users (show)

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

Attachments
the patch of this bug (1.31 KB, patch)
2007-01-04 01:22 UTC, liujiaxiang 		no flags Details | Diff
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this issue.
Description liujiaxiang 2006-12-19 07:47:51 UTC 
In the OOo 2.1.0
The state of the attribute is changed after reopen the setting dialog.
the bug can be described as follows:
1. Open a new presentation and input some words in text edit.
2. locate the cursor on the text edit->Format->Position and Size->select 
the "Position and Size" lable->checked the "Fit width to text" and "Fit height 
to text"->the "Width" and "Height" become gray and can not be changed->OK.
3. Format->Position and Size->select the "Position and Size" lable-
>the "Width" and "Height" can be changed.
Comment 1 peter.junge 2006-12-20 01:52:02 UTC 
pj->wg: This one needs a target. Is 2.x OK?
Added myself as CC
Comment 2 wolframgarten 2007-01-02 08:26:42 UTC 
Reassigned.
Comment 3 liujiaxiang 2007-01-04 01:22:05 UTC 
Created attachment 41870 [details]
the patch of this bug
Comment 4 christian.guenther 2007-01-04 14:02:01 UTC 
I can reproduce the bug.
liutao attached a patch.
Please have a look.
Comment 5 clippka 2007-01-05 12:33:30 UTC 
fixed in cws impress115,

What a mess (the original code, not the patch), I didn't apply the patch, I
consolidated all dynamic enable and disable calls into a single
SvxPositionSizeTabPage::UpdateControlStates() methods. Thats the only design
pattern to keep this consistent
Comment 6 clippka 2007-01-05 12:34:25 UTC 
changing target
Comment 7 liujiaxiang 2007-01-06 01:20:35 UTC 
@cl can you give me your patch? thank you.
Comment 8 clippka 2007-01-08 13:31:30 UTC 
@liutao, see 

svx/source/dialog/transfrm.cxx rev. 1.25.52.1
svx/source/dialog/transfrm.hxx rev. 1.7.130.1
Comment 9 clippka 2007-01-16 11:02:27 UTC 
verified in cws, back to qa
Comment 10 christian.guenther 2007-01-17 14:16:20 UTC 
CGU: Verifed in cws impress115
Comment 11 christian.guenther 2007-01-25 13:46:15 UTC 
CGU: Integrated in OOF680m2