Mailing-List: contact dev-help@flex.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@flex.apache.org
Received-SPF: error (nike.apache.org: local policy)
Content-Type: text/plain; charset=us-ascii
Mime-Version: 1.0 (Apple Message framework v1283)
Subject: Re: New Mustella Failures
From: Justin Mclean <justin@classsoftware.com>
In-Reply-To: <CDDD376F.B3CF%aharui@adobe.com>
Date: Wed, 12 Jun 2013 18:22:00 +1000
Content-Transfer-Encoding: quoted-printable
Message-Id: <DC3DA6B3-D3BA-4034-8885-D53524ADD3B6@classsoftware.com>
References: <CDDD376F.B3CF%aharui@adobe.com>
To: dev@flex.apache.org

HI,

> Well, I'm not sure it is only an issue with the test.=20

=46rom a quick look it the setStyle that's not causing everything to =
refresh. The extra validateNow fixed that but was also called when =
scrolling, editing etc etc slowing the ADG down.

Then perhaps a simple solution would to be this?  ie add layoutDirection =
to the styleChanged method.=20

    override public function styleChanged(styleProp:String):void
    {
        super.styleChanged(styleProp);

        if (styleProp =3D=3D "sortFontFamily"
            || styleProp =3D=3D "sortFontSize"
            || styleProp =3D=3D "sortFontStyle"
	    || styleProp =3D=3D "sortFontWeight"
	    || styleProp =3D=3D "layoutDirection")
        {
            itemsSizeChanged =3D true;
            rendererChanged =3D true;
            invalidateProperties();
            invalidateDisplayList();
        }
    }

Probably "direction" should also be in there.

Justin