Titanium JIRA Archive
Titanium SDK/CLI (TIMOB)

[TIMOB-24802] Windows: ListView's scrollToItem scroll position

GitHub Issuen/a
TypeNew Feature
PriorityMedium
StatusClosed
ResolutionFixed
Resolution Date2017-08-08T07:22:15.000+0000
Affected Version/sRelease 6.1.1
Fix Version/sRelease 6.2.0
ComponentsWindows
Labelsparity
ReporterVictor Vazquez Montero
AssigneeKota Iguchi
Created2017-06-09T18:31:36.000+0000
Updated2017-09-18T23:22:26.000+0000

Description

Description

In iOS, if you call the scrollToItem function and you pass an "animation" parameter, you can indicate a 'position' (bottom, middle, none and top) which is the "position to scroll the selected cell to". The issue is that with the implementation as is right now, when scrolling to items in different sections (with several list items on them), the scroll stops at a point in which the section is sometimes positioned in the middle or bottom of the screen, making difficult to the user to tell when does the last section ends and when does the section that they scrolled to begins.

Video Example

[Video](https://www.screencast.com/t/w161OuvI6WEB)

Feature Request

Parity in Windows

Sample Project

[Sample Code](https://propelics.box.com/s/rdot2vvg8js18e0b0ow15vw4gc0jx3uh)

Comments

  1. Victor Vazquez Montero 2017-07-05

    [~kiguchi] hey is there any update on this feature request?
  2. Kota Iguchi 2017-07-05

    [~vvazquezmontero] We need to investigate more on this but I'm guessing that this feature request is feasible. This ticket is most likely to be tackled in next sprint, likely to target for 6.2.0 SDK.
  3. Victor Vazquez Montero 2017-07-05

    Thank you [~kiguchi] :-)
  4. Kota Iguchi 2017-07-18

    https://github.com/appcelerator/titanium_mobile_windows/pull/1042 [~vvazquezmontero] Just pushed a PR, which only implements ListViewScrollPosition.TOP and ListViewScrollPosition.NONE for scrollToItem. There's no MIDDLE and BOTTOM due to the platform API spec (FYI see comment in my PR). Do you think TOP satisfies customer's needs?
  5. Victor Vazquez Montero 2017-07-18

    [~kiguchi] I will let them know the API limitation. This should hopefully be good.
  6. Kota Iguchi 2017-08-07

    I accidentally closed original PR last week. Pushing again: https://github.com/appcelerator/titanium_mobile_windows/pull/1063
  7. Samir Mohammed 2017-08-07

    [~kiguchi] Master has been merged, waiting for 6_2_X
  8. Kota Iguchi 2017-08-08

    6_2_X: https://github.com/appcelerator/titanium_mobile_windows/pull/1068
  9. Kota Iguchi 2017-08-08

    Merged to 6_2_X
  10. Samir Mohammed 2017-08-16

    Verified fix in SDK Version 7.0.0.v20170815065413 and SDK Version: 6.2.0.v20170815130745 Test and other information can be found at: Master: https://github.com/appcelerator/titanium_mobile_windows/pull/1063 6_2_X: https://github.com/appcelerator/titanium_mobile_windows/pull/1068
  11. Victor Vazquez Montero 2017-09-05

    [~kiguchi] Hello kota, Customer wanted to know when the documentation here: http://docs.appcelerator.com/platform/latest/#!/api/Titanium.UI.ListView-method-scrollToItem http://docs.appcelerator.com/platform/latest/#!/api/ListViewAnimationProperties would be updated to reflect the change? best regards, Victor Montero
  12. Kota Iguchi 2017-09-06

    [~vvazquezmontero] Thanks for heads up. Pushing https://github.com/appcelerator/titanium_mobile_windows/pull/1095 https://github.com/appcelerator/titanium_mobile_windows/pull/1094 https://github.com/appcelerator/titanium_mobile/pull/9405 https://github.com/appcelerator/titanium_mobile/pull/9406
  13. Victor Vazquez Montero 2017-09-18

    [~kiguchi] Hello Kota, when will the docs reflect the updates ?
  14. Kota Iguchi 2017-09-18

    We have a new entry for Titanium.UI.Windows.ListViewScrollPosition here: http://docs.appcelerator.com/platform/latest/#!/api/Titanium.UI.Windows.ListViewScrollPosition Then we should be ready to push this: https://github.com/appcelerator/titanium_mobile/pull/9406 now, Will be updated once it is reviewed and merged.

JSON Source