trunk: incorrect rendering of switch elements (rev >= 12532)

Bug #1249862 reported by su_v
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Inkscape
Fix Released
Undecided
Markus Engel

Bug Description

A file (attached) containing a <switch> element like this:

        <switch allowReorder="yes">
            <text systemLanguage="en">en</text>
            <text systemLanguage="de">de</text>
            <text systemLanguage="es">es</text>
            <text systemLanguage="fr">fr</text>
            <text systemLanguage="it">it</text>
            <text>other</text>
        </switch>

renders as expected with Inkscape 0.48.4 and trunk rev <= 12531. With rev >= 12532, the <switch> element without 'systemLanguage' attribute (content: "other") is rendered on top of an element selected based on the correctly detected system language.

Tested on OS X 10.7.5 by launching inkscape from the command line:
$ LANG=en_EN.UTF-8 inkscape testcase.svg
$ LANG=de_DE.UTF-8 inkscape testccase.svg

The <switch> renders as expected in Firefox 25.0, Chromium 33.0.1705.0 (234154) and Squiggle (Batik 1.7).

The test case was taken from bug #1249423 (which AFAICT is about a different issue since the user doesn't see any content of any element of the <switch> object).

Tags: regression svg
Revision history for this message
su_v (suv-lp) wrote :
Revision history for this message
su_v (suv-lp) wrote :

Screenshot of test case, rendered in r12786 with system language fr_FR
(the last element is differently colored to better visualized the overlaid objects).

Revision history for this message
Markus Engel (engelmarkus) wrote :

I was missing some "virtuals" there, causing the wrong method to be called. Try the attached patch.

Changed in inkscape:
assignee: nobody → Markus Engel (engelmarkus)
status: New → In Progress
Revision history for this message
su_v (suv-lp) wrote :

Patch tested successfully with r12790 on OS X 10.7.5.

Revision history for this message
Markus Engel (engelmarkus) wrote :

Fixed in r12794.

Changed in inkscape:
status: In Progress → Fix Released
su_v (suv-lp)
Changed in inkscape:
milestone: 0.91 → none
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.