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

Add Pixel 8 and 9 skins #601

Merged
merged 7 commits into from
Oct 14, 2024
Merged

Add Pixel 8 and 9 skins #601

merged 7 commits into from
Oct 14, 2024

Conversation

km1chno
Copy link
Contributor

@km1chno km1chno commented Oct 9, 2024

Resolves #583

Screenshots

Pixel 8

Pixel 9

@km1chno km1chno self-assigned this Oct 9, 2024
Copy link

vercel bot commented Oct 9, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
radon-ide ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 14, 2024 10:49am

@km1chno km1chno changed the title @km1chno/add pixel 8 and 9 Add Pixel 8 and 9 skins Oct 9, 2024
Copy link
Collaborator

@filip131311 filip131311 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure why you did not expirience this problem but it seems that mask size is missaligned on pixel9

Screenshot 2024-10-09 at 16 20 40

@filip131311
Copy link
Collaborator

Also It seems that the system does not know on what device it is run
Screenshot 2024-10-09 at 16 36 16

@km1chno
Copy link
Contributor Author

km1chno commented Oct 9, 2024

interesting, I didn't have this issue
i will look into it

This PR fixes pixel 9 mask issue and adds formating
Copy link
Collaborator

@filip131311 filip131311 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I approve, but please let @p-malecki take a look before merging, as he is currently working on devices names, and I am not sure if the name you use will still exist.

Copy link
Contributor

@p-malecki p-malecki left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As @filip131311 mentioned in the last comment, I am working on PR #597 in which we add the option for users to set and change device display names. In that PR, I use displayName for storing the user's custom device name and deviceName for detecting the device type and determining whether it is supported. There are some differences in the way you use them in this PR, but I think it's best to accept it as is and make changes later in #597.

I left some comments but you do not have to address them, they are changes that I intend to implement in #597.

@km1chno km1chno merged commit 4bfd046 into main Oct 14, 2024
3 checks passed
@km1chno km1chno deleted the @km1chno/add-pixel-8-and-9 branch October 14, 2024 11:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add new android device types (pixel 8,9)
3 participants