diff options
| author | David Drury <david@drury.me.uk> | 2014-02-14 13:23:16 +0000 |
|---|---|---|
| committer | David Drury <david@drury.me.uk> | 2014-02-14 13:23:16 +0000 |
| commit | 30ae3c595b6b2a5000207df5e2a357ff9f8a7f12 (patch) | |
| tree | b787fd0b164f0a2ef6924b135c40ef0e5999bd0e | |
| parent | 2c6ef1187923ea5e1e97da66eea2a75a8798d970 (diff) | |
| download | webtrees-30ae3c595b6b2a5000207df5e2a357ff9f8a7f12.tar.gz webtrees-30ae3c595b6b2a5000207df5e2a357ff9f8a7f12.tar.bz2 webtrees-30ae3c595b6b2a5000207df5e2a357ff9f8a7f12.zip | |
Validation & Bug fix
The closing </div> for <div id="medialist-page"> is within an if
statement (if ($search)) which is false on initial page load. Apart from
a validation error this causes the page structure to be wrong and the
footer to be incorrectly styled
| -rw-r--r-- | medialist.php | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/medialist.php b/medialist.php index ca04d34c48..dde4c19ec6 100644 --- a/medialist.php +++ b/medialist.php @@ -27,7 +27,8 @@ require_once WT_ROOT.'includes/functions/functions_edit.php'; require_once WT_ROOT.'includes/functions/functions_print_facts.php'; $controller = new WT_Controller_Page(); -$controller->setPageTitle(WT_I18N::translate('Media objects')); +$controller->setPageTitle(WT_I18N::translate('Media objects')) + ->pageHeader(); $search = WT_Filter::get('search'); $sortby = WT_Filter::get('sortby', 'file|title', 'title'); @@ -65,8 +66,6 @@ $medialist = WT_Query_Media::mediaList( $filter ); -$controller->pageHeader(); - ?> <div id="medialist-page"><h2><?php echo $controller->getPageTitle(); ?></h2> @@ -311,6 +310,7 @@ if ($search) { echo '</tr></tbody>'; echo '</table>'; } - echo '</div> - </div>'; + echo '</div>'; } +echo '</div>'; + |