Changeset 28744


Ignore:
Timestamp:
Jun 19, 2014, 10:39:12 PM (10 years ago)
Author:
rvelices
Message:

feature 3094: redesign of search form

  • fix comments page filter form (css changes affected also that .filter form - simplified also css rules)
  • better query for author list (using id for visible_images instead of image_id field)
  • don't show author list if there is only one author
Location:
trunk
Files:
4 edited

Legend:

Unmodified
Added
Removed
  • trunk/search.php

    r28710 r28744  
    222222      'forbidden_categories' => 'category_id',
    223223      'visible_categories' => 'category_id',
    224       'visible_images' => 'ic.image_id'
     224      'visible_images' => 'id'
    225225      ),
    226226    ' WHERE '
  • trunk/themes/default/template/comments.tpl

    r18750 r28744  
    1515    <legend>{'Filter'|@translate}</legend>
    1616
    17     <label>{'Keyword'|@translate}<input type="text" name="keyword" value="{$F_KEYWORD}"></label>
     17                <ul>
     18                        <li>
     19                                <label>{'Keyword'|@translate}</label>
     20                        </li>
     21                        <li>
     22                                <input type="text" name="keyword" value="{$F_KEYWORD}">
     23                        </li>
     24                </ul>
    1825
    19     <label>{'Author'|@translate}<input type="text" name="author" value="{$F_AUTHOR}"></label>
     26                <ul>
     27                        <li>
     28                                <label>{'Author'|@translate}</label>
     29                        </li>
     30                        <li>
     31                                <input type="text" name="author" value="{$F_AUTHOR}">
     32                        </li>
     33                </ul>
    2034
    21     <label>
    22       {'Album'|@translate}
    23       <select name="cat">
    24         <option value="0">------------</option>
    25         {html_options options=$categories selected=$categories_selected}
    26       </select>
    27     </label>
     35                <ul>
     36                        <li>
     37                                <label>{'Album'|@translate}</label>
     38                        </li>
     39                        <li>
     40                                <select name="cat">
     41                                        <option value="0">------------</option>
     42                                        {html_options options=$categories selected=$categories_selected}
     43                                </select>
     44                        </li>
     45                </ul>
    2846
    29     <label>
    30       {'Since'|@translate}
    31       <select name="since">
    32         {html_options options=$since_options selected=$since_options_selected}
    33       </select>
    34     </label>
     47                <ul>
     48                        <li>
     49                                <label>{'Since'|@translate}</label>
     50                        </li>
     51                        <li>
     52                                <select name="since">
     53                                        {html_options options=$since_options selected=$since_options_selected}
     54                                </select>
     55                        </li>
     56                </ul>
    3557
    3658  </fieldset>
     
    4062    <legend>{'Display'|@translate}</legend>
    4163
    42     <label>
    43       {'Sort by'|@translate}
    44       <select name="sort_by">
    45         {html_options options=$sort_by_options selected=$sort_by_options_selected}
    46       </select>
    47     </label>
     64                <ul>
     65                        <li>
     66                                <label>{'Sort by'|@translate}</label>
     67                        </li>
     68                        <li>
     69                                <select name="sort_by">
     70                                        {html_options options=$sort_by_options selected=$sort_by_options_selected}
     71                                </select>
     72                        </li>
     73                </ul>
    4874
    49     <label>
    50       {'Sort order'|@translate}
    51       <select name="sort_order">
    52         {html_options options=$sort_order_options selected=$sort_order_options_selected}
    53       </select>
    54     </label>
     75                <ul>
     76                        <li>
     77                                <label>{'Sort order'|@translate}</label>
     78                        </li>
     79                        <li>
     80                                <select name="sort_order">
     81                                        {html_options options=$sort_order_options selected=$sort_order_options_selected}
     82                                </select>
     83                        </li>
     84                </ul>
    5585
    56     <label>
    57       {'Number of items'|@translate}
    58       <select name="items_number">
    59         {html_options options=$item_number_options selected=$item_number_options_selected}
    60       </select>
    61     </label>
     86                <ul>
     87                        <li>
     88                                <label>{'Number of items'|@translate}</label>
     89                        </li>
     90                        <li>
     91                                <select name="items_number">
     92                                        {html_options options=$item_number_options selected=$item_number_options_selected}
     93                                </select>
     94                        </li>
     95                </ul>
    6296
    6397  </fieldset>
  • trunk/themes/default/template/search.tpl

    r28732 r28744  
    4949</fieldset>
    5050
     51{if count($AUTHORS)>1}
    5152<fieldset>
    5253  <legend>{'Search for Author'|@translate}</legend>
     
    5960  </p>
    6061</fieldset>
     62{/if}
    6163
    6264{if isset($TAGS)}
     
    136138</fieldset>
    137139<p>
    138   <input class="submit" type="submit" name="submit" value="{'Submit'|@translate}">
    139   <input class="submit" type="reset" value="{'Reset'|@translate}">
     140  <input type="submit" name="submit" value="{'Submit'|@translate}">
     141  <input type="reset" value="{'Reset'|@translate}">
    140142</p>
    141143</form>
  • trunk/themes/default/theme.css

    r28732 r28744  
    494494}
    495495
    496 SELECT.categoryList {
    497         height: 20em;
    498         width: 500px;
    499 }
    500 
    501496
    502497/**
     
    513508        display: block;
    514509        float: left;
    515         margin-right: 1em;
     510        margin: 0 1em 1em 0;
    516511        padding: 0;
    517512}
     
    524519.filter FIELDSET {
    525520  padding-bottom:0;
    526 }
    527 
    528 .filter LI LABEL {
    529         display: inline;
    530         float: none;
    531521}
    532522
     
    554544}
    555545
    556 /*IE6 needs override because of > selector*/
    557 .filter LABEL>INPUT,
    558 .filter LABEL>SELECT {
    559         /* display: block; */
    560         margin: 0.5em 0;
    561 }
    562 
    563 .filter FIELDSET * LABEL>INPUT {
    564         display: inline;
    565         vertical-align: top;
    566         margin: 0 0.5em 0 0;
    567 }
    568546
    569547
Note: See TracChangeset for help on using the changeset viewer.