Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Optional arguments for field type "media" are missing #3

Open
LordRazen opened this issue Aug 11, 2021 · 1 comment
Open

Optional arguments for field type "media" are missing #3

LordRazen opened this issue Aug 11, 2021 · 1 comment

Comments

@LordRazen
Copy link

Description for Enhancement

Since I updated my field types due to this issue (https://github.com/vdm-io/Joomla-Component-Builder/issues/788) a minute ago, I recognized two optional attributes are missing for the field type "media".

I work with very small images, so the default size of 200px height stretches the images a lot...

Enhancement:

Add the missing parameters to the Field Type "media" / the field type updater:

Properties:
preview_width
preview_height

Adjustable: true
Mandatory: false
Translateable: false
Value: 200

Description:
(optional) sets the max-width of preview image (default: "200").
(optional) sets the max-height of preview image (default: "200").

Here's the official Joomla Info:
https://docs.joomla.org/Media_form_field_type

grafik

@LordRazen LordRazen changed the title optional arguments "preview_width" and "preview_height" for field type "media" are missing Optional arguments "preview_width" and "preview_height" for field type "media" are missing Aug 11, 2021
@LordRazen LordRazen changed the title Optional arguments "preview_width" and "preview_height" for field type "media" are missing Optional arguments for field type "media" are missing Aug 11, 2021
@Llewellynvdm Llewellynvdm transferred this issue from vdm-io/Joomla-Component-Builder Aug 11, 2021
@Llewellynvdm
Copy link
Member

This is the repo where we manage these changes.

@TLWebdesign we should in a next update also change these. Lets do one change per pull request, and not mix them up.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants