# haxe-ui
- b
bright-gpu-74537
08/01/2021, 9:52 AMyessir - b
bright-gpu-74537
08/01/2021, 9:53 AMthe problem was that it was destroying the window (in handleRemoveComponentAt) and then trying to unregister all the events... tbf just a null check in unmapEvent would have been "fine" but the real fix was to sort out the order correctly - u
user
08/01/2021, 9:54 AMSeems fine - b
bright-gpu-74537
08/01/2021, 9:55 AMcool, so i think im going to refactor that code a little more too so things like this dont sneak in... would have been nice if it called "handleRemoveInternal" or something - b
bright-gpu-74537
08/01/2021, 9:55 AMbut for now this should be fine - b
bright-gpu-74537
08/01/2021, 9:55 AMalso, hope you dont mind, but im going to close that PR - i think its better to keep the removeComponentAt as a different "path" for a backend - u
user
08/01/2021, 9:56 AMYeah sure, it's outdated anyway, wasn't compatible with latest changes - u
user
08/01/2021, 10:31 AMare there any samples (or clues?) on how to hook a sortable Column to sort the TableView when you click the column header? tried googling and can't find anything 😦 - b
bright-gpu-74537
08/01/2021, 10:32 AMwhich backend? Composite? - u
user
08/01/2021, 10:32 AMhtml5, composite - b
bright-gpu-74537
08/01/2021, 10:34 AMso, i think you will be able to do it, lemme try and knock up a sample real quick... i recently added a "filter" option for datasources, it would be nice to have a sort function there also (which is on a todo list) so for the time being i think you'll have to do it semi manually - u
user
08/01/2021, 10:34 AMah it's not urgent then, I can wait - u
user
08/01/2021, 10:35 AMbut it sounds like I should look at the DataSource then? - b
bright-gpu-74537
08/01/2021, 10:35 AMsure, ill knock up the sample anyway, itll probably make it obvious to me exactly what needs to be done - b
bright-gpu-74537
08/01/2021, 10:35 AMwell, data sources dont actually have a sort function yet - u
user
08/01/2021, 10:35 AMah right - u
user
08/01/2021, 10:35 AMok, thanks! - b
bright-gpu-74537
08/01/2021, 11:04 AMSo, its pretty ugly but doable manually for now - b
bright-gpu-74537
08/01/2021, 11:05 AMthe main parts that should be handle by core are a) visual indicator of sort direction b) sort direction in
(not this hacky userData rubbish) c)Header
function being part ofsortDataSource - b
bright-gpu-74537
08/01/2021, 11:06 AM(and probably some other bits and bobs) - b
bright-gpu-74537
08/01/2021, 11:06 AMill have a play at making it "proper" sometime this week - it doesnt look to complex to make it much neater and generally handled internally by core (ie, you give it a sort function and thats that) - u
user
08/01/2021, 11:17 AMoh wow! thanks so much - b
bright-gpu-74537
08/01/2021, 11:18 AMnp - its ugly as hell at the moment, but will suffice until get a moment to do it properly in core - b
bright-gpu-74537
08/01/2021, 11:19 AMkeep in mind if you do use this code about the comment:// this will _only_ work with ArrayDataSource - u
user
08/01/2021, 1:57 PMCould a
API be added to components, which would be overriden by components likehasChild(child:Component):Bool
to check _contents.childComponents instead of childComponents?ScrollView - u
user
08/01/2021, 1:57 PMWould allow me to get rid of my other PR and use this instead in coconut.haxeui - u
user
08/01/2021, 1:59 PMOr is there already something like that? - b
bright-gpu-74537
08/01/2021, 2:01 PMwill
work?getComponentIndex(...) != -1 - u
user
08/01/2021, 2:03 PMHmm I suppose - b
bright-gpu-74537
08/01/2021, 2:03 PMi mean, we can certainly add a call hasComponent that just calls that