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

Viewport Addon #1740

Merged
Merged

Conversation

saponifi3d
Copy link

@saponifi3d saponifi3d commented Aug 26, 2017

Description

This is a fairly early PR of a viewport plugin which allows you to resize the iframe of storybook.

Remaining Tasks

  • Fix lint issues
  • Add unit tests
  • Add documentation and screenshots to readme.

Possible Features

  • Responsive hooks, basically add the ability to drag the ends of the iframe to change the size
  • Improve interaction with the iframe rather than just grabbing the iframe from the DOM add listeners to the iframe to allow us to interact with it

Screen Cap

viewport

Issues

Fixes #1624

@shilman shilman self-assigned this Aug 26, 2017
}

componentDidMount() {
this.iframe = document.getElementById(storybookIframe);
Copy link
Member

Choose a reason for hiding this comment

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

LOL 🤣

@ndelangen ndelangen changed the base branch from master to release/3.3 August 26, 2017 14:16
@usulpro
Copy link
Member

usulpro commented Aug 26, 2017

Really great addon! Thanks for doing this!

I would add "write readme" to "Remaining Tasks".

I didn't dig into a code yet but I wonder whether it will be compatible with upcoming multistory feature (#1394)? Although not sure that it should, just want to share this thought

@shilman
Copy link
Member

shilman commented Aug 26, 2017

@saponifi3d @ndelangen I'd like to merge this into a feature branch in the storybook repo and add @saponifi3d to the storybooks org so we can more easily collaborate on fixing this up. OK?

@shilman shilman changed the base branch from release/3.3 to 1624-viewport-addon August 28, 2017 17:43
@shilman shilman merged commit 34f3136 into storybookjs:1624-viewport-addon Aug 28, 2017
@ndelangen
Copy link
Member

Welcome @saponifi3d ! 👍

@markshust
Copy link

Where is the git repo for this? Downloaded module from npm at https://www.npmjs.com/package/@storybook/addon-viewport -- all cool, but I have possible bugfix.

@saponifi3d
Copy link
Author

saponifi3d commented Sep 11, 2017

@markoshust it's part of the storybook monorepo, if you go into addons/viewport it'll be there :)

https://github.com/storybooks/storybook/tree/release/3.3/addons/viewport

Is the link, it's only part of the 3.3 feature branch

@ndelangen
Copy link
Member

@markoshust contact me on slack if you need help getting set up.

There's also the CONTRIBUTING.md document.

We're currently working off the release/3.3 branch as @saponifi3d mentioned.

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.

6 participants