This should be moved to HOC though

This check is no longer needed
1. It didn't work accurately
2. The compose component is unmounted when the drawer is expanded

`reportID` comes with a delay when the prop comes from onyx
The user first selects a route in the drawer
The route immediately hides the drawer and reviews the report screen
`reportID` have not yet updated and the old report is visible for a moment

Switching to route params changes the `reportID` the moment the user selects
one of the sidebar chat links

Don't trigger a navigation event as this causes a few unintended re-renders

We don't need to trigger a navigation when can review the drawer from any
chat screen we're currently on

…ents

Keep the view mounted when the report doesn't change
Unmount the input when the drawer covers the view
Set autoFocus only for bigger screens as example

…itching

This is now handled by re-mounting
Remove action sorting performed on each render - apply only when actions change

This prop no longer exists
The calculation seems to be handled differently and this is leftover

This is no longer needed - functionality extracted outside

Nicer transition on iOS. I can't see any difference on Android

There are many prop type warnings when this is string.

…from parents"

This reverts commit 2f4a7ea

Additional weight on th PR - These can be proposed as a follow up improvement

Involved logic is already covered on mount and on unmount

…View

ReportActionsView is already handling all the required fetching so it's better
to just encapsulate that logic in a single place

While the Report Screen can just setup a small transition

…ent too

…e input

…ut here

This will:
- remove focus / hide the keyboard when we go back to the LHN
- prevent focusing on the input while it's covered by the LHN
- enable the field when LHN is collapsed (or permanent)

…-loading-state

…-loading-state