K3 modules update, themes questions

Shad Laws <shad-xpYdmXCiSuZWk0Htik3J/[email protected]> Thu, 28 Mar 2013 23:41:37 +0100
Newsgroups gmane.comp.web.gallery.devel
Message-ID <CA+z51A4Fetsswt5TKCaTUe1L0bE59nyTbSEjQcjBUDA--BuStg@mail.gmail.com>
Hey everyone,

So, I *think* I've finished the main move/rename/search/replace task for
(almost) all of the modules!  I've also corrected the class names I found
in bootstrap.php (several instances), index.php (1 instance in a comment),
and the installer (0 instances).  The only exceptions are:
- forge - will be replaced with formo
- kohana23_compat - will be deleted (but needs pagination to go
somewhere...)
- unit_test - revise? replace?
- gallery_unit_test - probably modified to fit new unit_test

Also, in a few cases, I found some files for which I couldn't really find a
new home yet.  For now, I put them in an "orphans" directory of the
corresponding module.  These cases are:
- stuff that overloads classes that no longer exist (MY_Input in
gallery/orphans)
- stuff that Bharat is already working on and I didn't want to disturb
him (MY_ORM
and Database in gallery/orphans)
- stuff that addresses how K2 handles errors (error and gallery_error in
gallery/orphans)
- stuff that needs the forge -> formo conversion first (rest of files in
recaptcha/orphans and gallery/orphans)

Next stop: fixing up the themes.  Here's what the directory structure of
wind looks like right now:

themes/wind
|-- theme.info
|-- thumbnail.png
|-- css
|   |-- fix-ie.css
|   |-- screen-rtl.css
|   |-- screen.css
|   `-- themeroller
|       |-- ui.base.css
|       `-- images
|           `-- ...
|-- images
|   |-- avatar.jpg
|   |-- ico-album.png
|   |-- ico-denied-inactive.png
|   |-- ico-denied-passive.png
|   |-- ico-denied.png
|   |-- ico-error.png
|   |-- ico-help.png
|   |-- ico-info.png
|   |-- ico-lock.png
|   |-- ico-print.png
|   |-- ico-separator-rtl.gif
|   |-- ico-separator.gif
|   |-- ico-success-inactive.png
|   |-- ico-success-passive.png
|   |-- ico-success.png
|   |-- ico-view-comments.png
|   |-- ico-view-fullsize.png
|   |-- ico-view-slideshow-rtl.png
|   |-- ico-view-slideshow.png
|   |-- ico-warning.png
|   |-- loading-large.gif
|   |-- loading-small.gif
|   `-- select-photos-backg.png
|-- js
|   `-- ui.init.js
`-- views
    |-- album.html.php
    |-- block.html.php
    |-- dynamic.html.php
    |-- movie.html.php
    |-- no_sidebar.html.php
    |-- page.html.php
    |-- paginator.html.php
    |-- photo.html.php
    `-- sidebar.html.php

Here's my first draft of what I think it should look like:

themes/wind
|-- theme.info
|-- thumbnail.png
|-- media
|   |-- ico-album.png
|   |-- ico-error.png
|   |-- ico-help.png
|   |-- ico-info.png
|   |-- ico-print.png
|   |-- ico-separator-rtl.gif
|   |-- ico-separator.gif
|   |-- ico-view-comments.png
|   |-- ico-view-fullsize.png
|   |-- ico-view-slideshow-rtl.png
|   |-- ico-view-slideshow.png
|   |-- ico-warning.png
|   |-- loading-large.gif
|   |-- loading-small.gif
|   |-- select-photos-backg.png       [doesn't seem to be used anywhere -
delete it?]
|   |-- screen-old-ie.css             [I renamed this]
|   |-- screen-rtl.css
|   |-- screen.css
|   |-- init.js                       [I renamed this]
|   `-- common                        [not sure I love this name...]
|       |-- avatar.jpg                [used in modules, but not in theme]
|       |-- ico-denied-inactive.png   [used in permissions_form.html.php,
but not in theme]
|       |-- ico-denied-passive.png    [used in permissions_form.html.php,
but not in theme]
|       |-- ico-denied.png            [used in permissions_form.html.php,
but not in theme]
|       |-- ico-lock.png              [used in permissions_form.html.php,
but not in theme]
|       |-- ico-success-inactive.png  [used in permissions_form.html.php,
but not in theme]
|       |-- ico-success-passive.png   [used in permissions_form.html.php,
but not in theme]
|       `-- ico-success.png           [used in permissions_form.html.php
*and* in theme]
|-- vendor
|   `-- themeroller
|       |-- ui.base.css
|       `-- images
|           `-- ...
`-- views
    |-- album.html.php  [note: these are the only views that sit in the
root views directory]
    |-- block.html.php
    |-- dynamic.html.php
    |-- movie.html.php
    |-- no_sidebar.html.php
    |-- page.html.php
    |-- paginator.html.php
    |-- photo.html.php
    `-- sidebar.html.php

Thoughts?

Take care,
Shad

------------------------------------------------------------------------------
Own the Future-Intel(R) Level Up Game Demo Contest 2013
Rise to greatness in Intel's independent game demo contest. Compete 
for recognition, cash, and the chance to get your game on Steam. 
$5K grand prize plus 10 genre and skill prizes. Submit your demo 
by 6/6/13. http://altfarm.mediaplex.com/ad/ck/12124-176961-30367-2

__[ g a l l e r y - d e v e l ]_________________________

[ list info/archive --> http://gallery.sf.net/lists.php ]
[ gallery info/FAQ/download --> http://gallery.sf.net ]