Joen A.

Results 739 comments of Joen A.

> How might we expose aspect ratio controls at the same time? The dropdown in the block toolbar can be a bit hidden for users, I've seen some people miss...

Thanks for working on this. This already feels vastly better than what exists. Nice work. I did encounter a glitch, I was using a portrait image, and resized it first,...

Thanks for keeping at this. I tried the test steps and things seem to work as intended, though I'm curious what's actually new. The min-column width was already in place,...

Nice! This makes good progress, but doesn't yet fully fix 54867. Quick GIF: ![Image](https://github.com/user-attachments/assets/347111cf-537e-4d9e-8bfe-995985526da4) We need only two changes: 1. Rename "Choose Image" to "Add image" (note the lowercase i)...

Nice. Took the latest for a spin: ![status](https://github.com/user-attachments/assets/2a84c2bc-2dd1-4786-846c-ee4aeb82195c) In the above I added a `min-height: 60px;` to the placeholder component. The default state is a bit small: ![Screenshot 2024-08-14 at...

> However, my only concern is that in the small size, the "Connected to post meta" text (lockUrlControlsMessage) is also hidden. Do you have any suggestions on where we can...

Works well! ![Screenshot 2024-08-15 at 09 57 42](https://github.com/user-attachments/assets/0b52f896-40e5-45aa-abe8-5c8ffab35f31) I'll defer to others for a code review. I also imagine that the brief flash of the white material is hard to...

@t-hamano apologies for the direct ping, do you still have bandwidth to look at the e2e tests? Otherwise I might be able to find someone else that can help, let...

Thanks so much for the clarification. I wonder: can we extract the little trick that changes the focusing to avoid the brief flash? It's one we can live without, at...

I would hate to lose the fact that the placeholder functions in narrow contexts, it's entirely broken at the moment. What I'm suggesting is we rewind back to maybe [this...