Sibtain Ali
Sibtain Ali
> Can you please revert extra commits by resetting them and only have one necessary commit?  > > This will resolve history confusion. Extra commits serve no purpose and...
## Proposal ### Please re-state the problem that we are trying to solve in this issue. The emoji selection suggestion is flickering when adding attachment ### What is the root...
@sobitneupane Thanks for the review. The alternative solution seems to work fine. Can you please share a video of what you're seeing?
@sobitneupane I'm wondering why we're adding this sort of complex logic when adding a slight delay solves this gracefully? I agree that we should avoid delays and try to come...
I think it will make more sense to add `toggleReportActionComposeView` to the function [here](https://github.com/allroundexperts/Expensify/blob/151833d2956052a2ba6fe7e85042c31eba3b5c36/src/pages/home/report/ReportActionsView.js#L346). At this place, we can check if scroll offset is close to 0. If so, we...
Still looking for proposals.
Yep, on it today.
@rojiphil I think it should be fine to just clear the report action but why do we need to refetch the report? I don't understand that part. Also, it would...
> > I think it should be fine to just clear the report action but why do we need to refetch the report? I don't understand that part. Also, it...
## 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 `###...