Skip to content

added 7 new icons#1156

Merged
flameshikari merged 10 commits intoDelta-Icons:masterfrom
ryeookie:master
Apr 19, 2025
Merged

added 7 new icons#1156
flameshikari merged 10 commits intoDelta-Icons:masterfrom
ryeookie:master

Conversation

@ryeookie
Copy link
Contributor

added icons and componentinfo for the ff. apps:

  • beep
  • go rewards
  • komo
  • landbank
  • virtual pag-ibig
  • rcbc
  • smtown

@ryeookie ryeookie closed this Apr 19, 2025
@ryeookie ryeookie reopened this Apr 19, 2025
@ryeookie ryeookie closed this Apr 19, 2025
@ryeookie ryeookie reopened this Apr 19, 2025
@ryeookie
Copy link
Contributor Author

Sorry, I'm not sure what happened but it seems that my previous commits were also pulled when I created this new pull request. I can't figure out how to remove those or post a new pull request without the other commits being pulled as well.

@flameshikari
Copy link
Collaborator

flameshikari commented Apr 19, 2025

Icons look good, but PNGs are missing canvas, they should be 192x192px.

image

About previous commits: we're squashing commits in pull requests (converting multiple commits to a single one) when merging, so old commits in the fork can be considered as new. You just need pull changes before commiting new stuff. You can do it by syncing your fork with the upstream (Sync fork → Update branch, see the screenshot).

image

@flameshikari flameshikari added this to the 2.3.0 milestone Apr 19, 2025
@flameshikari flameshikari merged commit 40ba031 into Delta-Icons:master Apr 19, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

2 participants