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

Add Alt Text to charts #848

Closed
wants to merge 3 commits into from
Closed

Add Alt Text to charts #848

wants to merge 3 commits into from

Conversation

yorch
Copy link

@yorch yorch commented Sep 24, 2020

Alt text for charts is not currently populated, so it's not fully accesible. Adding a descr property to the charts translates to Alt Text box in PPT.

@gitbrent gitbrent added this to the 3.5.0 milestone Dec 26, 2020
@gitbrent gitbrent self-assigned this Dec 26, 2020
@gitbrent gitbrent modified the milestones: 3.5.0, 3.6.0 Feb 26, 2021
gitbrent added a commit that referenced this pull request Apr 27, 2021
@gitbrent
Copy link
Owner

Thanks @yorch

Screen Shot 2021-04-26 at 22 18 33

Screen Shot 2021-04-26 at 22 17 56

@gitbrent gitbrent closed this Apr 27, 2021
@yorch
Copy link
Author

yorch commented Apr 28, 2021

@gitbrent awesome, thank you!

@yorch
Copy link
Author

yorch commented May 4, 2021

@gitbrent just realized that the commit with the change does not include altText support for images as I included in this PR (https://github.com/gitbrent/PptxGenJS/pull/848/files#diff-2e10387a0f5301958ef328618c21a1df1de827655aec5abbe99bf0ba59631f5dR1785-R1789). Is this is something you can include?
Thanks

gitbrent added a commit that referenced this pull request May 9, 2021
@gitbrent
Copy link
Owner

gitbrent commented May 9, 2021

@yorch - quite correct. I meant to modify the current alt text on Images, but i got distracted. It's always had an alt text value of the image path, so i've set to where it uses altText if provided, then the path if none.

Screen Shot 2021-05-09 at 16 16 29

@yorch
Copy link
Author

yorch commented May 13, 2021

@gitbrent great, thanks so much! looking forward for a new version :D

@yorch
Copy link
Author

yorch commented Jul 21, 2021

@gitbrent just saw a new version was released, thanks, been waiting for this new release for a while. Unfortunately, the TS definition for addImage is not correct, it's missing the altText option.

It's here:

export interface ImageProps extends PositionProps, DataOrPathProps {

But it's not here:

export interface ImageProps extends PositionProps, DataOrPathProps {

Thanks again

gitbrent added a commit that referenced this pull request Jul 22, 2021
gitbrent added a commit that referenced this pull request Jul 22, 2021
@gitbrent
Copy link
Owner

@yorch Oops! Thanks for letting me know. v3.7.1 has been released with altText included in the defs file.

@yorch
Copy link
Author

yorch commented Jul 22, 2021

@gitbrent thanks! much appreciated!

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

Successfully merging this pull request may close these issues.

2 participants