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

Icons not loading in Windows and MacOS #26

Open
abhi211199 opened this issue Oct 14, 2020 · 6 comments
Open

Icons not loading in Windows and MacOS #26

abhi211199 opened this issue Oct 14, 2020 · 6 comments
Labels
good first issue Good for newcomers

Comments

@abhi211199
Copy link
Member

Describe the bug
The app's icon(currently, the moja global icon) successfully loads in Ubuntu, but fails to load in Windows and MacOS based systems.
Making proper entries in package.json and main.js should fix it!

@abhi211199 abhi211199 added the good first issue Good for newcomers label Oct 14, 2020
@gmajan
Copy link
Member

gmajan commented Oct 15, 2020

Thx for this issue.

@waridrox
Copy link
Member

waridrox commented Jan 1, 2021

Describe the bug
The app's icon(currently, the moja global icon) successfully loads in Ubuntu, but fails to load in Windows and MacOS based systems.
Making proper entries in package.json and main.js should fix it!

I tried adding the icon file for specific operating systems, .icns, .png and .ico respectively, and the app icon does change but after rebuilding the app, the app size reduces to 192.8 MB which was 204.5 MB otherwise. The functionality is also hindered where the JSON interface is not loading. I think there are some dependencies that are broken.

app icon visible

Tested on MacOS 11.0.1

Same problem arises with the .exe file on Windows 10

@aornugent
Copy link

@waridrox - can you please describe which dependencies might not be working correctly?

@waridrox
Copy link
Member

waridrox commented Jan 16, 2021

@waridrox - can you please describe which dependencies might not be working correctly?

After researching a bit more about the problem, I found that the problem is not with the dependencies but with the Electron itself which has been linked here.

Screenshot 2021-01-16 at 4 58 05 PM

The app is not able to build properly even on the different electron app versions and methods as suggested in the above issue.
Adding the icon path, does change the icon but removes other images from the app. There are no problems though when testing on local development environment with npm start.

Screenshot 2021-01-16 at 8 09 22 PM

providing the icon path for the electron app

Screenshot 2021-01-16 at 8 13 57 PM

app running perfectly on the local server

I would like @abhi211199 to provide his opinions on the same and see if the issue can be solved.

@Axuby
Copy link

Axuby commented Sep 30, 2021

Nice @waridrox

@shreya024
Copy link

Hi, I am Shreya, a contributor to OpenForce 2022. I would like to work on this issue. I would be making a PR as soon as I am done with resolving the issue. Thank you

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

6 participants