pdfmake icon indicating copy to clipboard operation
pdfmake copied to clipboard

Feature/table cell vertical alignment

Open Kran67 opened this issue 2 years ago • 24 comments

Table cell vertical alignment - for stack, column, list, text, image, qr, svg

Kran67 avatar May 17 '22 08:05 Kran67

Hi, You can use table cell vertical alignment with one new property 'verticalAlign' (values: 'middle', 'bottom', see example)

Kran67 avatar May 17 '22 08:05 Kran67

@Gheeks @bpampuch if we could get this feature merged and released asap that would be lovely! Thank you

jbenjoy2 avatar Jun 01 '22 18:06 jbenjoy2

@Gheeks @bpampuch if we could get this feature merged and released asap that would be lovely! Thank you

@bpampuch definitely need to get this feature released asap so if we could please get this merged and released, we'd all really appreciate it.

jbenjoy2 avatar Jun 06 '22 17:06 jbenjoy2

Agreed. Eager for this feature. Thanks for your work!

Sliffcak avatar Jun 07 '22 14:06 Sliffcak

Can't wait for this to be merged

maxpropp avatar Jun 10 '22 07:06 maxpropp

why do we need all the begin/end - Vertical alignment comments everywhere? Git already shows a nice diff. Alos why do we need the __ prefix everywhere? Rest of this project seems to use "normal" names.

begin/end - Vertical alignment> only for information Prefix __ is only to avoid future naming conflicts

Kran67 avatar Jun 10 '22 07:06 Kran67

are there news regarding this?

maxpropp avatar Jun 22 '22 08:06 maxpropp

Any estimate on when this can be merged? I think this feature will greatly improve pdfmake tables!

Sliffcak avatar Jul 26 '22 15:07 Sliffcak

Guys please, we've been asking for this feature since 2014 ❤️

fe-art avatar Aug 31 '22 09:08 fe-art

It would be so awesome to have this merged in, plsplspls

DanielOfCourse avatar Sep 26 '22 15:09 DanielOfCourse

Is there a problem with this PR? So we can update it and finally merge it?

maxpropp avatar Oct 26 '22 11:10 maxpropp

@Kran67 have you addressed the changes so this can get merged?

ChrisSimoniMBT avatar Nov 17 '22 15:11 ChrisSimoniMBT

@bpampuch @liborm85 Any input on this PR?

ChrisSimoniMBT avatar Nov 17 '22 15:11 ChrisSimoniMBT

Would love to see this feature merged soon.

nw-onediscovery avatar Nov 29 '22 18:11 nw-onediscovery

Still waiting for this to be merged 👍🏼

MarcBollmann avatar Dec 14 '22 07:12 MarcBollmann

@bpampuch @liborm85 Please release this feature. Many thanks.

giangnghiemdinh avatar Aug 08 '23 04:08 giangnghiemdinh

Is there any logical reason why this branch is still not merged ? This is a really important feature.

merabtenei avatar Sep 07 '23 08:09 merabtenei

Wel I am in the team that created this PR. Is there any point in me taking the comments into account? will the PR eventually be integrated?

t0m-4 avatar Sep 14 '23 09:09 t0m-4

update pls.

anthonygacis avatar Oct 01 '23 03:10 anthonygacis

I'm in need of this feature, could this be merged and released?

manuelncfa avatar Nov 06 '23 14:11 manuelncfa

Would be great if this is merged 👍

jdwit avatar Feb 09 '24 14:02 jdwit

I prefer to install this library from the SirFull's fork rather than wait for this PR to be merged. Fortunately, package managers such as npm can do it directly from github from the specific branch: npm install SirFull/pdfmake#table_cell_vertical_alignment or from the specific commit: npm install SirFull/pdfmake#aec02a68af06c44991ae3dc45bdf443dde570ba4 I'm not sure any update will be as important as this one.

KostyanWest avatar Apr 01 '24 11:04 KostyanWest