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 CVAT Logo #200

Merged
merged 3 commits into from
Mar 1, 2024
Merged

Add CVAT Logo #200

merged 3 commits into from
Mar 1, 2024

Conversation

tisonkun
Copy link
Member

@tisonkun tisonkun commented Mar 1, 2024

Let's wait for CVAT's confirm.

image

Signed-off-by: tison <[email protected]>
@tisonkun tisonkun marked this pull request as draft March 1, 2024 04:08
@jihuayu
Copy link
Member

jihuayu commented Mar 1, 2024

How are our icons ordered?
Should they be ordered in the order of addition time or alphabetical order by company name?

@bsekachev
Copy link

Hello, thanks for contacting us.
I believe simple logo would look nicer. Attached the logo in .svg/.png here for your convenience

cvat-logo

image

Signed-off-by: tison <[email protected]>
@tisonkun
Copy link
Member Author

tisonkun commented Mar 1, 2024

@bsekachev Thank you! I test both and it seems the PNG version is good to fill the container:

image

I'm moving forward this patch with this new logo now.

@tisonkun tisonkun marked this pull request as ready for review March 1, 2024 08:13
@tisonkun
Copy link
Member Author

tisonkun commented Mar 1, 2024

@jihuayu Good point! We have a title field for metadata now. Let's try to sort logos by this field alphabetically

@tisonkun tisonkun merged commit 4e049e1 into main Mar 1, 2024
1 check passed
@tisonkun tisonkun deleted the cvat branch March 1, 2024 08:23
@tisonkun
Copy link
Member Author

tisonkun commented Mar 1, 2024

Twitter link https://x.com/tison1096/status/1763483779073790296?s=46&t=rCbLrJSBayvO_CKE6XmYKQ if you'd share the news or comment more on the background and experience.

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.

4 participants