Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

how to LinearLayoutManager is LinearLayoutManager.HORIZONTAL #15

Closed
sunny0092 opened this issue Jan 10, 2021 · 5 comments · Fixed by #38
Closed

how to LinearLayoutManager is LinearLayoutManager.HORIZONTAL #15

sunny0092 opened this issue Jan 10, 2021 · 5 comments · Fixed by #38
Assignees
Labels
bug Something isn't working

Comments

@sunny0092
Copy link

sunny0092 commented Jan 10, 2021

Is your feature request related to a problem?
I changed LinearLayoutManager.HORIZONTAL but it didn't work

Describe the solution you'd like:
holder.binding.recyclerViewContent.run {
setVeilLayout(R.layout.item_home_feature_shimmer)
setAdapter(featureHorizontalAdapter)
setLayoutManager(LinearLayoutManager(context, LinearLayoutManager.HORIZONTAL, false))
addVeiledItems(2)
}

thanks admin

@skydoves skydoves self-assigned this Jan 10, 2021
@skydoves skydoves added the bug Something isn't working label Jan 10, 2021
@skydoves
Copy link
Owner

Actually, it is not supporting the HORIZONTAL now but I will consider adding the HORIZONTAL.
Thanks for your issue :)

@G00fY2
Copy link

G00fY2 commented Mar 16, 2021

This should be stated in the Readme!

@Jigar-zest
Copy link

Any ETA on this?

@avalanchas
Copy link

avalanchas commented Jan 23, 2024

@skydoves do I see it correctly that you would have to set different layout params in the VeilLayout:addMaskElements to allow for horizontal alignment? Is this something you would accept a PR for?

@skydoves
Copy link
Owner

Hey guys, the new version 1.1.4 has been released, and now you can use LinearLayoutManager.HORIZONTAL.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants