Revision as of 09:25, 6 December 2013 by hamishwillee (Talk | contribs)

(diff) ← Older revision | Latest revision (diff) | Newer revision → (diff)

Talk:LongListSelector with bindable SelectedItem and better scrolling

From Nokia Developer Wiki
Jump to: navigation, search


Hamishwillee - Subedited/Reviewed

Hi Nico

Very nice article indeed - well explained. I've given it a minor subedit for wiki style and English. Please check I didn't break anything in the updates, and that you are OK the changes are "improvements". Mostly it is the same - minor changes in the introduction and I also ended up linking to the example test code for the extended class rather than your other repository. What I was trying to get to in the introductory changes are that the impact of the changes is mostly for MVVM, where you really care about "how" the problem is fixed.

In terms of room for improvement the only thing I can think of is that I don't really like the name. How about "LongListSelector with bindable SelectedItem and better scrolling" (I like specific topic names).

It might also be worth considering adding a screenshot of a LongListSelector in the introduction. Normally an image is a good idea as it immediately gives readers a flavour of what it is about. Not sure if it is necessary though given how standard this control is.

Thank you again



hamishwillee (talk) 09:06, 2 December 2013 (EET)

NicoVermeir -

Title can change sure, I like your title :)

I don't think an image is really needed because the control should be used in every wp app that has lists, but it couldn't hurt I suppose

NicoVermeir (talk) 09:28, 2 December 2013 (EET)

Hamishwillee - Moved

Thanks. I also agree on the image.

Really do appreciate you going "over and above" on this one.

Will delete these comments next time round as they are all addressed.

hamishwillee (talk) 01:41, 3 December 2013 (EET)

Hamishwillee - Source code license

OK if I copy the source to wiki and mark with MIT license?

hamishwillee (talk) 08:25, 6 December 2013 (EET)