fix(chat): correct file icon rendering in Files & Folders picker

[imbant]

File icons were not displaying in the chat context Files & Folders picker while folder icons worked correctly. This was due to using iconClass instead of iconClasses property.

The quick-input component has two icon rendering paths:

• iconClass: applies classes to icon container element
• iconClasses: applies classes to IconLabel component

File icon themes require the specific CSS context provided by IconLabel, so file icons must use iconClasses to render properly.

Changes:

• Add iconClasses property to IChatContextPickerPickItem interface
• Change FilesAndFoldersPickerPick to use iconClasses instead of iconClass
• File icons now render correctly using proper DOM structure

Fixes file icon display issue in chat context picker while maintaining backward compatibility with existing iconClass usage for simple codicons.

[imbant]

Just need to build vscode on my pc to verify it

[imbant]

Before:

After:

I think this is ready to review. This is my first PR to vscode so please let me know how to do better, thank you very much

[justschen]

thanks! lgtm.

[justschen]

looks like we approved this but we never merged it. we'll get this merged for next iteration. thanks @imbant !

[joaomoreno]

Ping @justschen

[bpasero]

This was NOT ⚠️ actually working at all without further changes. It actually broke the picker quite severely for a icon theme such as "Material".

With the changes applied, things look good:

Thanks @imbant for the right pointers.

[vs-code-engineering]

📬 CODENOTIFY

The following users are being notified based on files changed in this PR:

@TylerLeonhardt

Matched files:

• src/vs/platform/quickinput/common/quickInput.ts

[imbant]

This was NOT ⚠️ actually working at all without further changes. It actually broke the picker quite severely for a icon theme such as "Material".

With the changes applied, things look good:

Thanks @imbant for the right pointers.

Thanks for fixing it! Glad it helped. Would love to learn what additional changes were needed if you have time to share and what I can do for now as a newcomer!

[bpasero]

Yeah the change in e45528a