Center Justify about android-flowlayout

[ApmeM]

Hi, I have included your pull request #23 into master branch, but it looks like it contains a bug therefore I cant push it into maven.
With the configuration checked into master branch on my mobile phone it start looks like this:

You can see yellow arrow on the bottom. It is overlapped with the button.

Can you fix this issue please?

[z0lope0z]

👍 on fix @jpshelley

[jpshelley]

@ApmeM @z0lope0z Sorry guys been super busy. Don't know how soon I can look at fixing it. What is the yellow arrow for? Everything shows up fine when using it on my device.

[ApmeM]

Hi, yellow arrow in debug mode means child specified horizontal layout. In example app you can find it here:

        <Button
                f:layout_horizontalSpacing="32dip"
                android:layout_width="wrap_content"
                android:layout_height="wrap_content"
                android:text="@string/button_horizontalSpacing"/>

Can you try to open example from app folder. Maybe I have merged it incorrectly into main branch? Can you please check it as well?

Thanks.

[jpshelley]

@ApmeM I'll try to give it a shot this weekend. Should have some free time available.

[ApmeM]

Hi,

I have checked the code you and @AndreasMattsson have made and found, that center justification is already implemented in 'gravity' code.

To do center justification you can use (see the example in MyActivity class:

    layout.setGravity(Gravity.CENTER);

Therefore I've reverted this pull request #23 (but github still think it is merged into main branch).

BTW I have found, that this yellow line is incorrect 'debug' presentation because of 'horizontal' button was increased in size. I am not sure how it should looks like therefore I think it is ok for now.

I will upload latest code to maven repository at the end of this week.

[jpshelley]

Thanks for digging into that @ApmeM I haven't had any time, to look into it. The fork I used works okay from what I can tell. but if you suggest setting the Gravity is preferred, then I'll try to look into it as well.

[jekiapp]

oh I found out I can achieve it by setting Gravity.FILL_HORIZONTAL on layout, and set Gravity.CENTER on child. example:

flowLayout.setGravity(Gravity.FILL_HORIZONTAL);
FlowLayout.LayoutParams lp = new FlowLayout.LayoutParams(ViewGroup.LayoutParams.WRAP_CONTENT,ViewGroup.LayoutParams.WRAP_CONTENT);
// you can set the margin here lp.setMargins();
lp.setGravity(Gravity.CENTER);
child.setLayoutParams(lp);
flowLayout.addView(child);