476
edits
AniDB talk:Page layout: Difference between revisions
Comments
Epoximator (talk | contribs) (katinka) |
(Comments) |
||
| Line 1: | Line 1: | ||
== Content == | == Content == | ||
* 'g_section *_all' have been added to all pages. was this wanted? rename? g_content @ dev | * 'g_section *_all' have been added to all pages. was this wanted? rename? g_content @ dev | ||
::No, it's only wanted for pages with no internal sections (which is quite a few). Stuff like the calendar quite correctly has a g_section div per logical section (top and bottom nav and each anime listed). | |||
* should g_section be the only allowed element (first level childs) inside #layout-content (except h1)? | * should g_section be the only allowed element (first level childs) inside #layout-content (except h1)? | ||
::Pretty much. There can be multiples though, and it's not a hard rule (forms etc could usefully live outside and around sections). | |||
** general vertical margins. g_actionlist/etc. is subs of g_section? | ** general vertical margins. g_actionlist/etc. is subs of g_section? | ||
:::Vertical margins are a function of layout and should not be presented in the markup at all. | |||
* should g_navlist and g_numonpage be in one (parent) div? | * should g_navlist and g_numonpage be in one (parent) div? | ||
::Not just because. | |||
* should g_jumplist be inside the table div? | * should g_jumplist be inside the table div? | ||
::What is a 'table div'? They should be in the same section where appropriate. | |||
=== Text visibility === | === Text visibility === | ||
| Line 12: | Line 17: | ||
* v_pos (green) | * v_pos (green) | ||
* v_neg (red) | * v_neg (red) | ||
::These tread on the toes of both <code><nowiki><font></nowiki></code> level decoration and <code><nowiki><em></nowiki></code> level meaning. Classes are about what something is, not how it should be presented. Marking pos/neg is a different issue from 'text', and the current solution is not ideal. | |||
=== Text content === | === Text content === | ||
| Line 24: | Line 30: | ||
* c_icons (mylist state) | * c_icons (mylist state) | ||
* c_list (links, resources) | * c_list (links, resources) | ||
::The c_ prefix names were for collation purposes only - marked in table headers. There seems no reason for filling the namespace just because. | |||
=== Text alignment === | === Text alignment === | ||
| Line 36: | Line 43: | ||
* p_ne (northeast) | * p_ne (northeast) | ||
* ... | * ... | ||
::Alignment and text positioning are functions of layout, not markup. Using anything like this is just as stupid as doing <code><nowiki><span class="bold">hi mum</span></nowiki></code> etc. | |||
=== Table types === | === Table types === | ||
| Line 42: | Line 50: | ||
* dummy (table in title td in ep table, ...) | * dummy (table in title td in ep table, ...) | ||
* container (not inner, for layout only, f.ex. vote section @ anime page) | * container (not inner, for layout only, f.ex. vote section @ anime page) | ||
::Don't want is right. All these kinds of tables just want eradicating. | |||
=== Elements === | === Elements === | ||
| Line 69: | Line 78: | ||
|- | |- | ||
|} | |} | ||
::These need cleaning up. | |||
=== File states === | === File states === | ||
| Line 76: | Line 86: | ||
* g_fs_generic | * g_fs_generic | ||
* g_fs_lame | * g_fs_lame | ||
::These don't need to be in the global namespace. | |||
=== Messages === | === Messages === | ||
| Line 85: | Line 96: | ||
Could use g_box m_info / g_msg v_high / ...? | Could use g_box m_info / g_msg v_high / ...? | ||
unless g_*box is possible | unless g_*box is possible | ||
::Error handling needs redoing, no point fiddling till then. | |||
=== Other === | === Other === | ||
* td/tr/li.pause | * td/tr/li.pause | ||
** pause in lists, f.ex. between normal eps and special eps | ** pause in lists, f.ex. between normal eps and special eps | ||
:::This is a function of layout, not markup. No empty rows should exist for layout purposes. | |||
* td.fill, width/height 100% | * td.fill, width/height 100% | ||
::This is a function of layout, not markup. | |||
* .g_even, .g_odd | * .g_even, .g_odd | ||
::With IE7, I think _even is now redundant. | |||
* .g_description | * .g_description | ||
* .g_info | * .g_info | ||
* .g_howto | * .g_howto | ||
::These three probably want better and clearer definition. | |||
* .field, .value (in definition lists) | * .field, .value (in definition lists) | ||
::These seem redundant. | |||
* .nowrap | * .nowrap | ||
::This is a function of layout, not markup. | |||
* tr.mod (mod message) | * tr.mod (mod message) | ||
::This isn't a global meaning. | |||
* div.g_image | * div.g_image | ||
::Is this actually needed? | |||
* div.g_end (used to mark the end of content, used for clear: right, maybe not needed) | * div.g_end (used to mark the end of content, used for clear: right, maybe not needed) | ||
* .state (creq state) | * .state (creq state) | ||
::This isn't a global meaning. | |||
* li.here (inside g_navlist, currently selected) | * li.here (inside g_navlist, currently selected) | ||
::This isn't a global meaning. | |||