Child elements overlapping about android-flowlayout

[ApmeM]

HI, sorry, I cant find where are they overlapping.
As I can see from code and image flowlayout is using only for letters "DSTQQS", the other image parts are using different linear layouts.
As I can see those letters are not overlapping each other, and just did not fit in parent container (or this is a problem?) - maybe because of its size.
Can you provide more details (and remove information that is not related to the problem).

[douglasjunior]

I need to display 7 letters within the FlowLayout (D S T Q Q S S), and if you look at the screenshot you will see that the last letter is overlapped, it should be move to the second line, should not it?

Apparently the problem is with the margin. I put the letters in a "LinearLayout" to test, because only the "TextView" had the same problem.

I think I got a way to work. I kept the LinearLayout and put the margin in TextView.

<LinearLayout
  android:orientation="horizontal"
  android:layout_width="wrap_content"
  android:layout_height="wrap_content">
    <TextView
      android:layout_width="wrap_content"
      android:layout_height="wrap_content"
      android:textAppearance="?android:attr/textAppearanceMedium"
      android:text="D"
      android:singleLine="true"
      android:id="@+id/tvDomingo"
      android:layout_margin="5dp" />
</LinearLayout>

Looking at the source code of the FlowLayout, I noticed that he measures the child's size to see if it should be added to the current row or the next.

Apparently, when the margin was in LinearLayout, this measures was not correct.

Sorry, I put all the source code of my layout, so that they could understand the complete structure, and look at the wrap_content and fill_parent.

[ApmeM]

Hi, I have found, that it is reproducable for this layout as well:

<?xml version="1.0" encoding="utf-8"?>
<org.apmem.tools.layouts.FlowLayout
    xmlns:android="http://schemas.android.com/apk/res/android"
    xmlns:f="http://schemas.android.com/apk/res-auto"
    f:debugDraw="true"
    android:layout_width="wrap_content"
    android:layout_height="wrap_content">

    <TextView
        android:layout_margin="5dp"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:text="D"/>

</org.apmem.tools.layouts.FlowLayout>

and the problem was in spacings calculations.
I have fixed it and also create tests.

[douglasjunior]

@ApmeM Thank you for your dedication.

Off: I'm curious. You have planned to convert the FlowLayout as a LayoutManager for RecyclerView?

[dadino]

When will you release the updated library?

[ApmeM]

@douglasjunior : before your comment I even did not know about RecyclerView and it LayoutManagers. Therefore I did not have such plans, but now I will check it. I have created another issue: Support for 'RecyclerView' #59.
Can you please add example of usage or any code that is already implementing this feature to #59 issue so I can check and apply it to this FlowLayout?

@dadino : I was going to fix #50 issue before release. I think release will be today or tomorrow.