Jakub Kosmydel
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....