•  » Engine
  •  » [multiple size] configuration screen redesign

#1 2012-04-02 16:24:53

plg
Piwigo Team
Nantes, France, Europe
2002-04-05
14048

[multiple size] configuration screen redesign

The screen to configure the new feature multiple size contains far too many options to be usable by humans (in my opinion). It shows all what Piwigo can do but it doesn't help the user. Here is what I propose and questions related to implied changes:

1) we remove the sharpen parameter. Very useful for GD (any other value than 0 breaks image generation with ImageMagick) but too complicated for users. I think we'd better hardcode a default setting (this is what I did with ImageMagick since Piwigo support ImageMagick)

2) by default we don't display dimensions as form fields but as summary :

[x] XXS - tiny cropped 240x240, [change settings]
[x] XS - very small, 432x324, [change settings]
[ ] S - small

Size disabled = no settings summary

When you click on the "change settings" links, it show the form fields :

Code:

[x] [b]XXS - tiny[/b] cropped 240x240, [link: change settings]
  [x] Crop
  Width
  [240 ]
  Height
  [240 ]

[x] [b]XS - very small[/b], 432x324, [link: change settings]
  [] Crop
  Maximum width
  [432 ]
  Maximum height
  [324 ]

If you check the "crop" option, "Maximum width" becomes "Width".

I've played with the current width, height, crop(%), min width, min height and I haven't been able to understand the way these settings were working altogether :-/ I mean for example an crop 45% and a width != min width. I think that the ergonomy we had worked on with P@t was quite simple and understandable.

This ergonomy is equal to a crop 100% and width=minWidth and height=maxHeight in the new system. Maybe it's less possibilities for users but at least they can configure their gallery and expect the behavior they want.

The problem with these changes is that there won't be an "easy way" to use Piwigo settings at full capacity without generating something like :

$conf['derivatives_custom'] = ' a:3:{s:1:"d";a:9:{s:6:"square";O:16:"DerivativeParams":4:{s:13:"last_mod_time";i:0;s:6:"sizing";O:12:"SizingParams":3:{s:10:"ideal_size";a:2:{i:0;i:120;i:1;i:120;}s:8:"max_crop";d:1;s:8:"min_size";a:2:{i:0;i:120;i:1;i:120;}}s:7:"sharpen";i:0;s:7:"quality";i:85;}s:5:"thumb";O:16:"DerivativeParams":4:{s:13:"last_mod_time";i:0;s:6:"sizing";O:12:"SizingParams":3:{s:10:"ideal_size";a:2:{i:0;i:144;i:1;i:144;}s:8:"max_crop";d:0;s:8:"min_size";a:2:{i:0;i:0;i:1;i:0;}}s:7:"sharpen";i:0;s:7:"quality";i:85;}s:6:"2small";O:16:"DerivativeParams":4:{s:13:"last_mod_time";i:1333373332;s:6:"sizing";O:12:"SizingParams":3:{s:10:"ideal_size";a:2:{i:0;i:240;i:1;i:145;}s:8:"max_crop";d:1;s:8:"min_size";a:2:{i:0;i:240;i:1;i:145;}}s:7:"sharpen";i:0;s:7:"quality";i:95;}s:6:"xsmall";O:16:"DerivativeParams":4:{s:13:"last_mod_time";i:0;s:6:"sizing";O:12:"SizingParams":3:{s:10:"ideal_size";a:2:{i:0;i:432;i:1;i:324;}s:8:"max_crop";d:0;s:8:"min_size";a:2:{i:0;i:0;i:1;i:0;}}s:7:"sharpen";i:0;s:7:"quality";i:85;}s:5:"small";O:16:"DerivativeParams":4:{s:13:"last_mod_time";i:0;s:6:"sizing";O:12:"SizingParams":3:{s:10:"ideal_size";a:2:{i:0;i:576;i:1;i:432;}s:8:"max_crop";d:0;s:8:"min_size";a:2:{i:0;i:0;i:1;i:0;}}s:7:"sharpen";i:0;s:7:"quality";i:85;}s:6:"medium";O:16:"DerivativeParams":4:{s:13:"last_mod_time";i:1333225678;s:6:"sizing";O:12:"SizingParams":3:{s:10:"ideal_size";a:2:{i:0;i:792;i:1;i:594;}s:8:"max_crop";d:0;s:8:"min_size";a:2:{i:0;i:0;i:1;i:0;}}s:7:"sharpen";i:0;s:7:"quality";i:95;}s:5:"large";O:16:"DerivativeParams":4:{s:13:"last_mod_time";i:0;s:6:"sizing";O:12:"SizingParams":3:{s:10:"ideal_size";a:2:{i:0;i:1008;i:1;i:756;}s:8:"max_crop";d:0;s:8:"min_size";a:2:{i:0;i:0;i:1;i:0;}}s:7:"sharpen";i:0;s:7:"quality";i:85;}s:6:"xlarge";O:16:"DerivativeParams":4:{s:13:"last_mod_time";i:0;s:6:"sizing";O:12:"SizingParams":3:{s:10:"ideal_size";a:2:{i:0;i:1224;i:1;i:918;}s:8:"max_crop";d:0;s:8:"min_size";a:2:{i:0;i:0;i:1;i:0;}}s:7:"sharpen";i:0;s:7:"quality";i:85;}s:7:"xxlarge";O:16:"DerivativeParams":4:{s:13:"last_mod_time";i:0;s:6:"sizing";O:12:"SizingParams":3:{s:10:"ideal_size";a:2:{i:0;i:1656;i:1;i:1242;}s:8:"max_crop";d:0;s:8:"min_size";a:2:{i:0;i:0;i:1;i:0;}}s:7:"sharpen";i:0;s:7:"quality";i:85;}}s:1:"w";O:15:"WatermarkParams":6:{s:4:"file";s:0:"";s:8:"min_size";a:2:{i:0;i:500;i:1;i:500;}s:4:"xpos";i:50;s:4:"ypos";i:50;s:7:"xrepeat";i:0;s:7:"opacity";i:100;}s:1:"c";a:0:{}}';

Which is highly unreadable of course.

rvelices, how do you see a "full possibilities" settings system for derivatives while keeping an "easy to understand" user interface?

Offline

 

#2 2012-04-02 17:04:19

rvelices
Former Piwigo Team
2005-12-29
1960

Re: [multiple size] configuration screen redesign

plg wrote:

rvelices, how do you see a "full possibilities" settings system for derivatives while keeping an "easy to understand" user interface?

Not really. I won't be using any of the "advanced" features. I think they are useful only for some very large/extreme panoramas where he would like to have a crop of max "50%" in order to have a viewable thumbnail that is also contained.

For the sharpening OK, but something should be there for GD because GD images look really soft. Only Image magick as an extension implemetns the sharpening during resize - we could just remove the code that performs aditionnal sharpening here and there will be no error

Offline

 

#3 2012-04-03 21:22:28

Zaphod
Former Piwigo Team
2006-11-13
441

Re: [multiple size] configuration screen redesign

I think there is one parameter missing.

There two different things you may want to do:

-> resize to 800x600 with 800 = width, 600 = height
-> resize to 800x600 with 800 = long edge, 600 = short edge

In my opinion, that's important to be able to do both.
That was more or less possible with the current beta system (but not obvious...).
This not seems to be possible with the new presentation.

Offline

 

#4 2012-04-03 21:37:12

plg
Piwigo Team
Nantes, France, Europe
2002-04-05
14048

Re: [multiple size] configuration screen redesign

If I understand correctly, this is what P@t was calling the "follow orientation" option in Piwigo 2.3. This feature is not currently possible in Piwigo 2.4 (and I think it won't be in 2.4.0)

If what your describe is possible with the current configuration screen, it is very well hidden.

Offline

 

#5 2012-04-03 22:00:37

Zaphod
Former Piwigo Team
2006-11-13
441

Re: [multiple size] configuration screen redesign

I currently use this option (follow orientation) on all my thumbnail creation.

Currently, for example if you chose thumbnails :
144x144, crop, min size 108x108.
In most cases you will have 144x108 or 108x144 thumbnails (exactly what I want).
That does not work if the image has, for example, a native square ratio.
(that's why I think the previous "follow orientation" option was better)

You say it's not possible, is it a technical impossibility?
Or is it just the timing?

Last edited by Zaphod (2012-04-03 22:01:25)

Offline

 

#6 2012-04-03 22:13:30

plg
Piwigo Team
Nantes, France, Europe
2002-04-05
14048

Re: [multiple size] configuration screen redesign

OK, I thought you were comparing the 2.4.0RC1 config screen with what I've just shown on [Forum, post 128830 by plg in topic 19144] Multi size : crop false

On both cases, no "follow orientation" setting like we had on 2.3.

You say it's not possible, is it a technical impossibility? Or is it just the timing?

It has not been implemented in the current version of the "derivative" system.

Considering that we already have the language key 'Follow Orientation' coming from Piwigo 2.3, if rvelices/P@t/plg finds time to implement it, it can be done even after 2012-04-06 (deadline for changes in language files)

Offline

 

#7 2012-04-03 23:51:22

Zaphod
Former Piwigo Team
2006-11-13
441

Re: [multiple size] configuration screen redesign

plg wrote:

OK, I thought you were comparing the 2.4.0RC1 config screen with what I've just shown on [Forum, post 128830 by plg in topic 19144] Multi size : crop false

I also was. (I used twice "current", once for 2.3, once for 2.4rc1, that was not very obvious..)

In terms of thumbnail generation for my use, 2.3 > 2.4rc1 > 2.4 after redesign.
The 2.3 system was perfect for me.
With the 2.4rc1 system, I still got a way to do what I want with my thumbs (as I described, using for example 144x144, 100% crop and min 100x100).
I won't with the system you've just shown (you should put the screen captures here...)

Don't get me wrong, I find the way to present the multisize configuration screen far better after redesign, I just miss this tiny option.

Last edited by Zaphod (2012-04-03 23:53:04)

Offline

 

#8 2012-04-04 00:14:36

plg
Piwigo Team
Nantes, France, Europe
2002-04-05
14048

Re: [multiple size] configuration screen redesign

I understand what you mean.

http://piwigo.org/forum/showimage.php?pid=128830&filename=localhost+screen+capture+2012-4-3-20-53-26.png

I hope we'll have time to re-implement the "follow orientation" feature from 2.3 to 2.4.

Offline

 

#9 2012-04-04 14:05:56

Zaphod
Former Piwigo Team
2006-11-13
441

Re: [multiple size] configuration screen redesign

Anyway, is this the kind of change than can be part of a minor release?
Like 2.4.1?

Offline

 

#10 2012-04-04 14:17:08

plg
Piwigo Team
Nantes, France, Europe
2002-04-05
14048

Re: [multiple size] configuration screen redesign

Zaphod wrote:

Anyway, is this the kind of change than can be part of a minor release?
Like 2.4.1?

Yes, no problem. Because we already have the language keys and it can be considered as a bug because the feature was available in 2.3.

Offline

 

#11 2012-04-19 16:48:52

plg
Piwigo Team
Nantes, France, Europe
2002-04-05
14048

Re: [multiple size] configuration screen redesign

I've just commited some changes in [Subversion] r14221

http://piwigo.org/forum/showimage.php?pid=129189&filename=localhost+screen+capture+2012-4-19-16-46-24.png

I think that one thing is really missing: the ability to restore default settings. The problem is that we can't add any other language key now. We have "restore" but not "restore default settings". rvelices, would you like to add a link "restore" + "javascript:are you sure?" in the Multiple Size fieldset?

Offline

 

#12 2012-04-19 18:18:16

rvelices
Former Piwigo Team
2005-12-29
1960

Re: [multiple size] configuration screen redesign

plg wrote:

I think that one thing is really missing: the ability to restore default settings. The problem is that we can't add any other language key now. We have "restore" but not "restore default settings". rvelices, would you like to add a link "restore" + "javascript:are you sure?" in the Multiple Size fieldset?

I'll do something about it, but I'll let you handle the original size fieldset.
However please leave the other screen - (no watermarks here are a no go for me)
Also note that if someone combines the usage of the two screens, the cache might not be correctly cleared because of watermark usage changes ...

Offline

 

#13 2012-04-19 18:26:03

flop25
Piwigo Team
2006-07-06
7038

Re: [multiple size] configuration screen redesign

rvelices wrote:

(no watermarks here are a no go for me)

me too !
But I'm thinking that feature should not be displayed on that page. On a new tab maybe


To get a better help : Politeness like Hello-A link-Your past actions precisely described
Check my extensions : more than 30 available
who I am and what I do : http://fr.gravatar.com/flop25
My gallery : an illustration of how to integrate Piwigo in your website

Offline

 

#14 2012-04-19 18:33:27

plg
Piwigo Team
Nantes, France, Europe
2002-04-05
14048

Re: [multiple size] configuration screen redesign

flop25 wrote:

rvelices wrote:

(no watermarks here are a no go for me)

me too !
But I'm thinking that feature should not be displayed on that page. On a new tab maybe

Exactly, I plan to add another tab "Watermark" next to "Photo sizes"

Offline

 

#15 2012-04-19 18:36:23

plg
Piwigo Team
Nantes, France, Europe
2002-04-05
14048

Re: [multiple size] configuration screen redesign

rvelices wrote:

plg wrote:

I think that one thing is really missing: the ability to restore default settings. The problem is that we can't add any other language key now. We have "restore" but not "restore default settings". rvelices, would you like to add a link "restore" + "javascript:are you sure?" in the Multiple Size fieldset?

I'll do something about it, but I'll let you handle the original size fieldset.

Great! I was not thinking about restoring default settings on the original size, but only about multiple size (something like clear_derivative_cache(); + conf_update_param('derivatives', ''); I suppose but I prefer you to it cleanly)

rvelices wrote:

Also note that if someone combines the usage of the two screens, the cache might not be correctly cleared because of watermark usage changes ...

I'll ask you to check again once the new Watermark screen is implemented :-)

Offline

 
  •  » Engine
  •  » [multiple size] configuration screen redesign

Board footer

Powered by FluxBB

github linkedin newsletter Piwigo.org © 2002-2026 · Contact