Skip to content

Conversation

@jynus
Copy link
Contributor

@jynus jynus commented May 2, 2023

One-click deploy used to show an Android icon, when available. Nowadays, remote debug (aka one-click deploy) for web and Android found under the same submenu in the top right buttons.

Show the new icon with the submenu open instead.

Image has been lightly edited from a 4.0.X screenshot to avoid distracting elements.

Fixes #7226

@paddy-exe paddy-exe added bug area:getting started Issues and PRs related to the Getting Started section of the documentation topic:buildsystem content:images Issues and PRs involving outdated or incorrect images in articles labels May 2, 2023
Copy link
Contributor

@paddy-exe paddy-exe left a comment

Choose a reason for hiding this comment

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

Looks good to me. However I think we switched to .webp files for images now so this might be a good option now to switch this image as well.

@jynus
Copy link
Contributor Author

jynus commented May 2, 2023

I did a few tries and I couldn't get a good lossless webp image that was smaller than the png. What tools would you recommend for format change + size optimization (I already tried cwebp)?

@paddy-exe
Copy link
Contributor

I did a few tries and I couldn't get a good lossless webp image that was smaller than the png. What tools would you recommend for format change + size optimization (I already tried cwebp)?

From the docs: https://docs.godotengine.org/en/stable/contributing/documentation/docs_image_guidelines.html

https://squoosh.app/

One-click deploy used to show an Android icon, when available.
Nowadays, remote debug (aka one-click deploy) for web and Android
found under the same submenu in the top right buttons.

Show the new icon with the submenu open instead.

Image has been lightly edited from a 4.0.X screenshot to avoid
distracting elements. New image sent in .webp format.

No other references to the old image found on docs.

Fixes godotengine#7226
@jynus
Copy link
Contributor Author

jynus commented May 2, 2023

Screenshot_20230502_125132

@mhilbrunner mhilbrunner merged commit 97fc78b into godotengine:master May 2, 2023
@mhilbrunner
Copy link
Member

Thank you! Merged.

@jynus jynus deleted the remote_debug branch June 16, 2023 15:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:getting started Issues and PRs related to the Getting Started section of the documentation bug content:images Issues and PRs involving outdated or incorrect images in articles topic:buildsystem

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Click and Deploy image reference is wrong

3 participants