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

Missing application icon #28

Open
RadekCap opened this issue Dec 22, 2022 · 15 comments · Fixed by #68
Open

Missing application icon #28

RadekCap opened this issue Dec 22, 2022 · 15 comments · Fixed by #68
Assignees
Labels
bug Something isn't working

Comments

@RadekCap
Copy link
Collaborator

When you run an application compiled from the source it has a Netbeans icon instead of a TR icon during switching applications.

@ursjoss ursjoss added the bug Something isn't working label Dec 22, 2022
@ursjoss
Copy link
Collaborator

ursjoss commented Jan 17, 2023

This is not the case on my system. But I have a suspicion. @RadekCap Please check the following:

  • In NetBeans - with the project open - open the "Branding" dialog (Right-click on the the "ThinkingRock" node in the "Projects" group -> "Branding..." -> "Basic" Tab)
  • You can see the place-holder for icons in 6 sizes (16x16 - 1024x1024).Clicking on each will show you the icons chosen for that size.

Image

As you can see, we have actual ThinkingRock icons in the sizes 16x16 to 48x48.However, for 256x256 and larger icons, we don't have an icon specified, falling back to the default Netbeans icon:

Image

Can you experiment by selecting a different random icon for each of those missing sizes and check if that changes the situation on your system? I suspect that the screen resolution and/or pixel depth of your system falls back to one of the larger icons, which is not defined.

In case you can confirm that, we should try to get the icon in the missing sizes from Claire and Jeremy (or alternatively as a vector graphic, so we can convert it to the pngs we need ourselves.

@RadekCap RadekCap self-assigned this Jan 18, 2023
@RadekCap
Copy link
Collaborator Author

I've tried to create a larger icon without any effect on the application icon (on MacOS).

I can observe this behavior both on Linux and MacOS.

I'm going to experiment with this later.

@RadekCap
Copy link
Collaborator Author

Ehm, I tried it again on Linux and a replacement 256x256 icon will solve it.

Maybe I haven't used a proper format on macOS, I'll explore it more.

@ursjoss
Copy link
Collaborator

ursjoss commented Jan 18, 2023

I created a request for larger icons in ticket #67.

@RadekCap
Copy link
Collaborator Author

This solution doesn't work on MacOS. So it'll need another exploration and fix.

@RadekCap RadekCap reopened this Jan 18, 2023
@RadekCap
Copy link
Collaborator Author

Following this guide should fix the issue. I'll look into it later when there will be more time.

https://netbeans.apache.org/wiki/DevFaqExecutableIcon.html

@degrown
Copy link
Contributor

degrown commented Jan 23, 2023

Maybe it's irrelevant, but I can confirm that I don't see the TR icon; however, I don't see either the Netbeans icon, but something like this one (it's a bit different, but you get the idea):

image

@ursjoss
Copy link
Collaborator

ursjoss commented Jan 23, 2023

Maybe it's irrelevant, but I can confirm that I don't see the TR icon; however, I don't see either the Netbeans icon, but something like this one (it's a bit different, but you get the idea):

I think the installer that I provided was built before #68 was merged to main. I will update it and let you know to get the newer one when there...

@RadekCap
Copy link
Collaborator Author

@degrown is working on macOS where this bug is still valid, and I need to investigate it more. It's fixed on Linux.

The icon is the OpenJDK icon.

@thegamer87
Copy link

Hi,
trying to create the bundle for mac os x (through "ant build-mac") I noticed that the icon copied in the bundle is: netbeans-plat/17/harness/etc/applicationIcon.icns, trying to change this icon and doing a new build the bundle comes out with the updated icon, some simple google search point me to this netbeans wiki abount "DevFaqExecutableIcon":
https://netbeans.apache.org/wiki/DevFaqExecutableIcon.html

I hope this can help you to solve the problem.

@ursjoss
Copy link
Collaborator

ursjoss commented Mar 20, 2023

Hi @thegamer87

Thanks for your interest in TR, your report and investigation. The link looks relevant - even though on first glance, it looks like there is no out-of the box solution available from Netbeans. We may be able to implement a workaround though.

@thegamer87
Copy link

thegamer87 commented Mar 20, 2023

I've tried following the instructions in the link under "Mac Icons" and it seems to work, specifically I have

  1. added correct trgtd icon ".icns" file (converted from one of .gif file used by linux application) in a subfolder of branding (obviously the file can be placed anywhere so it is then referenced in the build.xml, see next step)

➜ thinkingrock_git git:(main) ls -l branding/MacOS
total 136
-rw-r--r--@ 1 verrocchiom staff 66774 Mar 20 08:59 trgtd.icns

  1. added to build.xml the descriptor from the documentation, this override build-mac target to replace the default .icns with my .icns (see attached patch file of build.xml)

In this way the bundle come out with my .icns instead of default .icns.

tr_build.xml.patch

@ursjoss
Copy link
Collaborator

ursjoss commented Mar 20, 2023

@thegamer87 Yes, I was thinking of something like that. Would you like to create a PR with your already established solution?

@thegamer87
Copy link

I've opened a PR #127 with more simple but out-of-the-box working solution (that not require any override in build manifest): digging into the build.mac target of suite.xml I've found that MacOS application icon is already read from the "app.icon.icns" property so I've simpy added this to nbproject/project.properties pointing to .icns file that i've added in branding/MacOS folder

@ursjoss
Copy link
Collaborator

ursjoss commented Jan 4, 2024

See also tibirna/qgit#142 for an issue (and a solution) regarding missing icons on Mac.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants