Add support for modules folder

[KingDarBoja]

References #650

Changes proposed:

• Add
• Delete
• Fix
• Prepare

This is a proposal for modules folder using the icon provided at #1022. Thoughts?

[codecov]

Codecov Report

Merging #1985 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master   #1985   +/-   ##
======================================
  Coverage     100%    100%           
======================================
  Files          78      78           
  Lines        5914    5914           
  Branches      137     137           
======================================
  Hits         5914    5914

Impacted Files	Coverage Δ
src/iconsManifest/supportedFolders.ts	100% <ø> (ø)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 03035d8...53c4b77. Read the comment docs.

[JimiC]

I'm not 100% OK with the icon or the colors (something is bothering on it but I can't pinpoint it yet) but if the team agrees I'm OK with it.

[KingDarBoja]

I'm not 100% OK with the icon or the colors (something is bothering on it but I can't pinpoint it yet) but if the team agrees I'm OK with it.

Feel free to change the colors. Regarding the icon, I think it was a proper choice because couldn't think anything for modules besides multiple boxes linked together... unless it is better for shared folder icon?

[robertohuertasm]

Personally, I don't dislike the icon. What I think is that the color combination is not the kind of combinations that we usually use. Normally we grab the color of the icon as a base for the color of the folder, but in this case I guess you chose a complementary.

I would make it more monochromatic and choose whether purple or yellow as the base color for everything.

[KingDarBoja]

@robertohuertasm I am working on it right now using the steps provided by JimiC on the other PR.

[robertohuertasm]

LGTM but let's wait for @JimiC's input and see if we have consensus.

[JimiC]

Looking at this in detail I believe the best course is to bind modules with the existing components folder icon. After all modules and components are similar in concept.

[robertohuertasm]

Certainly, the components icon could also work for modules. But since we already have this cool icon I don't see a reason not to use it. Components folder has a specific meaning in the UI world and modules seems to me more of a backend construct, so I wouldn't mind having two separated icons.

[JimiC]

OK. I'm going to fix the contrast a bit. Don't merge yet.

[JimiC]

I decided to leave the contrast as is. Will iterate on them if any issue arises.