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 does android:orientation work on the widget? #124

Closed
tiagoalmeida92 opened this issue May 19, 2017 · 6 comments
Closed

How does android:orientation work on the widget? #124

tiagoalmeida92 opened this issue May 19, 2017 · 6 comments
Assignees
Labels

Comments

@tiagoalmeida92
Copy link

Had some problems I though it was not necessary, and nothing shows when it's horizontal.

@zawadz88
Copy link
Contributor

Hi,
You shouldn't be using this attribute on this View. It works only in the vertical mode. If you look at ms_stepper_layout.xml you should see why nothing is shown.

What were you trying to achieve?

@tiagoalmeida92
Copy link
Author

Oh I understand the view extends LinearLayout, I think it would be best if you force vertical orientation in code, otherwise other developers might forget to add the attribute in the xml.

Great lib thanks

@zawadz88
Copy link
Contributor

Hi @tiagoalmeidagit,
This attribute shouldn't be needed at all when declaring a StepperLayout in your layout XML.
You're saying that if you don't specify it a horizontal orientation is used?

@tiagoalmeida92
Copy link
Author

tiagoalmeida92 commented May 19, 2017

Yes,

in your getting started wiki the example has orientation vertical, I tried to remove it and had problems.

<com.stepstone.stepper.StepperLayout xmlns:android="http://schemas.android.com/apk/res/android"
xmlns:app="http://schemas.android.com/apk/res-auto"
android:id="@+id/stepperLayout"
android:layout_width="match_parent"
android:layout_height="match_parent"
android:orientation="vertical"
app:ms_stepperType="progress_bar" />

@zawadz88
Copy link
Contributor

Wow, I can't believe we actually missed it :P This is definitely a bug.
Thank you for reporting this!
That's something we definitely have to fix.

@zawadz88
Copy link
Contributor

The changes are already on master and will be released in the upcoming patch release.
@tiagoalmeidagit again thank you for reporting this

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

No branches or pull requests

2 participants