tableView 2.0 delete a row without animation

Solar2D Widgets
#21

Brent! That’s excellent news, and thanks for your help. I’m looking forward to it!

#22

Any updates on this - supposed to be - upcoming implemented feature??

#23

<akak>: Agreed --it would be useful to have an update on this.

Brent, you mentioned in early January that the feature is already implemented and will be in an “upcoming daily build” – can you share any more information? Thanks!

#24

Hi corona273,

I reminded the engineers about this and they hope to include this in a daily build within the next week or so. If this doesn’t occur, can you please respond back to this thread so I can push the issue?

Thanks,

Brent

#25

Hello,
 

Would love to see this in the next daily build as well.

Thanks,
Shehab

#26

Brent: 'afternoon! You said “can you please respond back to this thread so I can push the issue” two weeks ago. I don’t believe Tableview has been updated, so here’s your reminder as requested. :slight_smile:

#27

Hi @corona273,

Thanks for the reminder… just please don’t kill the messenger (me). :slight_smile: Engineering got back-logged a bit on this, but they report it should go into testing next week and rolled into a daily build shortly after.

Brent

#28

Sounds great! Thanks, Brent!

#29

Hello!

Any update on this ? I’m about to submit my app but I was waiting to be able to implement this new feature. Is this still imminent (days) or has it been ulteriorly postponed?

thanks !!!

#30

Hi @akak and others,

I believe this fix/addition passed testing yesterday, so it should roll out into a daily build soon.

Brent

#31

Folks: I just ran into this today while deleting rows from a TableView and was surprised to see that there’s no way to remove the horizontal animation. Of course you can delete all rows and re-render, but then you get a bad case of the flickers.

It looks like the horizontal animation is at line 1320 of this file: https://github.com/coronalabs/framework-widget/blob/master/widgetLibrary/widget_tableview.lua#L1707

transition.to( self.\_rows[rowIndex].\_view, { x = - ( self.\_rows[rowIndex].\_view.contentWidth \* 0.5 ), transition = easing.inQuad, onComplete = removeRow } )

 

Can Corona make it is possible to pass a parameter to the transition to specify a time and override the 500ms default?

I’m not being lazy here: I could trivially change the above file. But if I go that path I’m going to be out of sync with bug fixes in the daily builds – there were a dozen fixes to scrollview and tableview on 10/29, for instance, but the repository wasn’t updated until 11/25…

#32

Hi corona273@,

Let me put in a request for this. For clarity, this would likely need to be a boolean passed along with “deleteRow()”? Or would a more ideal solution be to provide a number (integer) for the transition time (with 0 being, of course, no animation)?

http://docs.coronalabs.com/api/type/TableViewWidget/deleteRow.html

Brent

#33

Brent: Thanks again for your help with these points. I don’t speak for <ksan> and <skatan> and the other developers in this thread, but either would work for me; I simply want to turn off any animation and have the rows be deleted immediately.

So I prefer whichever solution leads to a quicker fix in a daily build! :slight_smile: If both approaches are equally speedy, then providing an integer for the transition time with 0 being no animation seems preferable. Thanks again!

#34

Hi @corona273,

The engineers report that this feature is already done, internally, and it will be released in an upcoming daily build. I’m not sure exactly when, but hopefully not too far off. The concept will be that you can pass a table of options to the row deletion API with various parameters that allow behavior control.

Brent

#35

Hey Brent! Thanks tons for that. I think the best would be to be able to choose the animation time, since that would provide even more customisation for whoever isn’t satisfied with the default animation speed to be able to tweak it and make it faster/slower depending on what they want, or remove it of course altogether :slight_smile:

Thanks again!!

Edit: Ah just saw your reply. That solution sounds even better. Can’t wait to try it out!

#36

Brent! That’s excellent news, and thanks for your help. I’m looking forward to it!

#37

Brent:

'evening! It looks like you folks have made a heck of a lot of changes to the widget library in this evening’s daily build:

http://forums.coronalabs.com/topic/55291-new-daily-build-20152581-revamps-widget-library/

Is this tableview delete row change included? And if it is, can you tell us what the interface is?

Thanks!

#38

Yes, like above, please :slight_smile: !

#39

Looks like this works with tableView:deleteRows( tableOfRowsToDelete, tableOfAnimationOptions )

The table of animationOptions looks like:

animationOptions.slideLeftTransitionTime = timeInMilliseconds

animationOptions.slideUpTransitionTime = timeInMilliseconds

So something like;

[lua]

local animationOptions = { slideLeftTransitionTime = 0, slideUpTransitionTime = 0 }

tableView:deleteRows( { 1, 2, 3, 4}, animationOptions )

[/lua]

I’m checking on the status of the documentation.

Rob

#40

The (daily) documentation should be updated on this point soon. In the meantime, Rob’s example is correct so you may mimic that.

Brent