Sibtain Ali

Results 97 comments of Sibtain Ali

> Can you please revert extra commits by resetting them and only have one necessary commit? ![image](https://user-images.githubusercontent.com/24370807/234738655-4b1cac96-8bd1-4436-92fa-81e2d0ae8885.png) > > 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...

@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 `###...