-
Notifications
You must be signed in to change notification settings - Fork 6
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
Button icon licensing issues #883
Comments
Also including @brettfiedler, since if we have things using font-awesome 5 icons, they can't be included in scenery-stack. |
This is also blocking any THREE.js integration (mobius now uses a FA-5 icon with phetsims/mobius@ffad97db). |
Unblocked by adding FA-4 warning sign back. |
It seems preferable to avoid licensing encumbrances for scenery-stack, so I agree we should avoid FA-5 there. Font Awesome 4 is licensed as SIL OFL 1.1 https://fontawesome.com/v4/license/, which does not require attribution (though outside of the license text, the author said attribution is not required but much appreciated. Where the icons look similar, it seems safe to go back to FA-4. We kept a few FA-4 things around in sherpa anyways (checkEmptySolidShape, checkSquareOSolidShape, scissorsShape). We can also consider open source MIT licensed icons libraries such as https://heroicons.com/ which look nice. |
Unassigning myself. PixelZoom does not provide legal advice or consulting on licensing. |
Sounds like we'll potentially replace some icons. Noting here I'd like to make a lint rule to ensure we don't have them re-occur in our MIT-licensed code. |
I'm redrawing these icons, and here is my list for tracking (in order of priority):
Optional (in scenery-phet, commonly used in sims, but could be omitted from scenerystack):
|
@jonathanolson I have all of the checked items above ready. There are two I'm not sure about though. Can you help me get the svgs for warningSignShape and scissorsShape? |
Here are the new icons! There are a few extras because I originally thought checkboxes were built differently, so I included them just in case it's helpful. I also made a different globe icon that I thought might look nicer than the solid one. |
I also uploaded the ai file to the scenery stack folder in case we ever want to modify the icons (the ai file includes more editable versions than the final svgs). |
…replacing SceneryStack usages, see phetsims/scenery-phet#883
…replacing SceneryStack usages, see phetsims/scenery-phet#883
…replacing SceneryStack usages, see phetsims/scenery-phet#883
…replacing SceneryStack usages, see phetsims/scenery-phet#883
…replacing SceneryStack usages, see phetsims/scenery-phet#883
…replacing SceneryStack usages, see #883
All of the shapes should be integrated! Can you test out the shapes to see how they are looking? function-builder, molecule-shapes (with ?webgl=false), gravity-and-orbits, and the scenery-phet demo (first screen, buttons) should have the majority of the shapes. |
We're advertising scenery-phet as MIT licensed, which means we can't use font-awesome 5 icons (which are CC-By licensed, explicitly noted at https://github.com/phetsims/sherpa/blob/main/licenses/fontawesome-free-5.15.2-desktop.txt#L8).
It looks like we have licensing issues with:
It looks like a lot of these conversions happened as part of phetsims/sherpa#83. I'm including @samreid.
So we can either:
Thoughts?
The text was updated successfully, but these errors were encountered: