Piwigo.org

plg · 2012-05-24 17:13:09

Hi rvelices,

In [Forum, post 127614 by rvelices in topic 18755] [multisize] AJAX load for thumbnails I'm not sure you replied to my question "how to I know which size is the default size?".

As we have changed the default dimensions for sizes, and that size L was 648x576 and is now 1008x756, this is now far too big for a default size. I would like to switch the default size to size M instead (which is currently 792x594 by default).

Billr · 2012-05-24 17:26:48

In support of this, a common default the past 10 years or so for monitor display has been 800x600 (SVGA).

rvelices · 2012-05-24 19:05:35

plg wrote:
As we have changed the default dimensions for sizes, and that size L was 648x576 and is now 1008x756, this is now far too big for a default size. I would like to switch the default size to size M instead (which is currently 792x594 by default).

OK, but please put the defaiult size in config_default

plg · 2012-05-24 20:02:22

Actually, I don't understand what in the code makes size L as default size. I search, I read, I can't find :-/

rvelices · 2012-05-24 21:43:39

plg wrote:
Actually, I don't understand what in the code makes size L as default size. I search, I read, I can't find :-/

picture.php function default_picture_content
$deriv_type = pwg_get_session_var('picture_deriv', IMG_LARGE);

plg · 2012-05-24 22:08:51

Perfect, I was searching in include/derivative_std_params.inc.php instead of performing a grep on IMG_LARGE on all *.php files!

Are you ok for this new parameter in include/config_defaults.inc.php?

Code:

// 'small', 'medium' or 'large'
$conf['derivative_default_size'] = 'medium';

and this change in picture.php?

Code:

$deriv_type = pwg_get_session_var('picture_deriv', $conf['derivative_default_size']);

(twice on the page)

We can't use the constants defined in include/derivative_std_params.inc.php.

rvelices · 2012-05-24 22:40:10

plg wrote:
Are you ok for this new parameter in include/config_defaults.inc.php?

perfect

plg · 2012-05-25 01:12:46

[Bugtracker, feature 2633, fixed] [multiple size] ability to select default size and [Subversion] r15299

Piwigo.org

Announcement

#1 2012-05-24 17:13:09

[multisize] default size

#2 2012-05-24 17:26:48

Re: [multisize] default size

#3 2012-05-24 19:05:35

Re: [multisize] default size

plg wrote:

#4 2012-05-24 20:02:22

Re: [multisize] default size

#5 2012-05-24 21:43:39

Re: [multisize] default size

plg wrote:

#6 2012-05-24 22:08:51

Re: [multisize] default size

Code:

Code:

#7 2012-05-24 22:40:10

Re: [multisize] default size

plg wrote:

#8 2012-05-25 01:12:46

Re: [multisize] default size

Board footer