Abdelhafidh Belalia
Abdelhafidh Belalia
What a solution! :joy: Given that this is an equally valid solution I'm okay with that. But let's keep the border in a class so just rename chatItemAttachBorder to something...
@narefyev91 Please tag me again once this is ready for review
## Reviewer Checklist - [X] I have verified the author checklist is complete (all boxes are checked off). - [X] I verified the correct issue is linked in the `###...
@narefyev91 Can you please update the tests to: 1. Open App 2. Open any chat 3. Verify the composer border does not overlap with the attachment button and tag me...
@narefyev91 Thank you!
@PrashantMangukiya Thanks for the proposal. Your RCA makes sense. But I think we need to understand why such change was required. @yuwenmemon Was there any reason to change the default...
@luacmartins ~I'm not sure I understood what you are referring to exactly, can you please attach a video and edit the OP if you think we should handle that too.~...
@luacmartins Yes, let's add that.
@PrashantMangukiya Please update your proposal to cover the wrong animation bug as well.
@dukenv0307 Thanks for the proposal. ### Double animation bug The RCA is correct. But the suggested fix is already proposed by @PrashantMangukiya as an alternative solution. You mentioned that setting...