Review
- Queries
- All Stories
- Search
- Advanced Search
- Transactions
- Transaction Logs
Advanced Search
Jul 13 2023
Address review
Jul 12 2023
Jul 11 2023
Use useCallback
Fix comments
Sorry, for the delay in the response, I had an exam session.
Add resetScrollBlockingModalStatus function
Early exit in componentDidMount
Jul 6 2023
Shorthand
Added comments
Requesting review to
Jul 5 2023
Removed unused OverlayContext
Move constants to seperate file
Address review
Hey, sorry for the miscommunication. Here are answers to your questions:
- A video showing the bug you are fixing
Jul 4 2023
Updated with a new approach:
when the user is in edit mode the modals scrolling is always unlocked.
Address review
Jul 3 2023
Address review
Added blur and focus listeners
Address review: changed removeEditMode return type
Jun 27 2023
Rebase & fix calling dispatch from reducer
Rebase & fix calling dispatch from reducer
Remove calling dispatch from reducer
Change to function syntax
Change to function syntax
Rebase
Jun 26 2023
address review
Address review
Address review
Jun 23 2023
Address review & fix issue with nested threads (unblocking the navigation didn't work correctly)
Refactor
Jun 21 2023
Refactor
Jun 20 2023
Address review
Reformat
Address @inka comments, disable anchor-overflow when in edit mode
Jun 16 2023
Address review
Jun 15 2023
New version: https://phab.comm.dev/D8224 (to prevent rebase issues)
New version in a new stack: https://phab.comm.dev/D8223
Rebase & new stack
As discussed with @michal, it is now done with a different approach.
Jun 14 2023
After a discussion with @inka we decided to add dynamic height calculation. Now, the maximum height of the edit box will be adjusted to the container height. We will scroll to the edited message only when there is insufficient space for the 150px text area in the edit box.
Added dynamic maximum height, addressed review
@michal created a new stack with this feature: https://phab.comm.dev/D8013
Respond to the review
Jun 12 2023
Jun 2 2023
Add comment
Jun 1 2023
Rebase
May 31 2023
May 30 2023
May 25 2023
Shorthand
Memoize param
May 24 2023
Shorthand
May 23 2023
In D7838#235423, @inka wrote:Why do we want to do that?