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

Center an image #187

Open
JoanGoAl opened this issue Feb 21, 2024 · 7 comments
Open

Center an image #187

JoanGoAl opened this issue Feb 21, 2024 · 7 comments

Comments

@JoanGoAl
Copy link

Hi!
I'm using ${table:...:image}, the cell in the xlsx template is centered vertically and horizontally, but when I run the code the images are placed at the top left.

I hope you can help me. 😁

@jdugh
Copy link
Contributor

jdugh commented Feb 23, 2024

Hi @JoanGoAl , Unfortunately actually the image override doesn't support cell styling (centered vertically, horizontally, etc.).
I am currently working on a new 'imageincell' substitution which allows the image not to be inserted "on the cell" (the current behavior of image), but "in the cell". This is a “new” feature of Excel via “richdata”. The image then respects the formatting of the cell !
J'espère faire cette MAJ très prochainement (j'en ai besoin moi aussi).
Le point bloquant sera que cette fonctionnalité sera disponible que pour les Excel rescent.
I would see with @optilude for its integration.
If you can wait (a few days maybe), I will ping you about PR.

My current issue : #186

@Geoje1dong
Copy link

Geoje1dong commented Mar 19, 2024

I understand that you are working on the 'imageincell' feature, aiming to center images. May I kindly inquire about the expected update release date for this feature?

@jdugh
Copy link
Contributor

jdugh commented Mar 27, 2024

hi @Geoje1dong and @JoanGoAl :
My PR has been merge on master 2 days ago !
You can now use imageincell substituion keyword !
Only work with most recent version of Excel ! (see PR message)
And don't work with table substitution for the moment

image

@Geoje1dong
Copy link

@jduh Thank you. Is it still not available for use on the table?

@jdugh
Copy link
Contributor

jdugh commented Apr 18, 2024

Hi @Geoje1dong , available as soon as this PR #190 is validated.

@jdugh
Copy link
Contributor

jdugh commented Apr 19, 2024

Hi @Geoje1dong and @optilude ,
The PR being validated and merged, can we close this issue?

@Geoje1dong
Copy link

Geoje1dong commented Apr 23, 2024

@jdugh Thank you.

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

3 participants