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 feature: vs code intellisense to devicon.json #1487

Merged
merged 6 commits into from
Dec 10, 2022

Conversation

lunatic-fox
Copy link
Contributor

Double check these details before you open a PR

  • PR does not match another non-stale PR currently opened

Features

VS Code Intellisense to devicon.json provides these features:

  • A snippet to create a new input object by typing new
  • Validates
    • name structure
    • color structure
    • version.svg[], version.font[], aliases.?base and aliases.?alias completion
  • Provides some description for the properties based on the documentation

This PR closes #1481

Notes

VS Code Intellisense provides these features:
- A snippet to create a new input object by typing `new`
- Validates
    - `name` structure
    - `color` structure
    - `version.svg[]`, `version.font[]`, `aliases.?base` and `aliases.?alias` completion
- Provides some description for the properties based on the documentation
@lunatic-fox lunatic-fox changed the title Add VS Code Intellisense to devicon.json add feature: VS Code Intellisense to devicon.json Oct 30, 2022
@lunatic-fox lunatic-fox changed the title add feature: VS Code Intellisense to devicon.json add feature: vs code intellisense to devicon.json Oct 30, 2022
@Snailedlt Snailedlt added enhancement devops Use this label for devops related enhancements labels Oct 30, 2022
Copy link
Collaborator

@Snailedlt Snailedlt left a comment

Choose a reason for hiding this comment

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

Looking great!
One of the features I'm most looking forwards too, since this will boost productivity so much!

As always, there's a few suggestions I'd like you to look at before we're ready for merging this. Great general structure though, definitely good work! 💯

.vscode/devicon-schema.json Show resolved Hide resolved
.vscode/devicon-schema.json Outdated Show resolved Hide resolved
.vscode/devicon.code-snippets Show resolved Hide resolved
.vscode/devicon.code-snippets Outdated Show resolved Hide resolved
.vscode/devicon-schema.json Show resolved Hide resolved
.vscode/devicon-schema.json Show resolved Hide resolved
lunatic-fox and others added 4 commits October 31, 2022 20:32
Co-authored-by: Jørgen Kalsnes Hagen <43886029+Snailedlt@users.noreply.github.com>
Great!

Co-authored-by: Jørgen Kalsnes Hagen <43886029+Snailedlt@users.noreply.github.com>
Agreed! It's more detailed now!

Co-authored-by: Jørgen Kalsnes Hagen <43886029+Snailedlt@users.noreply.github.com>
Great!

Co-authored-by: Jørgen Kalsnes Hagen <43886029+Snailedlt@users.noreply.github.com>
Copy link
Collaborator

@Snailedlt Snailedlt left a comment

Choose a reason for hiding this comment

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

Looking good! ✔️
Can't wait to get this into master, well done with the changes :)

@lunatic-fox
Copy link
Contributor Author

@Panquesito7, please review this request. 🙂
Here's another reason: #1514

@Snailedlt Snailedlt merged commit 678f0cb into devicons:develop Dec 10, 2022
@lunatic-fox lunatic-fox deleted the vscode_intellisense branch June 1, 2023 12:29
@Snailedlt Snailedlt mentioned this pull request Feb 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
devops Use this label for devops related enhancements enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants