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

🎨✨:add creative commons icons #1141

Merged
merged 16 commits into from
Feb 19, 2024
Merged

🎨✨:add creative commons icons #1141

merged 16 commits into from
Feb 19, 2024

Conversation

DerekNonGeneric
Copy link
Member

@DerekNonGeneric DerekNonGeneric commented Feb 18, 2024

The title says it all. These need to be made more spiffy (look at source), so opening pr as a draft.

Copy link

netlify bot commented Feb 18, 2024

👷 Deploy Preview for open-inf-is processing.

Name Link
🔨 Latest commit 132e5e6
🔍 Latest deploy log https://app.netlify.com/sites/open-inf-is/deploys/65d30870d3fc80000870751d

@DerekNonGeneric
Copy link
Member Author

DerekNonGeneric commented Feb 18, 2024

Looking forward to adding them to the bottoms of pages like #1137

Actually, happens in a template, see https:/OpenINF/openinf.github.io/pull/1123/files#r1483741286

@DerekNonGeneric
Copy link
Member Author

you can see how it would kind of look like here:

@DerekNonGeneric
Copy link
Member Author

DerekNonGeneric commented Feb 18, 2024

alright, moving right along, let's run these thru an optimizer:

@DerekNonGeneric
Copy link
Member Author

@DerekNonGeneric
Copy link
Member Author

to actuality! i love these colour names tbh

@DerekNonGeneric DerekNonGeneric added the 🚅 Status: Fast Track PRs that do not need to wait for 48 hours to land label Feb 19, 2024
@DerekNonGeneric
Copy link
Member Author

DerekNonGeneric commented Feb 19, 2024

re: #1119

i think these are for light color scheme bg


maybe they should be even more vibey tho

@DerekNonGeneric
Copy link
Member Author

DerekNonGeneric commented Feb 19, 2024

i think am just going to add the a11y and star tops and merge

and can't forget the license

@DerekNonGeneric

This comment was marked as resolved.

@DerekNonGeneric DerekNonGeneric marked this pull request as ready for review February 19, 2024 04:33
@DerekNonGeneric
Copy link
Member Author

let me see about a quick & concise desc

@DerekNonGeneric

This comment was marked as outdated.

@DerekNonGeneric
Copy link
Member Author

DerekNonGeneric commented Feb 19, 2024

the current official file naming scheme of the icons:

some are even pink lol:

@DerekNonGeneric

This comment was marked as resolved.

@DerekNonGeneric

This comment was marked as resolved.

@DerekNonGeneric
Copy link
Member Author

yoo this new color is lookin' good!

@DerekNonGeneric
Copy link
Member Author

ok, just going to quickly rename them and add zero icon then we should be good to merge

@DerekNonGeneric
Copy link
Member Author

and add zero icon

actually, we do not use CC0 anywhere - we use 0BSD, so going to remove that

@DerekNonGeneric DerekNonGeneric added the 🗳️ Status: Author Ready PRs that are considered ready by their author label Feb 19, 2024
@DerekNonGeneric
Copy link
Member Author

i think is good for now

@OpenINFbot OpenINFbot merged commit 246a79c into OpenINF:live Feb 19, 2024
16 checks passed
@DerekNonGeneric DerekNonGeneric deleted the feat/add-cc-svg-icons-n-licenense branch February 19, 2024 08:17
@OpenINFbot OpenINFbot removed 🗳️ Status: Author Ready PRs that are considered ready by their author 🚅 Status: Fast Track PRs that do not need to wait for 48 hours to land 📖 Category: Documentation Improvements or additions to documentation 🚸 Category: User Experience (UX) undefined 🚧 Status: Work In Progress (WIP) undefined 🌗 P2: Medium undefined ✨ Type: Feature undefined 🔔 Type: New undefined labels Feb 20, 2024
DerekNonGeneric added a commit that referenced this pull request Mar 3, 2024
PR URL: #1141
Reviewed-by: OpenINF Bot <[email protected]>
Signed-off-by: Derek Lewis <[email protected]>
Co-authored-by: Restyled.io <[email protected]>
Acked-by: Princess Irulen <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants