43

When i try to set the visibility of the group on button click,it doesn't affect the view's visibility.Using com.android.support.constraint:constraint-layout:1.1.0-beta4. I've tried setting it element-wise without problems,but no success with the group.

My MainActivity.kt

private fun toggleLoginUI(show: Boolean) {
    if (show) {
        group.visibility = VISIBLE
    } else {
        group.visibility = INVISIBLE
    }
}

fun onClick(view: View) {
    when (view.id) {
        R.id.button -> toggleLoginUI(true)
        R.id.button4 -> toggleLoginUI(false)
    }
}

My activity_main.xml

    <android.support.constraint.ConstraintLayout..

            <TextView
                android:id="@+id/textView"
... />

            <TextView
                android:id="@+id/textView2"
... />

            <Button
                android:id="@+id/button"
.../>

            <Button
                android:id="@+id/button4"
... />


            <android.support.constraint.Group
                android:id="@+id/group"
                android:layout_width="wrap_content"
                android:layout_height="wrap_content"
                android:visibility="visible"
                app:constraint_referenced_ids="textView,textView2" />
            </android.support.constraint.ConstraintLayout>
SS2095
  • 433
  • 1
  • 5
  • 6

5 Answers5

40

Update: This was reported as fixed in ConstraintLayout version 2.0.0 beta 6. See bug fixes for ConstraintLayout 2.0.0 beta 6 .



This looks like a bug to me. GONE works but INVISIBLE doesn't and I think it should. It may be worth a bug report unless someone can post where my thinking is wrong. (I am using constraint-layout:1.1.0-beta4.)

In the meantime, here is a work-around that explicitly retrieves the ids within the group and sets the visibility of each retrieved view.

Within MainActivity.kt

private fun toggleLoginUI(show: Boolean) {
    if (show) {
        setGroupVisibility(mLayout, group, Group.VISIBLE)
    } else {
        setGroupVisibility(mLayout, group, Group.INVISIBLE)
    }
}

private fun setGroupVisibility(layout: ConstraintLayout, group: Group, visibility: Int) {
    val refIds = group.referencedIds
    for (id in refIds) {
        layout.findViewById<View>(id).visibility = visibility
    }
}

mLayout is the ConstraintLayout.

Update: Here is another work-around that takes advantage of the fact that changing to/from GONE works as expected:

private fun toggleLoginUI(show: Boolean) {
    if (show) {
        group.visibility = GONE
        group.visibility = VISIBLE
    } else {
        group.visibility = GONE
        group.visibility = INVISIBLE
    }
}
qwertyfinger
  • 634
  • 7
  • 16
Cheticamp
  • 61,413
  • 10
  • 78
  • 131
  • that works but was hoping for a way that didn't require having to loop through the elements.thanks for the answer.hope it gets resolved. – SS2095 Dec 20 '17 at 12:24
  • @SS2095 If it's a bug then you will have to wait for a future release. (And I think that it is a bug.) This is the best that can be done in the interim if you need that functionality. – Cheticamp Dec 20 '17 at 12:28
  • Has somebody reported that bug and has a link to it? – Torsten Grote Mar 01 '19 at 18:15
  • 1
    @TorstenGrote Maybe https://issuetracker.google.com/issues/117485026? I also see a reference to a `ConstraintLayout` 2.0 "Layer" feature that may be somehow applicable. – Cheticamp Mar 01 '19 at 18:28
  • I actually raised an issue related to this a while ago as I agree with @Cheticamp that this is a bug in the Group class: https://issuetracker.google.com/issues/130524019 – Julien Arzul Dec 11 '19 at 04:37
33

You can also simply call requestLayout method after changing Group visibility to View.INVISIBLE.

fun makeGroupInvisible(group: Group) {
    group.visibility = View.INVISIBLE
    group.requestLayout()
}

Problem is that android.support.constraint.Group updates visibility of its members in updatePreLayout method which is called from onMeasure in ConstraintLayout.

lukjar
  • 7,220
  • 2
  • 31
  • 40
6

android.support.constraint.Group has a public method

 public void updatePreLayout(ConstraintLayout container) {
   ...
 }

that updates children visibilities, so calling

dataGroup.visibility = if (visible) View.VISIBLE else View.INVISIBLE
dataGroup.updatePreLayout(root)

worked for me

Oliver1123
  • 111
  • 1
  • 6
1

Had the same problem and nothing from above helps. My solution was to setVisibility(viewId, ConstraintSet.VISIBLE) inside constraint set and apply that to ConstraintLayout view.

For example:

myContstraintSet.apply {
    setVisibility(firstGroup.id, ConstraintSet.VISIBLE)
    setVisibility(secondGroup.id, ConstraintSet.GONE)

    connect(oneView.id, ConstraintSet.BOTTOM, R.id.secondView, ConstraintSet.TOP)
    clear(anotherView.id, ConstraintSet.TOP)
}
myContstraintSet.applyTo(myConstraintLayout)
zoki
  • 535
  • 1
  • 3
  • 17
0

just add follow line you can change it . so it visible .

group.visibility=ConstraintLayout.GONE
Divyesh Kalotra
  • 204
  • 1
  • 2
  • 13
  • 3
    I don't agree with this answer, this question is specifically about using View.INVISIBLE. Using ConstraintLayout.GONE is the same as using View.GONE. It's a different type of visibility setting that works. – filipproch Jun 04 '18 at 08:18