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.

Sign up for GitHub

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

Jump to bottom

Some early reviews #21

Closed
antoinerey opened this issue Apr 2, 2016 · 2 comments
Closed

Some early reviews #21

antoinerey opened this issue Apr 2, 2016 · 2 comments

Comments

@antoinerey
Copy link

First of all, this a very good idea ! :)

I find confusing that the story you select jumps to the top of the list. You can't rely on the order since it keeps changing every time you click on a new story. In my opinion, one solution could be to just expand the item in the list without touching the order.

Moreover, a way to empty the action logger box should be a very very good addition. It gets harder to find your informations when the box is filled.
@antoinerey antoinerey changed the title Keep the same order for stories, even when selecting a new one Some early reviews Apr 2, 2016
@arunoda
Copy link
Member

arunoda commented Apr 2, 2016

@antoinerey I agree on both. Will land a fix very early next week.

arunoda added a commit that referenced this issue Apr 4, 2016
@arunoda
Remove navigation order hijacking.
166365f 
Earlier, we move the selected kind and stories to the top.
It's kind a weird and discussed on #21
@arunoda
Copy link
Member

arunoda commented Apr 4, 2016

These two have been fixed in version 1.2.0.
Thanks for the feedback.

@arunoda arunoda closed this as completed Apr 4, 2016
ndelangen pushed a commit that referenced this issue Apr 5, 2017
@arunoda
Merge pull request #21 from kadirahq/remove-logger-name
32a1f9a 
Remove logger name with the updated react-inspector.
ndelangen pushed a commit that referenced this issue Apr 5, 2017
@arunoda
Merge pull request #21 from jasontwong/feature-select-knob
102e478 
Feature Select Knob
ndelangen pushed a commit that referenced this issue Apr 5, 2017
@arunoda
Merge pull request #21 from valscion/fix-missing-dependencies
636eec0 
Fix missing dependencies
ndelangen pushed a commit that referenced this issue Apr 5, 2017
@arunoda
Merge pull request #21 from kadirahq/arunoda-patch-1
8f02659 
Fix some typos
ndelangen pushed a commit that referenced this issue Apr 11, 2017
Merge pull request #21 from kadirahq/fix-user-info
cba1c52 
Rename getUsers to _getUsers
ndelangen pushed a commit that referenced this issue Apr 15, 2017
Merge pull request #21 from kadirahq/greenkeeper-mocha-3.1.0
33db142 
Update mocha to version 3.1.0 🚀
ndelangen pushed a commit that referenced this issue Nov 2, 2017
@ndelangen
allow default background (#21)
b987342 
* allow default background

* updated changelog for new default feature

* 0.0.6
JReinhold added a commit that referenced this issue May 31, 2023
@JReinhold
Merge pull request #21 from storybookjs/prerelease-pr-action
da5f210 
Initial prerelease PR workflow
ndelangen pushed a commit that referenced this issue Feb 23, 2024
@valentinpalkovic
Merge pull request #21 from storybookjs/feat/trigger-via-path
9320299 
Trigger via `/onboarding` path instead of query parameter
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@arunoda @antoinerey