royale-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Harbs <harbs.li...@gmail.com>
Subject Re: Restructuring Composite Components
Date Thu, 31 May 2018 15:09:44 GMT
I don’t think so. The List is a component within ComboBox. I think the basic structure of
ComboBox makes sense as is. I think it’s the View which needs work.

It’s possible that an Express ComboBox might have additional properties to make it easier
to set the labelField, listHeight, etc.

I’m primarily trying to figure out the best way to declare the “skin pieces” of the
view.

Harbs

> On May 31, 2018, at 5:17 PM, Piotr Zarzycki <piotrzarzycki21@gmail.com> wrote:
> 
> Hi Harbs,
> 
> What if ComboBox will extend DataContainer? It will be at some point list
> itself. Than the rest responsibility is in View. We know that ComboBox is
> always list, so if someone would like to change the other parts need to
> have dive into view.
> 
> On the other hand this solution probably is less flexible than having
> through css some parts of that component.
> 
> Thanks,
> Piotr
> 
> 
> czw., 31 maj 2018 o 15:35 Harbs <harbs.lists@gmail.com> napisał(a):
> 
>> I’m working on redoing ComboBox and I could use some input on best
>> practices.
>> 
>> Right now, ComboBox has ComboBox view which has three components
>> hard-wired in. It has TextInput, TextButton, and List. If any visuals or
>> behavior need to be changed, the entire view needs to be rewritten. We’ve
>> already had two cases where I needed a new view. One was to disable the
>> TextInput input and change the button appearance. The second was to support
>> images in a dropdown.
>> 
>> It seems to me like the ComboBoxView should add the three components based
>> on declared classes. I think there should be 1. A “Button” component. 2. An
>> ItemRenderer component for the “input” which displays the selected item. 3.
>> A List component for displaying the dropdown. The question I’m struggling
>> with is how to declare these.
>> 
>> I don’t think it makes sense to make these all beads. Currently Buttons
>> are not beads.
>> 
>> Should we add interfaces (i.e. IButton) and declare the classes using
>> interfaces? I mean something similar to IItemRenderer.
>> 
>> Maybe just use ValuesManager to just lookup some other “name”? We’re
>> currently doing something similar with ColumnClassName in DynamicDataGrid.
>> 
>> Other ideas?
>> 
>> Thanks,
>> Harbs
> 
> 
> 
> -- 
> 
> Piotr Zarzycki
> 
> Patreon: *https://www.patreon.com/piotrzarzycki
> <https://www.patreon.com/piotrzarzycki>*


Mime
View raw message