Changeset 357

Timestamp:

10/25/07 23:24:26 (2 years ago)

Author:

eevee

Message:

Added buttons to collapse the Pokemon page move table to just one version's column. (#159)

Location:

veekun/trunk

Files:

• root/script/dexutils.js (added)
• templates/dex/common.tt (modified) (1 diff)
• templates/dex/page/pokemon.tt (modified) (5 diffs)
• templates/style/dex.tt (modified) (1 diff)
• templates/wrapper.tt (modified) (3 diffs)

veekun/trunk/templates/dex/common.tt

@@ -6 +6 @@
 [% move_header = BLOCK %]
     <th class="name"> Move </th> <th class="type"> Type </th> <th class="type"> Dmg </th> <th class="pp"> PP </th>
-    <th class="power"> Power </th> <th class="acc"> Acc. </th> <th class="priority"> Pri. </th> <th class="blurb"> Description </th>
+    <th class="power"> Power </th> <th class="acc" title="RBY accuracy is about 0.4% lower"> Acc.* </th> <th class="priority"> Pri. </th> <th class="blurb"> Description </th>
 [% END %]
 

veekun/trunk/templates/dex/page/pokemon.tt

@@ +1 @@
+[% extra_js = 'dexutils.js' %]
 [% PROCESS 'dex/common.tt' %]
 
@@ -424 +425 @@
 <p> <a href="[% c.uri_for('moves/search', { pokemon => this.name }) %]"><img src="/images/see-also.png" alt="See also:"/> This list in the move search</a> </p>
 <p> <a href="[% c.uri_for('moves/search', { pokemon => this.name, view => 'contest' }) %]"><img src="/images/see-also.png" alt="See also:"/> Contest list</a> </p>
-<p> The TM table may be difficult to read depending on how you want to use it.  If you want to see just the TMs for one version, you may want to try the <a href="[% dex_uri('moves', 'search', { pokemon => this.name, method => 'machine' }) %]">move search</a>. </p>
-<p> Note: accuracies are for G/S/C and RuSa.  Keep in mind that R/B/Y accuracies are about 0.4% (1 in 255) lower than those shown. </p>
 [% IF preferred_damage %]
 <p> [% this.name %]'s [% preferred_damage == 'special' ? 'Special ' : '' %]Attack is higher, so it will inflict more damage with [% preferred_damage %] (<img src="/dex-images/gameui/[% preferred_damage %].png" alt=""/>) moves. </p>
@@ -433 +432 @@
  and [% type_name(this.type2) %]
 [% END %] moves. </p>
-<table class="dex-table" cellspacing="0">
+
+[% UNLESS header_icons.size == 1 %]
+<hr/>
+<p> NOT DONE: (todo only show this for js clients; don't use javascript:) </p>
+<p>
+    Only show moves from:
+[%     ver_idx = 0 %]
+[%     FOREACH ver IN header_icons %]
+[%         ver_idx = ver_idx + 1 %]
+    <a href="javascript:reduce_move_table([% ver_idx + 1 %], [% header_icons.size + 1 %]);">[% Icons.$ver %]</a>
+[%     END %]
+    <a href="javascript:reset_move_table();fix_move_table_highlighting();">all</a>
+</p>
+[% END %]
+
+<table class="dex-table" cellspacing="0" id="js-movetable">
 <!-- XXX: this is total crap and could stand to get one hell of a refactor.  again.  maybe for 5th gen. -->
 <tr class="heading">
@@ -486 +500 @@
 <tr class="color[% color %]">
  <td></td>
-[%         IF generation == 0 %] <td> </td>[% IF move_columns_inv.y %] <td> </td>[% END %][% END %]
+[%         IF generation == 0 %] <td class="level"> </td>[% IF move_columns_inv.y %] <td class="level"> </td>[% END %][% END %]
 [%         IF generation <= 1 %]
-[%             IF move_columns_inv.c %] <td> </td>[% END %]
- <td> [% Icons.c IF move.versions.c %] </td>
+[%             IF move_columns_inv.c %] <td class="level"> </td>[% END %]
+ <td class="level"> [% Icons.c IF move.versions.c %] </td>
 [%         END %]
 [%         IF generation <= 2 %]
 [%             IF move_columns_inv.frlg %]
- <td> [% Icons.e IF move.versions.e %] </td> <td> [% Icons.frlg IF move.versions.frlg %] </td>
+ <td class="level"> [% Icons.e IF move.versions.e %] </td> <td class="level"> [% Icons.frlg IF move.versions.frlg %] </td>
 [%             ELSE %]
- <td> [% Icons.e IF move.versions.e; Icons.frlg IF move.versions.frlg %] </td>
+ <td class="level"> [% Icons.e IF move.versions.e; Icons.frlg IF move.versions.frlg %] </td>
 [%             END %]
 [%         END %]
 [%         IF generation <= 3 %]
- <td> [% Icons.dp IF move.versions.dp %] </td>
+ <td class="level"> [% Icons.dp IF move.versions.dp %] </td>
 [%         END %]
 [%         move_cells(moveid) %]
@@ -512 +526 @@
 [%     FOREACH move IN moves.other %]
 [%         moveid = move.moveid %]
-<tr class="color[% color %]">
+<tr class="color[% color %] js-versionless">
  <td></td>
  <td colspan="[% move_columns.size %]" class="level"> [% move.method %] </td>

veekun/trunk/templates/style/dex.tt

@@ -163 +163 @@
     ul.item-list li a { display: block; }
   ul.item-list:after { content: 'v'; visibility: hidden; height: 1px; display: block; clear: left; margin-top: -1px; }
+
+.js-move-table-reduced { display: none; }

veekun/trunk/templates/wrapper.tt

@@ -7 +7 @@
 [% DEFAULT extra_headers = [] %]
 [% DEFAULT extra_css = [] %]
+[% DEFAULT extra_js = [] %]
 [% extra_css.push(template.extra_css) IF template.extra_css %]
 <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
@@ -14 +15 @@
 <link rel="shortcut icon" type="image/png" href="/images/favicon.png"/>
 <link rel="stylesheet" type="text/css" href="/style/global"/>
-[% IF extra_css %]
-[%     FOREACH css IN extra_css %]
+[% FOREACH css IN extra_css %]
 <link rel="stylesheet" type="text/css" href="/style/[% css %]"/>
-[%     END %]
 [% END %]
 [% extra_headers.join("\n") %]
@@ -31 +30 @@
 <script type="text/javascript" src="/script/jquery.js"></script>
 <script type="text/javascript" src="/script/dexsuggest.js"></script>
+[% FOREACH js IN extra_js %]
+<script type="text/javascript" src="/script/[% js %]"></script>
+[% END %]
 </head>
 <body>