Problem/Motivation:-
Currently in the media entity browser the number of images shown per page is limited to 12 by default.
Proposed Resolution:-
A configuration settings to be provided where the user can set this number of images to be shown per in the entity browser.
Tasks
✔ 1. Add a new form group to the DAM Settings page "Entity Browser Settings"
✔ 2. Add a numeric field to the form "Number of images per page"
✔ 3. Create a config setting in media_acquiadam.settings.settings.yml file and its default value to 12
num_images_per_page: 12
✔ 4. Via dependency injection call the config service in Entity browser plugin in Acquiadam.php file.
✔ 5. At this line number 236 call the config setting
// Number of assets to show per page.
$num_per_page =12
replace with
// Number of assets to show per page.
$num_per_page = $config->get('num_images_per_page');
Comment | File | Size | Author |
---|---|---|---|
#8 | Screen Shot 2020-05-18 at 3.41.30 PM.png | 18.83 KB | nitvirus |
#3 | 3137182-2.patch | 6.01 KB | tdnshah |
|
Comments
Comment #2
tdnshah CreditAttribution: tdnshah as a volunteer commentedComment #3
tdnshah CreditAttribution: tdnshah as a volunteer commentedAdding Patch with this feature
Comment #4
tdnshah CreditAttribution: tdnshah as a volunteer commentedComment #5
tdnshah CreditAttribution: tdnshah as a volunteer commentedComment #6
nitvirus CreditAttribution: nitvirus at Srijan | A Material+ Company commentedComment #7
nitvirus CreditAttribution: nitvirus at Srijan | A Material+ Company commentedComment #8
nitvirus CreditAttribution: nitvirus at Srijan | A Material+ Company commentedLooks good to me. The config is present at "/admin/config/media/acquiadam"
RTBC
Comment #10
jmoreira CreditAttribution: jmoreira at Acquia commentedComment #11
jmoreira CreditAttribution: jmoreira at Acquia commented