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

Remember width of down panel when it's displayed on right #780

Closed
wants to merge 2 commits into from

Conversation

willowcheng
Copy link

Issue: storybook-eol/storybook-ui#62

What I did

Enable localStorage to remember position for both displaying down panel on bottom or right.

  • Change function/variables names to be more semantic.

(Carried over from storybook-eol/storybook-ui#80)

How to test

Follow storybook-ui/example/README.md to manual test, should be able to inspect changes in Local Storage from browser developer tool.

@willowcheng willowcheng changed the title Remember width of down panel when it's displayed on right 🎨 Remember width of down panel when it's displayed on right Apr 25, 2017
@ndelangen
Copy link
Member

Hey @willowcheng, Thanks for this PR!

We changed from a fork of react-split-view to the original version, so some things have changed. Could you have a look and possibly resolve the merge conflicts that have arisen because of this?

Let me know if you need anything, 🙇

@willowcheng
Copy link
Author

Hi @ndelangen, I think #956 already fixed my concern in a better way for this PR 😃, and it can be closed now.

@ndelangen ndelangen closed this Apr 28, 2017
@ndelangen
Copy link
Member

Ok awesome, thanks anyway!

@shilman shilman added the misc label May 27, 2017
Copy link

nx-cloud bot commented Dec 30, 2024

View your CI Pipeline Execution ↗ for commit c46464c.

Command Status Duration Result
nx run-many -t build --parallel=3 ✅ Succeeded 1m 36s View ↗

☁️ Nx Cloud last updated this comment at 2024-12-30 13:45:51 UTC

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