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

Drag to desktop when using sizing="contain" #76

Open
Westbrook opened this issue May 10, 2016 · 1 comment
Open

Drag to desktop when using sizing="contain" #76

Westbrook opened this issue May 10, 2016 · 1 comment

Comments

@Westbrook
Copy link
Contributor

Description

The iron-image component uses CSS backgroundImage when containing the image or having the image cover it. While this is awesome for layout, it applies [hidden] the actual img tag making it inaccessible to dragging to the desktop.

Desired outcome

It is possible to drag the image to the desktop.

Actual outcome

You can't drag the image.

Live Demo

https://elements.polymer-project.org/elements/iron-image?view=demo:demo/index.html&active=iron-image

Steps to reproduce

  1. Put a iron-image element in the page.
  2. Set the sizing property to "contain".
  3. Open the page in a web browser.

Conversation

Are there any accessibility issues with using something like position: relative; opacity: 0; width: 100%; height: 100%; in order to allow the image to be draggable but not effect the actual layout? In which case it would seem to make sense to either surface a CSS mixin, a component property to change the application of the [hidden], or possible both. I'd be happy to prepare a PR for what made the most sense to the health of the component.

@bicknellr
Copy link
Contributor

See comment on #77.

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

No branches or pull requests

2 participants