Cleanup public directory

zikula / core

Zikula Core Framework

GNU Lesser General Public License v3.0

237 stars 69 forks source link

Cleanup public directory #4257

Closed Guite closed 4 years ago

Guite commented 4 years ago

The subfolders in the public/ directory should be grouped.

Proposal:

- bundles/
- components/
  - bootstrap/
  - bootswatch/
  - font-awesome/
  - fontawesome-iconpicker/
  - jquery/
  - jquery-ui/
  - jstree/
  - magnific-popup/
  - mmenu/
  - require/
- js/
- media/
- modules/
- overrides/
- themes/
- uploads/

craigh commented 4 years ago

why?

Guite commented 4 years ago

Because it looks sloppy.

Example from shopware:

- bundles/
- css/
- fonts/
- js/
- media/
- recovery/
- sitemap/
- theme/
- thumbnail/

Guite commented 4 years ago

I don't see that as critical. If you think it's unnecessary, we can drop it. But if we're gonna clean it up, we do it now, not after 3.0.0.

craigh commented 4 years ago

adding an additional layer of organization means altering asset() and other functioning items. I don't see a reason to do this.