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

MacOS: Make SVG/HiDPI icons work in the main menu bar #7477

Merged
merged 1 commit into from
Jul 17, 2024

Conversation

eirikbakke
Copy link
Contributor

@eirikbakke eirikbakke commented Jun 13, 2024

Adjust the Icon implementation in ImageUtilities to make SVG/HiDPI icons work in the MacOS menu bar.

This requires Java 9 or above, due to the need to implement the MultiResolutionImage interface.

The effect can be seen e.g. for the Undo and Redo actions in the Edit menu, which have SVG icons present. Some more menu actions will have SVG icons once #7463 is merged.

(I'd upload a screenshot here but I don't have a Mac available at the moment. I have tested the patch on MacOS many times in the last 6 months, though, and it has also been confirmed not to cause trouble on the IDE on my Windows machine.)

@eirikbakke eirikbakke added Platform [ci] enable platform tests (platform/*) UI User Interface os:macos ci:dev-build [ci] produce a dev-build zip artifact (7 days expiration, see link on workflow summary page) labels Jun 13, 2024
@eirikbakke
Copy link
Contributor Author

I see a test failing called "NetBeans / APISupport Modules on Linux/JDK 8 (pull_request)". Is the openide.util.ui module (which contains ImageUtilities) required to work with Java 8? In that case this patch can't work.

@eirikbakke eirikbakke requested a review from mbien June 14, 2024 12:06
@eirikbakke eirikbakke added the do not merge Don't merge this PR, it is not ready or just demonstration purposes. label Jun 14, 2024
@eirikbakke eirikbakke removed the do not merge Don't merge this PR, it is not ready or just demonstration purposes. label Jun 27, 2024
mbien
mbien previously requested changes Jun 27, 2024
Copy link
Member

@mbien mbien left a comment

Choose a reason for hiding this comment

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

  • source/target needs to be changed to release: javac.relese=11
  • and remove the OpenIDE-Module-Java-Dependencies manifest entry again since it is automatically added now

@eirikbakke
Copy link
Contributor Author

@mbien Ah, sorry, I misunderstood that comment to be about something that happened in the other PR. Now done and pushed again... waiting for tests to pass.

@mbien mbien added this to the NB23 milestone Jun 27, 2024
@mbien mbien self-requested a review June 27, 2024 06:51
@neilcsmith-net
Copy link
Member

@mbien this is still marked as requesting changes from you?

@mbien
Copy link
Member

mbien commented Jul 17, 2024

@neilcsmith-net nope, I marked the two points I commented on as resolved - all good from my side.

@neilcsmith-net
Copy link
Member

@mbien you should be able to dismiss the review then if you don't want to switch to approve.

@eirikbakke over to you to merge.

@mbien mbien dismissed their stale review July 17, 2024 13:29

all requests resolved - thanks!

@mbien
Copy link
Member

mbien commented Jul 17, 2024

done, I thought dismiss review would delete everything i said, but all it does is to remove the red marker. So I always pressed the re-request review button to get rid of the red marker.

@eirikbakke eirikbakke merged commit 8d39338 into apache:master Jul 17, 2024
33 checks passed
@eirikbakke
Copy link
Contributor Author

Thanks!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci:dev-build [ci] produce a dev-build zip artifact (7 days expiration, see link on workflow summary page) os:macos Platform [ci] enable platform tests (platform/*) UI User Interface
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants