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

[Lightbox media viewer] add isolated image (media) view function #1106

Closed
wants to merge 1 commit into from

Conversation

shixiedesign
Copy link
Contributor

Related Ticket(s)

carbon-design-system/carbon-for-ibm-dotcom#6534

Description

Changelog

Changed

  • add events to 2.1 Media, where tap and tap again will enter and exist isolated image view

@shixiedesign shixiedesign changed the title add events to 2.1 Media [Lightbox media viewer] add isolated image (media) view function Jul 23, 2021
@ibmdotcom-bot
Copy link
Contributor

Copy link
Contributor

@photodow photodow left a comment

Choose a reason for hiding this comment

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

I see this is under media. Is this only for Image though?

I also need to think if there are any other edge cases, or roadblocks we might be neglecting throughout all of this. Have we considered accessibility on this, and how this works with a mouse and keyboard? Or transitioning between a lower breakpoint to a higher breakpoint? Will we need a new component for this? (I'm kind of thinking yes) Are there any transitional animations? Do we need a prototype (if we don't already have one) to make sure we've covered everything?

Comment on lines +91 to +94
| event | description |
|:-----------|:-------------|
| on tap | Triggers isolated media view on mobile. This view allows user to pinch, zoom and drag. |
| when zoomed out, on tap again | Returns to default view heading and copy present in the viewer |
Copy link
Contributor

Choose a reason for hiding this comment

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

Missing the reference number on these tables so we can reference them later if needed.


| event | description |
|:-----------|:-------------|
| on tap | Triggers isolated media view on mobile. This view allows user to pinch, zoom and drag. |
Copy link
Contributor

Choose a reason for hiding this comment

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

It's more events than just tap here. Each of the events you called out in this line needs its own row. Like ping, drag to name a couple. We have to be very explicit here, and make sure we get all of them.

@shixiedesign
Copy link
Contributor Author

Gonna icebox this for now and wait for other 2 features to merge first.

@jeffchew jeffchew deleted the fs-isolated-image-view branch May 11, 2022 15:27
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

Successfully merging this pull request may close these issues.

3 participants