Ticket #215 (closed enhancement: fixed)

Opened 2 years ago

Last modified 14 months ago

[Management App] Default still size adjustable in management interface

Reported by: Frans Owned by: robert
Priority: minor Milestone: MediaMosa 2.3
Component: Core Version:
Keywords: Cc:
MoSCoW: Must Have Estimated time after impact analysis:
Related to project: none Tested: yes
Accepted: yes Estimated Hours: 6

Description

Still size is defined in:

sites/all/modules/vpx_shared/vpx_shared_defines.php
Line 121:

// Still default size (qcif = 176x144)
define('VPX_STILL_DEFAULT_SIZE', 'qcif');

Would be nice to have this in the Management App.

Change History

Changed 22 months ago by Frans

  • moscow set to Should Have
  • milestone changed from MediaMosa X.X to MediaMosa 2.2

Changed 21 months ago by Frans

also per EGA

Changed 20 months ago by Frans

Setting per EGA (option 1)
Setting in the REST call (option 2)

Changed 20 months ago by robert

  • estimated_hours set to 8

Changed 17 months ago by Frans

  • moscow changed from Should Have to Must Have
  • milestone changed from MediaMosa 2.2 to MediaMosa 2.3

Changed 17 months ago by peter

  • estimated_hours changed from 8 to 6

Changed 16 months ago by robert

  • owner set to forgacs
  • status changed from new to assigned

Changed 16 months ago by MC-arjen

If an ega is created the default ega still size should be: 176x144

The size is adjustable per ega.

Is the ega setting is empty, then an empty size should be given to the still generation, and the default of ffmpeg should be taken: which is the size of the video.

Next to size, the "maintain_aspect_ratio" and "padding" should also be possible to define per ega.

Changed 16 months ago by forgacs

  • owner changed from forgacs to robert

If an ega is created the default ega still size should be: 176x144

Ready.

The size is adjustable per ega.

Ready.

Is the ega setting is empty, then an empty size should be given to the still generation, and the default of ffmpeg should be taken: which is the size of the video.

Ready.

Next to size, the "maintain_aspect_ratio" and "padding" should also be possible to define per ega.

Padding is implemented. Still aspect ratio is maintained as default, if there isn't padding parameter.

Robert: Needs a pair review.

Changed 16 months ago by robert

  • status changed from assigned to closed
  • resolution set to fixed

Changed 14 months ago by Michiel.Schok

  • tested set to yes
  • accepted changed from no to yes

Looks correctly implemented. Tested with some different sizes of stills and padding on/off.

Note: See TracTickets for help on using tickets.