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

fix(image-with-caption): a11y fixes #4395

Merged

Conversation

asudoh
Copy link
Contributor

@asudoh asudoh commented Nov 4, 2020

Related Ticket(s)

Refs #4265.

Description

Fixes alt attribute of <dds-image-with-caption> not being propagated to <dds-image> in non-lightbox mode.

Also:

  • Introduces launch-lightbox-button-assistive-text attribute as the assistive text for the launcher button of the lightbox.
  • Adds a change to ensure undefined/null property values won't be set to <dds-image>, etc.

Changelog

New

  • launch-lightbox-button-assistive-text attribute as the assistive text for the launcher button of the lightbox.

Changed

  • Fixes alt attribute of <dds-image-with-caption>`` not being propagated to ` in non-lightbox mode.
  • Adds a change to ensure undefined/null property values won't be set to <dds-image>, etc.

@asudoh asudoh added the package: web components Work necessary for the IBM.com Library web components package label Nov 4, 2020
Fixes `alt` attribute of `<dds-image-with-caption>` not being
propagated to `<dds-image>` in non-lightbox mode.

Also:

* Introduces `launch-lightbox-button-assistive-text` attribute as the
  assistive text for the launcher button of the lightbox.
* Adds a change to ensure `undefined`/`null` property values won't be
  set to `<dds-image>`, etc.

Refs carbon-design-system#4265.
@asudoh asudoh force-pushed the image-with-caption-a11y-fixes branch from 0959121 to 7a10725 Compare November 4, 2020 11:05
@ibmdotcom-bot
Copy link
Contributor

ibmdotcom-bot commented Nov 4, 2020

@ibmdotcom-bot
Copy link
Contributor

ibmdotcom-bot commented Nov 4, 2020

Copy link
Member

@jeffchew jeffchew left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@jeffchew jeffchew added the Ready to merge Label for the pull requests that are ready to merge label Nov 5, 2020
@kodiakhq kodiakhq bot merged commit 8326e77 into carbon-design-system:master Nov 5, 2020
@asudoh asudoh deleted the image-with-caption-a11y-fixes branch November 6, 2020 00:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
package: web components Work necessary for the IBM.com Library web components package Ready to merge Label for the pull requests that are ready to merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants