8000 feat(icon): add tsdoc icon to tsdoc.json by Gajdascz · Pull Request #3569 · vscode-icons/vscode-icons · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

feat(icon): add tsdoc icon to tsdoc.json #3569

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

Merged
merged 4 commits into from
Mar 28, 2025

Conversation

Gajdascz
Copy link
Contributor
@Gajdascz Gajdascz commented Aug 21, 2024

Adds support to the tsdoc.json file.

Fixes #3566

Changes proposed:

  • Add
  • Fix

@robertohuertasm robertohuertasm added file icons needs icon rework The icon doesn't follow the guidelines labels Sep 8, 2024
@robertohuertasm robertohuertasm added this to the Backlog milestone Sep 8, 2024
@robertohuertasm
Copy link
Member

Hi @Gajdascz, thank you for your contribution. Unfortunately, the icon needs some rework before accepting this PR.

As you can see below, comparing both the original and the proposed one, it's clear that the icons are very different.

image
image

@Gajdascz
Copy link
Contributor Author
Gajdascz commented Sep 8, 2024

Hello @robertohuertasm,

Thank you for the feedback. I've adjusted the icon to (hopefully) match the proposal and standards. If anything is off or requires more work please let me know and I will gladly adjust accordingly.

committed_tsdoc

@robertohuertasm robertohuertasm removed the needs icon rework The icon doesn't follow the guidelines label Mar 28, 2025
@robertohuertasm robertohuertasm modified the milestones: Backlog, Next Mar 28, 2025
Copy link

Icons Background Test

file_type_tsdoc (808 bytes)

Preview

Copy link
Member
@robertohuertasm robertohuertasm left a comment

Choose a reason for hiding this comment

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

LGTM 🚀 Thanks for your contribution ❤️

@robertohuertasm robertohuertasm merged commit 42354e2 into vscode-icons:master Mar 28, 2025
19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Icon Request]: tsdoc
2 participants
0