Skip to content
This repository has been archived by the owner on Nov 10, 2017. It is now read-only.

Remove UPPERCASE text-transform from labels #53

Closed
philcockfield opened this issue Oct 4, 2016 · 5 comments
Closed

Remove UPPERCASE text-transform from labels #53

philcockfield opened this issue Oct 4, 2016 · 5 comments

Comments

@philcockfield
Copy link

philcockfield commented Oct 4, 2016

I understand stylistically the reason to put that transform on the property label, BUT with camelCase there is information contained with the name (the word-parts) that you are stomping out by applying this style. It's valid and helpful information for the developer working with the knobs:

image

It would be great if you reset this to display as the actual name of the property! Digging knobs, nice work!

@arunoda
Copy link
Contributor

arunoda commented Oct 4, 2016

I get the point. Happy to accept to a PR :)

@philcockfield
Copy link
Author

Ha ha! "Happy to accept to a PR :)" should be a keyboard shortcut. I'll take a look.

Also, want to talk about the index selector on StoryBook itself. Is that a pluggable module - there's some improvements I'd like to make to it. What's the feasibility of that?

@arunoda
Copy link
Contributor

arunoda commented Oct 4, 2016

What's index selector?
If that's something new, let's discuss in a new issue.

@philcockfield
Copy link
Author

philcockfield commented Oct 6, 2016

Hey @arunoda - I was referring to this when I said "index selector":

image

But that's not knobs, or even this issue, so I'll start a new issue on the storybook repo.

@arunoda
Copy link
Contributor

arunoda commented Oct 6, 2016

Yep. It's on the Storybook UI repo.
That's not pluggable yet.
But we want to customize it (To show stories only for changed snapshots).
Currently, there no plans on how we should do it.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

No branches or pull requests

3 participants