blink icon indicating copy to clipboard operation
blink copied to clipboard

Support new M4 iPad display sizes

Open carloscabanero opened this issue 1 year ago • 22 comments

Via https://discord.com/channels/441939332442619905/441940293240225793/1240131069433348116

carloscabanero avatar May 15 '24 02:05 carloscabanero

👍

xaeroseven avatar May 15 '24 19:05 xaeroseven

Assume this includes new iPadPro M4 as well . . .

prairie-guy avatar May 17 '24 02:05 prairie-guy

Assume this includes new iPadPro M4 as well . . .

Yep, only the pros are on M4 at this stage.

I'm not able to view the discord link - does this report include the bug regarding the layout (cover/fit/fill) being ignored and text truncated at the top of the screen?

If not I'll open a new issue about that (ref: https://www.reddit.com/r/BlinkShell/comments/1ct7i0h/ipads_rounded_corners_cuts_text_in_blink/ )

anomaly256 avatar May 17 '24 09:05 anomaly256

Yes, this should cover all the cases, Reddit issue included.

We should have this on 17.3.0 right before WWDC, but if anything happens, may release a patch sooner than that.

carloscabanero avatar May 17 '24 14:05 carloscabanero

Discord link does not work for me - just gives a "no text channels error"

chrisbennight avatar May 19 '24 01:05 chrisbennight

@carloscabanero Any uodates on when a fix for this will be released?

ghost avatar Jun 07 '24 11:06 ghost

TF today and everyone over the weekend-Monday, once approved.

carloscabanero avatar Jun 07 '24 13:06 carloscabanero

I just downloaded v17.3.0.903 and it still do not work on my 13” M4 imac pro. Still get text cut by the corners and when I double press command the layout option is missing from the shortcut bar. Changing the default layout in Settings - > Appearance does have no effect on new terminal screens.

ivahos avatar Jun 11 '24 06:06 ivahos

The 905 build works now apart from one small nitpicking issue on the 13” m4 ipad pro. It looks like the offset from the top left corner of the screen is a couple of pixels to short. The b in the blink prompt looses a pixel or 2 at the top. Other than that it works great now.

ivahos avatar Jun 11 '24 12:06 ivahos

@ivahos Are you having any issues with the Software Keyboard? Is the Smart Keys bar showing properly?

carloscabanero avatar Jun 11 '24 12:06 carloscabanero

@ivahos Are you having any issues with the Software Keyboard? Is the Smart Keys bar showing properly?

I am using the magic keyboard (a hw keyboard)

ivahos avatar Jun 11 '24 12:06 ivahos

Would you mind to disconnect it and let me know if those are showing properly? I cannot replicate in the Simulator and want to make sure it is working and displaying as it should. Thanks!

carloscabanero avatar Jun 11 '24 12:06 carloscabanero

Would you mind to disconnect it and let me know if those are showing properly? I cannot replicate in the Simulator and want to make sure it is working and displaying as it should. Thanks!

It actually gets a little worse with the SW keyboard Here are 2 screenphotos (since the screenshots dont show the curved display) https://webserver.webhotel.au/IMG_0827.jpg (with hw keyboard) https://webserver.webhotel.au/IMG_0828.jpg (with SW keyboard)

ivahos avatar Jun 11 '24 12:06 ivahos

Sorry, the screenshots are not working :(

carloscabanero avatar Jun 11 '24 12:06 carloscabanero

Sorry, the screenshots are not working :(

Sorry, wrong machine. https://webby.webhotel.au:8443/IMG_0827.jpg (with hw keyboard) https://webby.webhotel.au:8443/IMG_0828.jpg (with SW keyboard)

ivahos avatar Jun 11 '24 12:06 ivahos

As you see in the 2nd screenphoto the b looks more like a o. Reconnecting the hw keyboard does not revert the shift to the pre-detachment state. I have to force quit blink and restart to get back to the 1st screenshot state

The font size also make a difference. It looks like the text starts to high up on the display. when I use Cmd+- to make the font smaller if I go far enough the first lines b disappears behind the curve and the 2nd lines b starts to loose the bar in the b

edit2 I made another screenphoto of btop running in a tmux session with a very small font. It shows the problem also happens at the bottom of the display.

https://webby.webhotel.au:8443/IMG_0829.jpg. Sorry about the sharpness and the dirty screen

ivahos avatar Jun 11 '24 13:06 ivahos

Thank you for the thorough testing! I think the issue is that although the resolution and corners in these devices has changed slightly, they haven't changed the insets themselves. I will try to adjust from our side.

Thanks again!!

carloscabanero avatar Jun 11 '24 14:06 carloscabanero

Fixed now. So although new OLED iPads have an effective usable size of 20 px more, that means the radius is also slightly bigger so you need to adjust the margins and make them bigger by around 3 pixels. It is more dramatic on the "safe fit", where you almost go down a half line just to make the left and right margins 0. Anyway, fitted to pixel perfection ;)

This was before Screenshot 2024-06-13 at 4 37 46 PM

This is after Screenshot 2024-06-13 at 4 38 06 PM

carloscabanero avatar Jun 13 '24 20:06 carloscabanero

Fixed now. So although new OLED iPads have an effective usable size of 20 px more, that means the radius is also slightly bigger so you need to adjust the margins and make them bigger by around 3 pixels. It is more dramatic on the "safe fit", where you almost go down a half line just to make the left and right margins 0. Anyway, fitted to pixel perfection ;)

This was before Screenshot 2024-06-13 at 4 37 46 PM

This is after Screenshot 2024-06-13 at 4 38 06 PM

When will this update reach testflight?

ivahos avatar Jun 13 '24 22:06 ivahos

I pushed a few hours ago. Let me know.

carloscabanero avatar Jun 14 '24 22:06 carloscabanero

I pushed a few hours ago. Let me know.

Thanks for the fix. Looks much better now. No clipping of characters at the top and no clipping at the bottom unless you make the font size so small that the characters are unreadable anyway :)

ivahos avatar Jun 15 '24 02:06 ivahos

Thanks for confirming. Looks like we are wrapping up for release then (yay!).

carloscabanero avatar Jun 15 '24 03:06 carloscabanero

Done 17.3.0

carloscabanero avatar Jul 01 '24 17:07 carloscabanero