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

Chore: Update folder-favorite icon to ayon icon #5718

Merged
merged 5 commits into from
Dec 22, 2023

Conversation

BigRoy
Copy link
Collaborator

@BigRoy BigRoy commented Oct 7, 2023

Changelog Description

Updates old "Pype-2.0-era" (from ancient greece times) to AYON logo equivalent.

I believe it's only used in Nuke.

Additional info

Other icons seemed to have no usages across the codebase - so removed those and renamed this one. They are separate commits so feel free to cherry pick.

Due to the rename to folder-favorite.png the Github changelog shows different sizes, but the new icon is basically the exact same as folder-favorite3.png but with the P. icon in the folder replaced with AYON icon.

OLD

folder-favorite3

NEW

folder-favorite

Testing notes:

  1. Launch nuke
  2. Open a browser that should show the "favorite folders"
  3. It should now show the new icon

@BigRoy BigRoy added host: Nuke type: refactor Structural changes not affecting functionality community contribution labels Oct 7, 2023
@ynbot ynbot added the size/XS Denotes a PR changes 0-99 lines, ignoring general files label Oct 7, 2023
@BigRoy BigRoy requested a review from mkolar October 24, 2023 23:56
@BigRoy
Copy link
Collaborator Author

BigRoy commented Dec 21, 2023

@mkolar is this still relevant?

Copy link
Member

@mkolar mkolar left a comment

Choose a reason for hiding this comment

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

Not great, not terrible, Not really important. happy to merge :D

@mkolar mkolar merged commit e9b0dd8 into ynput:develop Dec 22, 2023
3 checks passed
@ynbot ynbot added this to the next-patch milestone Dec 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community contribution host: Nuke size/XS Denotes a PR changes 0-99 lines, ignoring general files type: refactor Structural changes not affecting functionality
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

None yet

3 participants