Account created on 24 July 2012, almost 12 years ago
  • Senior Full Stack Drupal Developer at Agileana 
#

Merge Requests

More

Recent comments

🇺🇾Uruguay rpayanm

We could compare them using "==" or apply the attached patch.

🇺🇾Uruguay rpayanm

It works like a charm. Thank you!

🇺🇾Uruguay rpayanm

Gutenberg >= 3.0

🇺🇾Uruguay rpayanm

I added some suggestions and fixed the test error. Please review.

🇺🇾Uruguay rpayanm

rpayanm made their first commit to this issue’s fork.

🇺🇾Uruguay rpayanm

Added requirements.

🇺🇾Uruguay rpayanm

I tried this patch and works well.
But I had the same issue as #45 on a 1400px width when the sidebar is shown:

There is no issue is the sidebar is hidden.

🇺🇾Uruguay rpayanm

I confirm that the error is still in v1.16 and the #7 patch solves the issue.

🇺🇾Uruguay rpayanm

rpayanm made their first commit to this issue’s fork.

🇺🇾Uruguay rpayanm

I have created a patch for the 8.x-10.x version.
Please review.

🇺🇾Uruguay rpayanm

rpayanm made their first commit to this issue’s fork.

🇺🇾Uruguay rpayanm

rpayanm made their first commit to this issue’s fork.

🇺🇾Uruguay rpayanm

rpayanm made their first commit to this issue’s fork.

🇺🇾Uruguay rpayanm

rpayanm made their first commit to this issue’s fork.

🇺🇾Uruguay rpayanm

rpayanm made their first commit to this issue’s fork.

🇺🇾Uruguay rpayanm

I swapped out withConsecutive() for willReturnOnConsecutiveCalls().

🇺🇾Uruguay rpayanm

rpayanm made their first commit to this issue’s fork.

🇺🇾Uruguay rpayanm

rpayanm made their first commit to this issue’s fork.

🇺🇾Uruguay rpayanm

rpayanm made their first commit to this issue’s fork.

🇺🇾Uruguay rpayanm

oh! makes sense, awesome, thank you @poker10

🇺🇾Uruguay rpayanm

I rerolled the patch and applied the #27's suggestions. Please review.

🇺🇾Uruguay rpayanm

Weird, the same code failed using an MR.

🇺🇾Uruguay rpayanm

rpayanm made their first commit to this issue’s fork.

🇺🇾Uruguay rpayanm

rpayanm made their first commit to this issue’s fork.

🇺🇾Uruguay rpayanm

rpayanm made their first commit to this issue’s fork.

🇺🇾Uruguay rpayanm

rpayanm made their first commit to this issue’s fork.

🇺🇾Uruguay rpayanm

rpayanm made their first commit to this issue’s fork.

🇺🇾Uruguay rpayanm

I added # because we can have URL like this https://example.com?foo=1#my-string.
And tried to fix the tests.

🇺🇾Uruguay rpayanm

rpayanm made their first commit to this issue’s fork.

🇺🇾Uruguay rpayanm

Sorry, I can see the all tests passed. Which tests failed?

🇺🇾Uruguay rpayanm

rpayanm made their first commit to this issue’s fork.

🇺🇾Uruguay rpayanm

rpayanm made their first commit to this issue’s fork.

🇺🇾Uruguay rpayanm

rpayanm made their first commit to this issue’s fork.

🇺🇾Uruguay rpayanm

rpayanm made their first commit to this issue’s fork.

🇺🇾Uruguay rpayanm

@smustgrave sorry, I don't see the comment, can you link it in a comment, please.

🇺🇾Uruguay rpayanm

rpayanm made their first commit to this issue’s fork.

Production build 0.69.0 2024