Stirling-PDF icon indicating copy to clipboard operation
Stirling-PDF copied to clipboard

Multiple images in one page

Open sorydi3 opened this issue 1 year ago • 2 comments

Description

This pull address the following user request. request

AFFECTED UI LINK:

Note: All other features work as usual.

New Feature: Add multiple images in one page.

  • Template Selection: Users can choose from a set of predefined templates that dictate how images are organised on each page. Available templates include:

    • 1x2 (Two per page): Two images will be arranged on each page. (from left to right)
    • 2x2 (Four per page): Four images will be arranged on each page. (from left to right)
    • 2x3 (Six per page): Six images will be arranged on each page. (from left to right)
  • Handling Fewer Images: If fewer images are selected than the number of slots in the chosen template, the extra slots will be left empty. Only the selected images will be displayed.

  • Size and Spacing: The size and spacing of images will be automatically adjusted according to the selected template, maintaining the aspect ratio of each image. Users will not need to make any manual adjustments, as the templates will optimize the layout to fit the specified number of images per page.

Closes #(1400)

Exemple:

UI:

Screenshot 2024-08-25 at 20 52 07

Output for 1x2 ( 1 row two colomn ):

Screenshot 2024-08-25 at 20 57 18

Checklist:

  • [x] I have read the Contribution Guidelines
  • [x] I have performed a self-review of my own code
  • [x] I have commented my code, particularly in hard-to-understand areas
  • [x] My changes generate no new warnings

sorydi3 avatar Aug 25 '24 19:08 sorydi3

Love the idea, I noticed in your screenshot 2 images produced 1 page with image but also made one blank page?

I also think instead of having the templates defined like the grid layout done by our https://stirlingpdf.io/split-pdf-by-sections would be better suited image

Frooodle avatar Aug 25 '24 20:08 Frooodle

Yep look good to me also , I did not know we had this feature. I'll do the changes.
The blank page is because i uploaded the wrong foto.

sorydi3 avatar Aug 27 '24 19:08 sorydi3

🚀 Translation Verification Summary

🔄 Reference Branch: pr-branch-messages_en_GB.properties

📃 File Check: messages_ar_AR.properties

  1. Test Status:Passed
  2. Test Status:Passed

📃 File Check: messages_bg_BG.properties

  1. Test Status:Passed
  2. Test Status:Passed

📃 File Check: messages_ca_CA.properties

  1. Test Status:Passed
  2. Test Status:Passed

📃 File Check: messages_cs_CZ.properties

  1. Test Status:Passed
  2. Test Status:Passed

📃 File Check: messages_de_DE.properties

  1. Test Status:Passed
  2. Test Status:Failed
  • Issue:
    • Extra keys in messages_de_DE.properties: pdfPrompwt that are not present in pr-branch-messages_en_GB.properties.
    • Missing keys in pr-branch-messages_en_GB.properties: pdfPrompt that are not present in messages_de_DE.properties.

📃 File Check: messages_el_GR.properties

  1. Test Status:Passed
  2. Test Status:Passed

📃 File Check: messages_en_GB.properties

  1. Test Status:Passed
  2. Test Status:Passed

📃 File Check: messages_en_US.properties

  1. Test Status:Passed
  2. Test Status:Passed

📃 File Check: messages_es_ES.properties

  1. Test Status:Passed
  2. Test Status:Passed

📃 File Check: messages_eu_ES.properties

  1. Test Status:Passed
  2. Test Status:Passed

📃 File Check: messages_fr_FR.properties

  1. Test Status:Passed
  2. Test Status:Passed

📃 File Check: messages_hi_IN.properties

  1. Test Status:Passed
  2. Test Status:Passed

📃 File Check: messages_hr_HR.properties

  1. Test Status:Passed
  2. Test Status:Passed

📃 File Check: messages_hu_HU.properties

  1. Test Status:Passed
  2. Test Status:Passed

📃 File Check: messages_id_ID.properties

  1. Test Status:Passed
  2. Test Status:Passed

📃 File Check: messages_it_IT.properties

  1. Test Status:Passed
  2. Test Status:Passed

📃 File Check: messages_ja_JP.properties

  1. Test Status:Passed
  2. Test Status:Passed

📃 File Check: messages_ko_KR.properties

  1. Test Status:Passed
  2. Test Status:Passed

📃 File Check: messages_nl_NL.properties

  1. Test Status:Passed
  2. Test Status:Passed

📃 File Check: messages_no_NB.properties

  1. Test Status:Passed
  2. Test Status:Passed

❌ Overall Check Status: Failed

@sorydi3 please check your translation if it conforms to the standard. Follow the format of messages_en_GB.properties

github-actions[bot] avatar Jan 06 '25 19:01 github-actions[bot]