Re: [webkit-dev] MathML layout refactor proposal

2015-12-28 Thread Alejandro Garcia Castro

Hi Dave,

thanks very much for your comments and feedback, this information is
great to document the whole design process of the code.

I agree with the goals you are pointing out, actually I think the
change to use flexbox was a good option to try at that point. But
currently considering the result of the design in the code we can see
that the devil is in the detail.

Trying to use box spacing to create complex anonymous render trees
that produce the complex layout result of some mathml formula, brings
a complex code difficult to improve to get the best math render
possible. That is the main reason for this change, you can see bugs
such as this were developers are not even sure how the whole layout is
happening:

https://bugs.webkit.org/show_bug.cgi?id=137330

The problem is worse if we try to add more complex layout calculations
to match best formula rendering.

Also we can find flexbox changes just avoiding mathml tests because
it is not clear how the anonymous nodes structure is handled for some
cases.

So this takes us to the current proposal, if we do not get more
feedback probably we will start asking for reviews in the next weeks,
and we can discuss in more detail in the bugs the pros and cons of the
change.

Thanks again,

Alex
___
webkit-dev mailing list
webkit-dev@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-dev


Re: [webkit-dev] Wincairo support for wss:// urls with websockets

2015-12-28 Thread Isaac Devine
How are the webkit specific changes seperated from the upstream source in
that repository?

Is there a directory of patches somewhere?

On 29 December 2015 at 09:27, Alex Christensen 
wrote:

> If you want to send a pull request for that, too, I would gladly accept
> it.  Also, the output needs to be called libicuuc and libicuin instead of
> icuuc and icuin to match the AppleWin port, and it would be nice if it
> built it into the same bin64, bin32, lib64, lib32, and include directories
> as everything else.
>
> On Dec 28, 2015, at 12:13 PM, Isaac Devine 
> wrote:
>
> Thanks,
>
> If you are updating to ICU 56.1, it doesn't build in visual studio 2015.
>
> A patch that fixes this can be found at:
> http://bugs.icu-project.org/trac/ticket/11822
>
>
> On 29 December 2015 at 08:37, Alex Christensen 
> wrote:
>
>> Send a pull request to
>> https://github.com/achristensen07/WinCairoRequirements or just send me
>> an email with the changes you want to include.  I’m right in the middle of
>> updating icu in that repository, so you might want to go back to
>> a9b631e1146a696dc81b584c837ccfb7db5524b9
>>
>> Alex
>>
>> On Dec 21, 2015, at 4:44 PM, Isaac Devine 
>> wrote:
>>
>> Hi all,
>>
>> I have a patch that enables support for wss:// urls for websockets with
>> the wincairo port.
>>
>> I haven't created a bug and attachment yet because it requires curl 7.45,
>> which is newer than the curl that the current wincairo build depends on.
>>
>> How do I go about proposing updates of the curl dependency?
>>
>> Thanks,
>> Isaac
>>
>> ___
>> webkit-dev mailing list
>> webkit-dev@lists.webkit.org
>> https://lists.webkit.org/mailman/listinfo/webkit-dev
>>
>>
>>
>
>
>


-- 
Isaac Devine
Director
Devine Systems Limited

+64 21 1700 929
___
webkit-dev mailing list
webkit-dev@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-dev


Re: [webkit-dev] Wincairo support for wss:// urls with websockets

2015-12-28 Thread Alex Christensen
Send a pull request to https://github.com/achristensen07/WinCairoRequirements 
 or just send me an 
email with the changes you want to include.  I’m right in the middle of 
updating icu in that repository, so you might want to go back to 
a9b631e1146a696dc81b584c837ccfb7db5524b9

Alex

> On Dec 21, 2015, at 4:44 PM, Isaac Devine  wrote:
> 
> Hi all,
> 
> I have a patch that enables support for wss:// urls for websockets with the 
> wincairo port.
> 
> I haven't created a bug and attachment yet because it requires curl 7.45, 
> which is newer than the curl that the current wincairo build depends on. 
> 
> How do I go about proposing updates of the curl dependency?
> 
> Thanks,
> Isaac
> 
> ___
> webkit-dev mailing list
> webkit-dev@lists.webkit.org
> https://lists.webkit.org/mailman/listinfo/webkit-dev

___
webkit-dev mailing list
webkit-dev@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-dev


Re: [webkit-dev] Wincairo support for wss:// urls with websockets

2015-12-28 Thread Isaac Devine
Thanks,

If you are updating to ICU 56.1, it doesn't build in visual studio 2015.

A patch that fixes this can be found at:
http://bugs.icu-project.org/trac/ticket/11822


On 29 December 2015 at 08:37, Alex Christensen 
wrote:

> Send a pull request to
> https://github.com/achristensen07/WinCairoRequirements or just send me an
> email with the changes you want to include.  I’m right in the middle of
> updating icu in that repository, so you might want to go back to
> a9b631e1146a696dc81b584c837ccfb7db5524b9
>
> Alex
>
> On Dec 21, 2015, at 4:44 PM, Isaac Devine 
> wrote:
>
> Hi all,
>
> I have a patch that enables support for wss:// urls for websockets with
> the wincairo port.
>
> I haven't created a bug and attachment yet because it requires curl 7.45,
> which is newer than the curl that the current wincairo build depends on.
>
> How do I go about proposing updates of the curl dependency?
>
> Thanks,
> Isaac
>
> ___
> webkit-dev mailing list
> webkit-dev@lists.webkit.org
> https://lists.webkit.org/mailman/listinfo/webkit-dev
>
>
>
___
webkit-dev mailing list
webkit-dev@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-dev


Re: [webkit-dev] Wincairo support for wss:// urls with websockets

2015-12-28 Thread Alex Christensen
If you want to send a pull request for that, too, I would gladly accept it.  
Also, the output needs to be called libicuuc and libicuin instead of icuuc and 
icuin to match the AppleWin port, and it would be nice if it built it into the 
same bin64, bin32, lib64, lib32, and include directories as everything else.

> On Dec 28, 2015, at 12:13 PM, Isaac Devine  wrote:
> 
> Thanks,
> 
> If you are updating to ICU 56.1, it doesn't build in visual studio 2015.
> 
> A patch that fixes this can be found at:
> http://bugs.icu-project.org/trac/ticket/11822 
> 
> 
> 
> On 29 December 2015 at 08:37, Alex Christensen  > wrote:
> Send a pull request to https://github.com/achristensen07/WinCairoRequirements 
>  or just send me an 
> email with the changes you want to include.  I’m right in the middle of 
> updating icu in that repository, so you might want to go back to 
> a9b631e1146a696dc81b584c837ccfb7db5524b9
> 
> Alex
> 
>> On Dec 21, 2015, at 4:44 PM, Isaac Devine > > wrote:
>> 
>> Hi all,
>> 
>> I have a patch that enables support for wss:// urls for websockets with the 
>> wincairo port.
>> 
>> I haven't created a bug and attachment yet because it requires curl 7.45, 
>> which is newer than the curl that the current wincairo build depends on. 
>> 
>> How do I go about proposing updates of the curl dependency?
>> 
>> Thanks,
>> Isaac
>> 
>> ___
>> webkit-dev mailing list
>> webkit-dev@lists.webkit.org 
>> https://lists.webkit.org/mailman/listinfo/webkit-dev 
>> 
> 
> 
> 

___
webkit-dev mailing list
webkit-dev@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-dev