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

[NEW ICON]: Tally Prime and Canva Icons #75

Merged
merged 12 commits into from
Jun 7, 2024
Merged

Conversation

josemateuss
Copy link
Contributor

@josemateuss josemateuss commented Jun 5, 2024

Add New Icon(s)

Icon Details

Icon ID Icon
tallyprime tallyprime
canva-dark canva-dark
canva-light canva-light

Notes

  • Tally Prime is an Indian based accounting software, it's too famous as same as SAP, as mentioned in [NEW ICON]: TALLY Prime  #74.
  • Canva is a graphic design platform that allows users to create social media graphics and other visual content.

Checklist

  • Added icon(s) to the appropriate directory
  • Updated documentation with the new icon(s)
  • Verified the icon(s) display correctly
  • If the icon has light and dark, verified the auto switch works correctly

Closing

closes #74, closes tandpfun/skill-icons#184, closes tandpfun/skill-icons#358, closes tandpfun/skill-icons#805

@josemateuss josemateuss changed the title [NEW ICON]: Tally Prime an Canva Icons [NEW ICON]: Tally Prime and Canva Icons Jun 5, 2024
@WorldOfDex
Copy link

WorldOfDex commented Jun 5, 2024

thanks man
, shall i expect black and white version?

Copy link

@WorldOfDex WorldOfDex left a comment

Choose a reason for hiding this comment

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

good

@josemateuss
Copy link
Contributor Author

thanks man
, shall i expect black and white version?

Hi @WorldOfDex, from what i've seen here, either we have the dark and light version or we have a single one, I don't think @LelouchFR will accept both the dark and light but the single one. What do you think @LelouchFR?

@LelouchFR
Copy link
Owner

I totally agree with @josemateuss, since it's only squares, it is better to just fill up the entire icon place instead of putting it in the middle. I would prefer to have it like right now, it makes a little bit more sense

@josemateuss
Copy link
Contributor Author

josemateuss commented Jun 5, 2024

I totally agree with @josemateuss, since it's only squares, it is better to just fill up the entire icon place instead of putting it in the middle. I would prefer to have it like right now, it makes a little bit more sense

Agreed. So, the PR is ready for review man, thanks.

@LelouchFR
Copy link
Owner

@josemateuss I'm going to look at that tomorrow, starting to get late here (and tomorrow I have exams)

@WorldOfDex
Copy link

thanks man
, shall i expect black and white version?

Hi @WorldOfDex, from what i've seen here, either we have the dark and light version or we have a single one, I don't think @LelouchFR will accept both the dark and light but the single one. What do you think @LelouchFR?

ok thanks

@josemateuss
Copy link
Contributor Author

@LelouchFR I added the pull request template so that we don't run out of templates until we understand how template selection works, it will help new contributors as a guideline too.

@josemateuss
Copy link
Contributor Author

@LelouchFR have you reviewed the PR?

@LelouchFR
Copy link
Owner

@LelouchFR have you reviewed the PR?

Yes, don't you see the comment about canva-auto.svg I made?

@josemateuss
Copy link
Contributor Author

Yes, don't you see the comment about canva-auto.svg I made?

No, I don't, where is it? Hahaha

@josemateuss
Copy link
Contributor Author

I don't know what do you have commented, but I remade the auto icon, please, take a look, thanks.

Copy link
Owner

@LelouchFR LelouchFR left a comment

Choose a reason for hiding this comment

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

please use this version for the canva-auto.svg file:

<svg width="256" height="256" viewBox="0 0 256 256" fill="none" xmlns="http://www.w3.org/2000/svg" id="canva">
<style>
#canva {
    rect {fill: #242938}

    @media (prefers-color-scheme: light) {
        rect {fill: #F4F2ED}
    }
}
</style>
<g clip-path="url(#clip0_1_15)">
<rect width="256" height="256" rx="60" fill="#242938"/>
<g clip-path="url(#clip1_1_15)">
<path d="M128 228C183.228 228 228 183.228 228 128C228 72.7715 183.228 28 128 28C72.7715 28 28 72.7715 28 128C28 183.228 72.7715 228 128 228Z"
fill="#7D2AE7"/>
<path d="M128 228C183.228 228 228 183.228 228 128C228 72.7715 183.228 28 128 28C72.7715 28 28 72.7715 28 128C28 183.228 72.7715 228 128 228Z"
fill="url(#paint0_radial_1_15)"/>
<path d="M128 228C183.228 228 228 183.228 228 128C228 72.7715 183.228 28 128 28C72.7715 28 28 72.7715 28 128C28 183.228 72.7715 228 128 228Z"
fill="url(#paint1_radial_1_15)"/>
<path d="M128 228C183.228 228 228 183.228 228 128C228 72.7715 183.228 28 128 28C72.7715 28 28 72.7715 28 128C28 183.228 72.7715 228 128 228Z"
fill="url(#paint2_radial_1_15)"/>
<path d="M171.173 148.514C170.347 148.514 169.621 149.211 168.865 150.733C160.331 168.039 145.589 180.284 128.531 180.284C108.807 180.284 96.5923 162.479 96.5923 137.881C96.5923 96.2143 119.808 72.1235 140.199 72.1235C149.727 72.1235 155.546 78.1114 155.546 87.6403C155.546 98.9497 149.121 104.938 149.121 108.926C149.121 110.717 150.235 111.801 152.443 111.801C161.316 111.801 171.729 101.605 171.729 87.2026C171.729 73.2375 159.575 62.9725 139.184 62.9725C105.485 62.9725 75.5353 94.2151 75.5353 137.443C75.5353 170.904 94.6429 193.016 124.125 193.016C155.417 193.016 173.511 161.882 173.511 151.777C173.511 149.539 172.366 148.514 171.173 148.514Z"
fill="white"/>
</g>
</g>
<defs>
<radialGradient id="paint0_radial_1_15" cx="0" cy="0" r="1" gradientUnits="userSpaceOnUse"
gradientTransform="translate(66.6325 205.264) rotate(-49.416) scale(154.683)">
<stop stop-color="#6420FF"/>
<stop offset="1" stop-color="#6420FF" stop-opacity="0"/>
</radialGradient>
<radialGradient id="paint1_radial_1_15" cx="0" cy="0" r="1" gradientUnits="userSpaceOnUse"
gradientTransform="translate(80.947 50.7364) rotate(54.703) scale(174.434)">
<stop stop-color="#00C4CC"/>
<stop offset="1" stop-color="#00C4CC" stop-opacity="0"/>
</radialGradient>
<radialGradient id="paint2_radial_1_15" cx="0" cy="0" r="1" gradientUnits="userSpaceOnUse"
gradientTransform="translate(66.6316 205.263) rotate(-45.1954) scale(152.811 70.2796)">
<stop stop-color="#6420FF"/>
<stop offset="1" stop-color="#6420FF" stop-opacity="0"/>
</radialGradient>
<clipPath id="clip0_1_15">
<rect width="256" height="256" fill="white"/>
</clipPath>
<clipPath id="clip1_1_15">
<rect width="200" height="200" fill="white" transform="translate(28 28)"/>
</clipPath>
</defs>
</svg>

assets/canva-auto.svg Outdated Show resolved Hide resolved
@josemateuss
Copy link
Contributor Author

@LelouchFR can you review? Sorry for the late, your comments weren't showing up here.

Copy link
Owner

@LelouchFR LelouchFR left a comment

Choose a reason for hiding this comment

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

You're 4 commits behind, can you get your branch up to date ? thank you ;)

@LelouchFR LelouchFR merged commit 6480169 into LelouchFR:main Jun 7, 2024
2 checks passed
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.

[NEW ICON] Canva [NEW ICON] Skill Name [NEW ICON]: TALLY Prime [NEW ICON] Canva
3 participants