CommandFusion / iViewer-For-Android

CommandFusion iViewer for Android
2 stars 0 forks source link

First list item not updating properly #84

Closed DevinCook closed 11 years ago

DevinCook commented 11 years ago

Ok,

I appears the first item added to list is drawing before it's strings are initialized. After scrolling down and back to the top it appears correctly.

fpillet commented 11 years ago

Does this happen with build #105 that has been released this week? The list refresh issue is an issue that has specifically been handled in this one, so if you're still seeing it we'd like to get our hands on your GUI to investigate.

DevinCook commented 11 years ago

From the debugger:

Javascript starting up. iViewer version: 1.0 (#105-a116da3)

Here is the demo gui. It seems to happen on the first fill. Press Setup List a few times to add items and you should see the top item sub added but string not filled or filled improperly.

After there are enough items to allow scrolling, you can fix the issue by scrolling down and back.

From: Florent Pillet [mailto:notifications@github.com] Sent: Saturday, December 01, 2012 1:02 PM To: CommandFusion/iViewer-For-Android Cc: Devin Cook (Triton) Subject: Re: [iViewer-For-Android] First list item not updating properly (#84)

Does this happen with build #105 that has been released this week? The list refresh issue is an issue that has specifically been handled in this one, so if you're still seeing it we'd like to get our hands on your GUI to investigate.

— Reply to this email directly or view it on GitHubhttps://github.com/CommandFusion/iViewer-For-Android/issues/84#issuecomment-10922072.

DevinCook commented 11 years ago

With further testing I discovered:

self.buildList = function() {
    CF.listRemove('l100');
    CF.listAdd("l100", [{s1: "Item 1a",s2: "Item 1b",},]);
    CF.listAdd("l100", [{s1: "Item 2a",s2: "Item 2b",},]);
    CF.listAdd("l100", [{s1: "Item 3a",s2: "Item 3b",},]);
    CF.listAdd("l100", [{s1: "Item 4a",s2: "Item 4b",},]);
};

When I run this code it clears the list, but on Android I get 4a,4b, 1a,1b, 2a,2b, 3a,3b.

On iViewer Next I get the expected result.

fpillet commented 11 years ago

Issue fixed for next build. Thanks a lot for reporting it!