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

qucs-s windows uses old icon mac uses new one #929

Closed
dsm opened this issue Sep 4, 2024 · 7 comments · Fixed by #932 or #933
Closed

qucs-s windows uses old icon mac uses new one #929

dsm opened this issue Sep 4, 2024 · 7 comments · Fixed by #932 or #933
Milestone

Comments

@dsm
Copy link
Collaborator

dsm commented Sep 4, 2024

Hi, this is the difference mac and windows icon and qucs-s.exe normally uses different icon but innosetup changed icon to old icon but not visible in explore it's strange. We should set app icon using Qt API and CMake. I created new.ico based on qucs-s.svg in qucs/bitmaps .

image

@ra3xdh
Copy link
Owner

ra3xdh commented Sep 4, 2024

It's need to replace icons in contrib/InnoSetup/misc It still use 64x64 icons. Maybe switch to higher resolution or vector icon if supported.

@dsm
Copy link
Collaborator Author

dsm commented Sep 4, 2024

256x256 looks good on Full HD screen i changed in innosetup locally and build exe, Qt page says convert icon to ico file, i think svg not supported.

@dsm
Copy link
Collaborator Author

dsm commented Sep 4, 2024

ico file can store multiple resoultion icons in a single file using

magick convert 16.png 32.png 48.png 128.png 256.png -colors 256 icon.ico

@dsm
Copy link
Collaborator Author

dsm commented Sep 5, 2024

I sent a PR #932 to fixed windows icon but app title icon not changed it needs to be changed in Qt code
App icon:
image

title icon:
image

@ra3xdh
Copy link
Owner

ra3xdh commented Sep 5, 2024

The window title icon is set inside QucsApp::InitView I think the existing icon should remain, but XPM should be replaced by SVG or higher resolution.

@ra3xdh ra3xdh added this to the 24.3.1 milestone Sep 5, 2024
@dsm
Copy link
Collaborator Author

dsm commented Sep 5, 2024

#ifndef __APPLE__
  setWindowIcon (QPixmap(QString(":/bitmaps/hicolor/scalable/apps/qucs.svg")));
#else

I tested this usage and succesfully render both app and title svg icon. about diolog also used 128x128 png maybe this one can be used there. Why you think the existing icon should remain (big.qucs.xpm) generally app and title icons are same.

@ra3xdh ra3xdh linked a pull request Sep 5, 2024 that will close this issue
@ra3xdh
Copy link
Owner

ra3xdh commented Sep 5, 2024

I tested this usage and succesfully render both app and title svg icon. about diolog also used 128x128 png maybe this one can be used there.

Yes, this one could be accepted.

thomaslepoix added a commit to thomaslepoix/Qucs-RFlayout that referenced this issue Sep 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants