adam/vscode-front-matter
1
0 Fork 0
mirror of https://github.com/estruyf/vscode-front-matter.git synced 2026-11-04 01:22:21 +01:00
Code Issues 77 Packages Projects Releases Wiki Activity

Feedback: Zero value shouldn't be treated as empty field #667

New Issue
Closed
opened 2026-10-05 02:59:32 +02:00 by adam · 6 comments
adam commented 2026-10-05 02:59:32 +02:00
Owner
Copy link
Copy source

image

When I set 0 to number field, it always treats as empty field. And if that field is required, it will reporting error. I don't think it is good design because in many cases, 0 has meaning. I suggest we should treat 0 like other digits and null/undefined as an empty field.

Info:

  • framework: Astro 4.15.1 
     experimental: {
    contentLayer: true,
    contentIntellisense: true,
 },
  • os: Linux
  • vscode: 1.92.2
![image](https://github.com/user-attachments/assets/b096201d-ae6d-4859-9cce-11f27674cefd) When I set 0 to number field, it always treats as empty field. And if that field is required, it will reporting error. I don't think it is good design because in many cases, `0` has meaning. I suggest we should treat `0` like other digits and `null`/`undefined` as an empty field. Info: - framework: Astro `4.15.1` ``` experimental: { contentLayer: true, contentIntellisense: true, }, ``` - os: Linux - vscode: `1.92.2` <!-- github-sync-id:2500193574 -->
adam closed this issue 2026-10-05 02:59:32 +02:00
adam commented 2026-10-05 02:59:33 +02:00
Author
Owner
Copy link
Copy source

Thanks @ngdangtu-vn for opening this issue. I have implemented a fix for this in the latest beta. Feel free to give it a try and let me know how it works.

Thanks @ngdangtu-vn for opening this issue. I have implemented a fix for this in the latest beta. Feel free to give it a try and let me know how it works. <!-- github-comment-id:2331350718 -->
adam commented 2026-10-05 02:59:34 +02:00
Author
Owner
Copy link
Copy source

Cool, how do I test it?

Cool, how do I test it? <!-- github-comment-id:2331633523 -->
adam commented 2026-10-05 02:59:35 +02:00
Author
Owner
Copy link
Copy source

You will have to uninstall or disable the stable version, and once that is done, you can install the beta: https://marketplace.visualstudio.com/items?itemName=eliostruyf.vscode-front-matter-beta

You will have to uninstall or disable the stable version, and once that is done, you can install the beta: https://marketplace.visualstudio.com/items?itemName=eliostruyf.vscode-front-matter-beta <!-- github-comment-id:2331669517 -->
adam commented 2026-10-05 02:59:35 +02:00
Author
Owner
Copy link
Copy source

Just tested, it works beautiful. Thanks!

Just tested, it works beautiful. Thanks! <!-- github-comment-id:2331721356 -->
adam commented 2026-10-05 02:59:36 +02:00
Author
Owner
Copy link
Copy source

Thanks for verifying 🙏

Thanks for verifying 🙏 <!-- github-comment-id:2331807810 -->
adam commented 2026-10-05 02:59:37 +02:00
Author
Owner
Copy link
Copy source

This functionality is now part of the v10.4.0 main release.

This functionality is now part of the `v10.4.0` main release. <!-- github-comment-id:2373238756 -->
Sign in to join this conversation.
No Branch/Tag Specified
Labels
No items
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
adam (Adam Wojcieszonek)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: adam/vscode-front-matter#667