Skip to content
This repository has been archived by the owner on Jun 7, 2021. It is now read-only.

List component has misleading information about horizontal view #392

Open
dseeker opened this issue May 8, 2019 · 6 comments
Open

List component has misleading information about horizontal view #392

dseeker opened this issue May 8, 2019 · 6 comments
Assignees

Comments

@dseeker
Copy link
Contributor

dseeker commented May 8, 2019

The List component, which inspired by Android RecyclerView, is a core component, and it provides the most popular features for using a list of items. which support vertical and horizontal list.

The documentation says list component can have horizontal view, but this is not the case, using the same parameter of the scroller does not result in horizontal slider in web view

@YorkShen
Copy link
Contributor

Sorry for taking so long to response.

I don't understand your question. The List itself can be horizontal or vertical. What's the problem here.

@dseeker
Copy link
Contributor Author

dseeker commented Jun 20, 2019

is there a demo of the list in horizontal mode? couldn't make it work
http://dotwe.org/vue/edd704ff8360bad596e9497d2f0acebd

<list class="list" scroll-direction="horizontal" style="flex-direction: row">
    <cell class="cell" v-for="num in lists">
      <div class="panel">
        <text class="text">{{num}}</text>
      </div>
    </cell>
  </list>

@YorkShen
Copy link
Contributor

YorkShen commented Oct 31, 2019

  1. The scroll-direction is only supported in recycle-list, not list.
  2. You can just change list to hlist, then everything is fine.
  3. Please give give us a PR about hlistif you can, and you can always contact me in mailing list, where I will reply in time. I may check github issues months later, I am sorry if this bothers you.

@dseeker
Copy link
Contributor Author

dseeker commented Nov 9, 2019

Thanks for the information
The goal of this issue is to change the documentation page which says that list is able to have horizontal direction. I don't have a proposal for hlist, I just wanted to point out the info on the current site is misleading and documentation should change to reflect the information you have just provided here.

dseeker added a commit to dseeker/incubator-weex-site that referenced this issue Nov 9, 2019
@dseeker
Copy link
Contributor Author

dseeker commented Nov 9, 2019

@YorkShen opened a PR with documentation changes, please review
#546

@YorkShen
Copy link
Contributor

YorkShen commented Nov 10, 2019

#546 is merged, thanks.

hlist is already implemented in current Weex sdk, it just doesn't show up in our doucumentaion somehow. You could add an document about hlist is possbile, every hevhaior of hlist is the same as list, except for its horizontal direcion.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants