Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support new M4 iPad display sizes #2009

Open
carloscabanero opened this issue May 15, 2024 · 22 comments
Open

Support new M4 iPad display sizes #2009

carloscabanero opened this issue May 15, 2024 · 22 comments
Labels
feature Feature requested by users

Comments

@carloscabanero
Copy link
Member

carloscabanero commented May 15, 2024

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

@carloscabanero carloscabanero added bug Bug that's either related to Blink Shell or prevents the user to use the app in normal conditions. feature Feature requested by users and removed bug Bug that's either related to Blink Shell or prevents the user to use the app in normal conditions. labels May 15, 2024
@xaeroseven
Copy link

👍

@prairie-guy
Copy link

Assume this includes new iPadPro M4 as well . . .

@anomaly256
Copy link

anomaly256 commented May 17, 2024

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/ )

@carloscabanero
Copy link
Member Author

carloscabanero commented May 17, 2024

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.

@chrisbennight
Copy link

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

@blerg-sh
Copy link

blerg-sh commented Jun 7, 2024

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

@carloscabanero
Copy link
Member Author

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

@ivahos
Copy link

ivahos commented Jun 11, 2024

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
Copy link

ivahos commented Jun 11, 2024

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.

@carloscabanero
Copy link
Member Author

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

@ivahos
Copy link

ivahos commented Jun 11, 2024

@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)

@carloscabanero
Copy link
Member Author

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!

@ivahos
Copy link

ivahos commented Jun 11, 2024

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)

@carloscabanero
Copy link
Member Author

Sorry, the screenshots are not working :(

@ivahos
Copy link

ivahos commented Jun 11, 2024

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
Copy link

ivahos commented Jun 11, 2024

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

@carloscabanero
Copy link
Member Author

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
Copy link
Member Author

carloscabanero commented Jun 13, 2024

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

@ivahos
Copy link

ivahos commented Jun 13, 2024

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?

@carloscabanero
Copy link
Member Author

I pushed a few hours ago. Let me know.

@ivahos
Copy link

ivahos commented Jun 15, 2024

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 :)

@carloscabanero
Copy link
Member Author

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Feature requested by users
Projects
None yet
Development

No branches or pull requests

7 participants