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

scide: Fix missing icon #4269

Merged
merged 1 commit into from
Jan 27, 2019
Merged

Conversation

nhthn
Copy link
Contributor

@nhthn nhthn commented Jan 23, 2019

Purpose and Motivation

fixes #3905 -- SCIDE is missing an icon. turns out it was something silly, the QIcon path should be :icons/blah rather than :/icons/blah.

unfortunately we can't use the svg icon because of #2646 (icon appears tiny on some linux DEs), so i just used the PNGs.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)

Checklist

  • This PR is ready for review

@nhthn nhthn added this to the 3.10.2 milestone Jan 23, 2019
Copy link
Contributor

@mossheim mossheim left a comment

Choose a reason for hiding this comment

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

the icon is fine on my machine, I suppose I am not the intended target of this though.

@patrickdupuis
Copy link
Contributor

On Ubuntu 18.04, the SCIDE icon shows up again in the "About SuperCollider" window. I haven't noticed any other changes.

icon

@mossheim mossheim merged commit e7c9dcc into supercollider:3.10 Jan 27, 2019
@nhthn nhthn deleted the topic/scide-icon branch January 28, 2019 07:04
@nhthn nhthn mentioned this pull request Feb 4, 2019
@jreus
Copy link
Contributor

jreus commented Nov 29, 2021

The missing icon is still an issue for me. I'm running KDE on Kubuntu 20.04 and built the latest 12.1 SuperCollider from source.
The icon is missing in the upper left corner of the IDE, and when using the task switcher.
Screenshot_20211129_131258

@telephon
Copy link
Member

the best is you reopen #3905

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants