Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Comparing changes

Choose two branches to see what's changed or to start a new pull request. If you need to, you can also compare across forks.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also compare across forks.
base fork: gallery/gallery3
base: 7b65a29
...
head fork: gallery/gallery3
compare: 5020f21
Checking mergeability… Don't worry, you can still create the pull request.
  • 2 commits
  • 2 files changed
  • 0 commit comments
  • 1 contributor
Commits on Nov 04, 2011
@bharat bharat Revert "Fix a bug introduced in dc21cf3 where"
It doesn't work for search because there are no children.  And this
just feels clunky to me - we should arrange for the root album to have
the right dimensions of thumbnail instead.

This reverts commit 4cb49b9.
1189ced
@bharat bharat Better fix for dc21cf3 - let's just fix up
the root album thumbnail when the proportions change so that we avoid
this problem altogether.
5020f21
View
3  modules/gallery/helpers/graphics.php
@@ -229,7 +229,6 @@ static function find_dirty_images_query() {
return db::build()
->from("items")
->and_open()
- ->and_open()
->where("thumb_dirty", "=", 1)
->and_open()
->where("type", "<>", "album")
@@ -239,8 +238,6 @@ static function find_dirty_images_query() {
->where("resize_dirty", "=", 1)
->where("type", "=", "photo")
->close()
- ->close()
- ->where("id", "<>", 1)
->close();
}
View
2  themes/wind/views/page.html.php
@@ -24,7 +24,7 @@
<link rel="apple-touch-icon-precomposed"
href="<?= url::file(module::get_var("gallery", "apple_touch_icon_url")) ?>" />
<? if ($theme->page_type == "collection"): ?>
- <? if (($thumb_proportion = $theme->thumb_proportion($children[0])) != 1): ?>
+ <? if (($thumb_proportion = $theme->thumb_proportion($theme->item())) != 1): ?>
<? $new_width = round($thumb_proportion * 213) ?>
<? $new_height = round($thumb_proportion * 240) ?>
<style type="text/css">

No commit comments for this range

Something went wrong with that request. Please try again.