-
Notifications
You must be signed in to change notification settings - Fork 446
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
Add a 3rd definable icon size and fix precompiled header paths #2097
base: master
Are you sure you want to change the base?
Conversation
Olive6841
commented
Jan 10, 2025
- Changed #include <stdafx.h> to #include "stdafx.h"
- Add a 3rd definable "medium" icon size which is 24x24px by default on 96DPI
Adds a 3rd custom "Medium" icon size which is 24x24 by default, but can be adjusted via the settings GUI
I am guessing that at one time, an update to MSVC (not sure exactly) removed stdafx from being a standard header therefore requiring <stdafx.h> to be replaced with "stdafx.h"
There is certainly no need for this change. What problem did you encounter? |
Can you expand a bit more about the motivation for adding this? |
It would be able to be used in the .skin file |
It more for consistency sake rather than issues compiling, openshell currently has about half <stdafx.h> and half "stdafx.h" so i just moved them all to the second one |