Add new file extension suffixes to naming conventions guide

[denniskigen]

This update adds information about three file type suffixes:

• *.modal.tsx for modals
• *.extension.tsx for extensions
• *.workspace.tsx for workspace forms

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
o3-docs	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jun 24, 2024 9:41pm

[NethmiRodrigo]

LGTM, thanks @denniskigen!

[brandones]

Why *.modal.component.tsx and not *.modal.tsx ?

[denniskigen]

Why *.modal.component.tsx and not *.modal.tsx ?

I could only find one occurrence of modal.tsx in Core, for example. It looks like we're not super consistent about that one pattern, or that the other custom extensions are more prevalent in our codebase. Either way, happy to change it to modal.tsx, but that'll require refactoring various files to match the new convention.

[brandones]

Yeah let's do modal.tsx and refactor :)

I think it would be awesome to have a consistent norm that each top-level element in the routes file has its own file extension, [whateveritis].tsx. And then component.tsx is anything that isn't loaded via the routes file.

[ibacher]

I love @brandones suggestion!

[denniskigen]

Refactor is done everywhere, I think. I'm proceeding with merging this in.