Jakub Kosmydel

Results 10 comments of Jakub Kosmydel

> I can reproduce it. Couldn't find a recent issue, looks like @kosmydel fixed it once before here in Oct 2023: #24502 The mentioned issue was related to mWeb only,...

> The code looks good to me. I assume that inverting the list actually apply a transformation that is not applied to the content origin offset, right? That's right. I...

I've addressed the review and solved the mentioned issues

Hey, today I won't have time to investigate it. > we're doing this transform at the wrong layer I think Could you provide more details on where the change should...

> Ah, got it. Wow... that still feels like quite the stretch of a name? Why not name it something more friendly like `AvatatarWithArrowButton` or something? I guess that's a...

Hey, I'm Kuba from Software Mansion and I will work on this issue. The draft of the PR is already prepared, but we are waiting for the #35603 to be...

> If this requires C+ review, I can help cc @mountiny, as this PR blocks other design changes

Ready for review! I've pushed the wrong commit here by mistake, that's why we have one revert.

> Not sure what's happening. Lots of checks are failing Looks like it is broken on main. https://github.com/Expensify/App/pull/35972 should fix it.

Hello, I made changes last week and applied a transform only where necessary. I would appreciate it if someone could review and confirm that these changes are what we want....