Export to GitHub

mad-components - issue #41

UIList alt pull down refresh


Posted on Sep 2, 2012 by Swift Kangaroo

Modified UIList to support pull down refresh as in other apps, where you have to pull down, and then release to start the update.

Also, while looking into adding this I happened to see a performance improvement possible inside clearCells (the previous code was being evaluated constantly).

Attachments

Comment #1

Posted on Sep 3, 2012 by Grumpy Bird

I like your improvements - and especially your care not to break anything in the process. I'm wrapped-up in the MadComponents3D release at the moment - but after I've started the ball rolling with GPU-acceleration, I'd like to get a few more developers involved. The framework could benefit from some fresh blood - and people who see issues that I miss (because I only see what I THINK I've written).

So I'd like to add you as an author when that happens. Are you interested?

Comment #2

Posted on Sep 3, 2012 by Grumpy Bird

(No comment was entered for this change.)

Comment #3

Posted on Sep 3, 2012 by Swift Kangaroo

"and especially your care not to break anything in the process"

Well, I care, but may not manage it. For example, while looking again at this patch, I've seen:

  • override protected function startMovement0():Boolean {
  • if (_refreshState) {

Should be:

  • override protected function startMovement0():Boolean {
  • if (_refreshState && _altPullDown) {

"So I'd like to add you as an author when that happens. Are you interested?"

I'd gladly do so, but I don't know if I'd be worthy of it, while I work with MadComponents (which hopefully will be for quite some time), I'll be contributing as much as I can, but I'm quite busy with a lot of other projects completely unrelated to this.

Status: Started

Labels:
Type-Defect Priority-Medium