Fixes Overlapping Conversation Banner Position for Hidden Toolbar

[Sagar0-0]

First time contributor checklist

• I have read how to contribute to this project
• I have signed the Contributor License Agreement

Contributor checklist

• I am following the Code Style Guidelines
• I have tested my contribution on these devices:

• Device A, Android X.Y.Z
• Device B, Android Z.Y
• Virtual device W, Android Y.Y.Z

• My contribution is fully baked and ready to be merged as is
• I ensure that all the open issues my contribution fixes are mentioned in the commit message of my first commit using the Fixes #1234 syntax

---

Description

Before:

After:
Store the Toolbar height in a variable and use it in MultiSelect Mode.

[greyson-signal]

Ah, good catch on the bug. I agree it should be fixed, but in general, I try to avoid manually setting things like margins and such dynamically whenever possible. Is there something in the view hierarchy that could be fixed so that this overlap doesn't happen?

[Sagar0-0]

Is there something in the view hierarchy that could be fixed

I'll explore the possibilities and update it here

[Sagar0-0]

@greyson-signal Only setting the isInvisible can work instead of setting visible = false. Are there any cons for using this?

[greyson-signal]

Changing visibility is fine, I would just like to avoid having to manually change margins in code

[Sagar0-0]

Changing visibility is fine, I would just like to avoid having to manually change margins in code

This change I've proposed won't be required anymore, we can only change the visible= false to isInvisible = true. And everything will work as expected.

[greyson-signal]

That sounds much better, will TAL when the PR is updated

[Sagar0-0]

@greyson-signal there we go!
Also, I will slow down any new contributions for now and will try to make sure all my opened PRs are getting merged.

[greyson-signal]

Looks great, thanks! Will go in 7.34