rvelices, thanks for [Subversion] r14228
Either I code the "restore" for original size, or I move the button to the "Multiple Size" fieldset. What do you think is better?
Offline
plg wrote:
Either I code the "restore" for original size, or I move the button to the "Multiple Size" fieldset. What do you think is better?
Move the button
Offline
You know what's cool? I've found this in common.lang.php: $lang['Reset to default values'] (used on profile.php)
Offline
- I don't know if you plan to remove the original "dev" multi size config screen, but if it's the case, is it ok to add the sharpen param in the new screen (it works well now with the imagick extension on recently upgraded piwigo.org) ?
- Before we release 2.4 is it ok to make the jpeg quality a global parameter in the code as it is in the new screen ?
Offline
Yes, it is planned to remove the original configuration screen. I used it to check my dev was OK.
I'm perfectly OK with the changes you propose. The only problem I see is that we have no "Sharpen" language key and it's too late to add one :-/
Offline
plg wrote:
The only problem I see is that we have no "Sharpen" language key and it's too late to add one :-/
Can we say errors are human and we add it now ? (I would really like to use this param)
Offline
rvelices wrote:
Can we say errors are human and we add it now ? (I would really like to use this param)
I've asked ddtddt.
Offline
ddtddt is OK. You can add $lang['Sharpen'].
Offline
I still have a "multiple size" screen like it was before, in addition to the "options / photo sizes".
Is this normal?
I get the following errors:
Notice: Undefined property: DerivativeParams::$quality in /homepages/(...)/betatest/admin/derivatives.php
Offline
If I find the time before 2.4.0 (or right after, that's not blocking), I'll change the text field for sharpen/quality into a slider http://jqueryui.com/demos/slider/#steps this will avoid any mistake by users.
Offline
Zaphod wrote:
I still have a "multiple size" screen like it was before, in addition to the "options / photo sizes".
Is this normal?
I get the following errors:Code:
Notice: Undefined property: DerivativeParams::$quality in /homepages/(...)/betatest/admin/derivatives.php
Screen [Administration > Configuration > Multiple Size] will be removed before 2.4.0
Offline
plg wrote:
I hope we'll have time to re-implement the "follow orientation" feature from 2.3 to 2.4.
Any new for this?
Even if it's not implemented in the interface, currently it's not possible at all... even with something like:
{define_derivative name='derivative_params' width=160 height=90 crop=true}in a template...
Offline
Zaphod wrote:
plg wrote:
I hope we'll have time to re-implement the "follow orientation" feature from 2.3 to 2.4.
Any new for this?
Even if it's not implemented in the interface, currently it's not possible at all... even with something like:Code:
{define_derivative name='derivative_params' width=160 height=90 crop=true}in a template...
{define_derivative name='derivative_params' width=144 height=144 crop=true min_width=108 min_height=108}Offline
Thanks, that's perfect.
So I will be able to switch to piwigo 2.4 as soon at is released.
Offline