1. Seems to be a data binding problem. (See other thread.) Yishay committed a 
temporary work-around, but we don’t think that’s the right way to fix the 
problem.
2. seems to be a measurement problem. The titleBar height is not being measured 
correctly. This is a workaround (in a custom item renderer)

            override public function get collapsedHeight():Number{
                return 30;
            }


> On Jun 4, 2017, at 12:41 AM, Harbs <harbs.li...@gmail.com> wrote:
> 
> I worked around the Promise issue (by copying js.swc to my project and not 
> using the one in the SDK).
> 
> There’s at least 2 issues still:
> 1. The title property from the model is not being applied to the view of the 
> item renderer.
> 2. The collapsed height of the collapsed items are 0 instead of the title bar 
> height.
> 
>> On Jun 4, 2017, at 12:10 AM, Harbs <harbs.li...@gmail.com> wrote:
>> 
>> Yishay did the implementation of this so I was a bit shaky on the details.
>> 
>> I just looked at our code, and it appears that we did not actually use 
>> panels for the children.
>> 
>> It turns out the children are actually Containers which have a TitleBarModel 
>> bead. Sorry about the confusion. It might make sense to have an interface 
>> for an accordion-compatible container.
>> 
>> We will put together an example which should work better in the morning.
>> 
>> I cannot test my app which uses the Accordion right now because Promise is 
>> currently broken (like I wrote in my other email).
>> 
>> Harbs
>> 
>>> On Jun 2, 2017, at 7:01 PM, Peter Ent <p...@adobe.com.INVALID> wrote:
>>> 
>>> Hi,
>>> 
>>> It looks like this is the last thing to be resolved before we can make
>>> FlexJS 0.8 release.
>>> 
>>> I'm seeing two title bars per item in the Accordion. Any suggestions for
>>> how to resolve this, based on the information I've given below?
>>> 
>>> Thanks,
>>> Peter
>>> 
>>> On 6/1/17, 3:49 PM, "Peter Ent" <p...@adobe.com> wrote:
>>> 
>>>> I've checked in my changes to the Accordion components. It still is not
>>>> working correctly and I cannot figure out what is happening. The
>>>> <js:Panel> used as the data to the Accordion are being placed as children
>>>> of AccordionItemRenderers which are themselves Panels. So there are two
>>>> TitleBars present per Accordion section.
>>>> 
>>>> The layout mechanism changed so that the GroupView (the base view bead for
>>>> all container-type view beads) no longer controls when layouts run; that
>>>> is done by the layouts themselves. GroupView et al has a beforeLayout()
>>>> and afterLayout() functions called by the layout classes which might be
>>>> helpful, I'm not sure.
>>>> 
>>>> Panel also changed quite a bit. A Panel is now a Group with its own layout
>>>> that controls the placement of the TitleBar and Container which is its
>>>> content area. When you specify a layout bead on a Panel, the Panel
>>>> actually moves it to the content area Container. Perhaps this has
>>>> something to do with the behavior we are now seeing.
>>>> 
>>>> Please let me know if you have any suggestions on how to handle the
>>>> Accordion as it now sits and I'll be happy to answer any questions about
>>>> how the current view/layout system works now.
>>>> 
>>>> If I may, perhaps Accordion could be changed as follows:
>>>> 
>>>> <js:Accordion selectedIndex="0">
>>>> <js:AccordionSection title="Panel 1">
>>>>    <!‹ a single child here, such as a Group, Container, or List ‹>
>>>> </js:AccordionSection>
>>>> <js:AccordionSection title="Panel 2">
>>>>     <!‹ a single child here, such as a Group, Container, or a List ‹>
>>>> </js:AccordionSection>
>>>> </js:Accordion>
>>>> 
>>>> The Accordion + AccordionView would create 2 children for each
>>>> AccordionSection in the Accordion's space: an AccordionHeader + <single
>>>> child>. 
>>>> 
>>>> The model would indicate which <single child> is being viewed and the
>>>> layout, such as OneFlexibleChildVerticalLayout or
>>>> OneFlexibleChildHorizontalLayout, would take care of sizing and
>>>> positioning the AccordionHeader and <single child> elements. The <single
>>>> child> elements not visible would simply have visible=false set; the
>>>> layout will then ignore them.
>>>> 
>>>> For the HTML side, this example would create a <DIV> with four children
>>>> and not have any deep nesting unless that what the <single child>
>>>> produces. The <single child> that's visible would have overflow:auto or
>>>> overflow:hidden while the other <single child> elements would have
>>>> display:none set. 
>>>> 
>>>> Merely a suggestion, and could probably use some refinement.
>>>> 
>>>> ‹peter
>>>> 
>>>> 
>>>> On 6/1/17, 2:03 PM, "yishayw" <yishayj...@hotmail.com> wrote:
>>>> 
>>>>> Harbs wrote
>>>>>> \2. The Collapse bead can only infer that it¹s collapsed by the fact
>>>>>> that
>>>>>> the size is the collapsed size ‹ which only makes sense if the size is
>>>>>> set.
>>>>> 
>>>>> Shouldn't .height return the measured height, regardless of whether it
>>>>> was
>>>>> explicitly set?
>>>>> 
>>>>> 
>>>>> 
>>>>> 
>>>>> --
>>>>> View this message in context:
>>>>> https://na01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fapache-fl
>>>>> e
>>>>> x-development.2333347.n4.nabble.com%2FFlexJS-Accordion-broken-tp61937p620
>>>>> 0
>>>>> 8.html&data=02%7C01%7C%7C9b640697ac694828308808d4a91a8573%7Cfa7b1b5a7b344
>>>>> 3
>>>>> 8794aed2c178decee1%7C0%7C0%7C636319378749470812&sdata=I%2BJ9TjnMxtY8VD4b4
>>>>> h
>>>>> 6ljmTghd1Wy8yG8xo2eR9s6OY%3D&reserved=0
>>>>> Sent from the Apache Flex Development mailing list archive at Nabble.com.
>>>> 
>>> 
>> 
> 

Reply via email to