You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@flex.apache.org by Alex Harui <ah...@adobe.com.INVALID> on 2017/07/06 23:00:04 UTC

Re: git commit: [flex-asjs] [refs/heads/develop] - Fix merge error removing support for numeric font size.

I just tried swapping SimpleCSSValuesImpl for AllCSSValuesImpl in the
develop branch in examples/flexjs/DataBindingExample and it worked for me.

Does it still not work for you?

-Alex

On 6/29/17, 3:40 AM, "Harbs" <ha...@gmail.com> wrote:

>Check your basic-manifest.xml for Core.
>
>Do you see <component id="AllCSSValuesImpl"
>class="org.apache.flex.core.AllCSSValuesImpl”/>?
>
>> On Jun 29, 2017, at 12:54 PM, Justin Mclean <ju...@classsoftware.com>
>>wrote:
>> 
>> Hi,
>> 
>> The class is there I’m on the latest version of develop of all 3 repos
>>and I’m still getting that compile error. Did you actually try using
>>that class / compiling it?
>> 
>> Thanks,
>> Justin
>

Re: git commit: [flex-asjs] [refs/heads/develop] - Fix merge error removing support for numeric font size.

Posted by Justin Mclean <ju...@classsoftware.com>.
Hi,

> Well, then maybe you can help us figure out why.  Upthread there are
> suggestions to check the manifest, etc.  What do you see in yours?

I sorted out the issue in the end. The alternate AllXXX class did suit our needs in terms of size / performance / quality so we went with an alternative method of using CSS and classList.add and classList.remove calls.

I’ve reverted the change.

Thanks,
Justin

Re: git commit: [flex-asjs] [refs/heads/develop] - Fix merge error removing support for numeric font size.

Posted by Alex Harui <ah...@adobe.com.INVALID>.
Well, then maybe you can help us figure out why.  Upthread there are
suggestions to check the manifest, etc.  What do you see in yours?

-Alex

On 7/6/17, 4:27 PM, "Justin Mclean" <ju...@classsoftware.com> wrote:

>Hi,
>
>> I just tried swapping SimpleCSSValuesImpl for AllCSSValuesImpl in the
>> develop branch in examples/flexjs/DataBindingExample and it worked for
>>me.
>> 
>> Does it still not work for you?
>
>It still does not work for me.
>
>Justin

Re: git commit: [flex-asjs] [refs/heads/develop] - Fix merge error removing support for numeric font size.

Posted by Justin Mclean <ju...@classsoftware.com>.
Hi,

> I just tried swapping SimpleCSSValuesImpl for AllCSSValuesImpl in the
> develop branch in examples/flexjs/DataBindingExample and it worked for me.
> 
> Does it still not work for you?

It still does not work for me.

Justin