Issue 92101 - Vertical alignment "center" does not work for checkboxes
Summary: Vertical alignment "center" does not work for checkboxes
Status: CLOSED FIXED
Alias: None
Product: General
Classification: Code
Component: ui (show other issues)
Version: OOo 3.0 Beta 2
Hardware: All All
: P3 Trivial (vote)
Target Milestone: OOo 3.0
Assignee: michael.ruess
QA Contact: issues@framework
URL:
Keywords: regression
Depends on:
Blocks: 88888
  Show dependency tree
 
Reported: 2008-07-24 17:14 UTC by daniel.rentz
Modified: 2008-08-15 07:48 UTC (History)
2 users (show)

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

Attachments
document to reproduce the bug case (7.78 KB, application/vnd.oasis.opendocument.text)
2008-07-24 20:55 UTC, Frank Schönheit 		no flags Details
suggested patch (1.27 KB, patch)
2008-07-24 21:17 UTC, Frank Schönheit 		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 daniel.rentz 2008-07-24 17:14:05 UTC 
Vertical alignment -> "centered" does not work for checkboxes/optionbuttons
(still on top of bounding box). "Top"/"bottom" works fine. Pushbuttons work
fine. OOo 2.4.1 works fine.
Comment 1 Frank Schönheit 2008-07-24 20:55:20 UTC 
Created attachment 55352 [details]
document to reproduce the bug case
Comment 2 Frank Schönheit 2008-07-24 20:56:40 UTC 
attached is a document which shows the problem: It contains a radio button and a
check box, whose texts appear vertically centered when you open them in OOo 2.4,
but at the top when you open them in DEV300m27.

Investigating.
Comment 3 Frank Schönheit 2008-07-24 21:08:59 UTC 
fs->pl: At VCL level, the controls properly get their WB_VCENTER flag, but this
is not respected upon drawing.
Comment 4 Frank Schönheit 2008-07-24 21:17:34 UTC 
Created attachment 55353 [details]
suggested patch
Comment 5 Frank Schönheit 2008-07-24 21:18:29 UTC 
fs->pl: looks like a regression of your fix for issue 84824. The attached patch
seems to fix it for me.
Comment 6 philipp.lohmann 2008-07-25 09:00:41 UTC 
fixed in CWS vcl30stop2
Comment 7 philipp.lohmann 2008-07-25 15:36:15 UTC 
please verify in CWS vcl30stop2
Comment 8 michael.ruess 2008-08-01 12:16:53 UTC 
Cerified in CWS vcl30stop2.
Comment 9 michael.ruess 2008-08-15 07:48:03 UTC 
Checked in OOO300m2.