Skip to content

Fix the Java logo with the latest official version#4831

Merged
kenyonj merged 4 commits into
github:mainfrom
brunoborges:master
Aug 13, 2025
Merged

Fix the Java logo with the latest official version#4831
kenyonj merged 4 commits into
github:mainfrom
brunoborges:master

Conversation

@brunoborges

Copy link
Copy Markdown
Contributor

Please confirm this pull request meets the following requirements:

Which change are you proposing?

  • Suggesting edits to an existing topic or collection
  • Curating a new topic or collection
  • Something that does not neatly fit into the binary options above

Editing an existing topic or collection

I'm suggesting these edits to an existing topic or collection:

  • Image (and my file is *.png, square, dimensions 288x288, size <= 75 kB)
  • Content (and my changes are in index.md)

The current logo is wrong (red+blue). The latest Java logo is orange+blue.


Please note: we will close your PR without comment if you do not check the boxes above and provide ALL requested information.

Copilot AI review requested due to automatic review settings July 16, 2025 18:26
@brunoborges brunoborges requested a review from a team as a code owner July 16, 2025 18:26

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.

@jmeridth

Copy link
Copy Markdown
Contributor
topic images should be 288px wide.
Expected: 288
  Actual: 234

@jmeridth jmeridth added the awaiting-response Waiting for a response from PR author label Jul 28, 2025
@DeveloperSiqueira

Copy link
Copy Markdown

Acabei de acessar o HUB e, somente agora, às 00h10, vi a mensagem enviada por vocês. Pelo que entendi, o prazo indicado era até o dia 16 de julho, mas não tive qualquer aviso de acesso no meu e-mail anteriormente.

Segue a correção do novo conforme solicitado.

Acabei de acessar o HUB e, somente agora, às 00h10, vi a mensagem enviada por vocês. Pelo que entendi, o prazo indicado era até o dia 16 de julho, mas não tive qualquer aviso de acesso no meu e-mail anteriormente.

Segue a correção do novo conforme solicitado.
java

@DeveloperSiqueira DeveloperSiqueira left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

java

@kenyonj kenyonj left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

please update the PR with the proper image and we can re-review

@brunoborges

Copy link
Copy Markdown
Contributor Author

Done @kenyonj.

Thanks @DeveloperSiqueira for the fixed logo size

@kenyonj kenyonj left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks great. Thanks for your contribution!

@kenyonj kenyonj added this pull request to the merge queue Aug 13, 2025
Merged via the queue into github:main with commit d055002 Aug 13, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting-response Waiting for a response from PR author

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants