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

Finish coding the hero animation #28

Closed

Conversation

cdedreuille
Copy link
Collaborator

@cdedreuille cdedreuille commented Apr 15, 2024

This is a new design to show the core features of Storybook. The UI is 100% custom and works on mobile, tablet and desktop. We were missing the visual test addon part.

I also improved the slider animation to make sure that if you click on a tab it would reset the timer.

CleanShot 2024-04-15 at 16 54 27@2x

Copy link

linear bot commented Apr 15, 2024

Copy link

netlify bot commented Apr 15, 2024

Deploy Preview for storybook-web ready!

Name Link
🔨 Latest commit e3f13f9
🔍 Latest deploy log https://app.netlify.com/sites/storybook-web/deploys/661d4cc0e0d35f0008a12a7e
😎 Deploy Preview https://deploy-preview-28--storybook-web.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@cdedreuille cdedreuille self-assigned this Apr 15, 2024
}, []);

const handleSlideChange = (newSlide: number) => {
Copy link
Contributor

Choose a reason for hiding this comment

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

I expected it to stop automatically switching between the four tabs once I clicked on one. Wdyt?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

To stop automatically? If you click it reset the timer and will start again after 3 seconds. I don't think we should make it stop forever.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

After some thoughts and discussion you were right. It will look better if we stopped the animation if the user clicks on it. I'll change that.

import { Doc } from './doc';
import { PanelInteractions } from './panel-interactions';
import { Canvas } from './canvas';
import { PanelVisualTesting } from './panel-visual-testing';

export const Manager: FC<{ slide: number }> = ({ slide }) => {
return (
<div
Copy link
Contributor

Choose a reason for hiding this comment

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

(Not sure where this feedback belongs.)

I find myself missing the animated cursor. It's pretty hard to parse the difference between each feature, as it is.

The simulated interactions for each feature in the current version also did a lot to illustrate them. E.g. navigating between different stories, changing control values. And while it's not in the current one, I think clicking on a couple different interactions steps would be similarly helpful.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I think we should not confuse 2 things:

  • A guided tour of the product
  • A presentation of the tool in general

With this update I'm going after the second one so people can understand through the tab and the visual the main features. You don't need to go in details but just understand if this looks "cool" enough or with the right keywords to learn more about it.

The guided tour should be done perhaps in a video as you need people to stick to it. The version we have today is way too slow, incomplete and with no way to come back to a specific point.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

We discussed this one on Slack and in our design Jam and you're right, we could introduce some animations on each slide to make it more interactive. This will require more work but it is worth it. I'll work on it.

@cdedreuille cdedreuille deleted the charles/sb-1382-home-finish-coding-the-hero-animation branch May 15, 2024 16:32
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.

None yet

2 participants