https://issues.apache.org/ooo/show_bug.cgi?id=122610

            Bug ID: 122610
        Issue Type: DEFECT
           Summary: [SVG] In case of missing perserveAspectRatio the
                    default value "xMidYMid meet" has to be used
           Product: Draw
           Version: AOO400-dev
          Hardware: PC
               URL: http://www.w3.org/TR/SVG/struct.html#SVGElement
                OS: Windows 7
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: code
          Assignee: [email protected]
          Reporter: [email protected]
                CC: [email protected], [email protected]

Created attachment 80927
  --> https://issues.apache.org/ooo/attachment.cgi?id=80927&action=edit
Example with set and with missing preserveAspectRadio

Open attached graphic file in a browser, and insert the file into a Draw
document. It contains two inner svg elements. The upper svg element has got no
preserveAspectRatio attribute, the lower svg element has set
preserveAspectRatio="xMidYMid meet" explicitly. When a preserveAspectRatio
attribute is missing, the default value "xMidYMid meet" has to be used, see
linked spec 5.1 item 'preserveAspectRatio'. Therefore the two svg elements in
the test file should look similar.

In AOO the upper svg element looks as if preserveAspectRatio="none" is used.

The constructor of SvgAspectRatio sets already maSvgAlign(Align_xMidYMid) and
mbMeetOrSlice(true). But on the same time sets mbSet(false). I wonder what is
the purpose of mbSet and why it is 'false' here. Because of the specified
default values the attribute has always a value.

If there is a valid reason to have it this way, then the member
maSvgAspectRatio of SvgSvgNode should be initialized with the default value
"xMidYMid meet" in the constructor of SvgSvgNode.

What do you think?

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are the assignee for the bug.

Reply via email to