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

Feature/zhusee editable text #40

Merged
merged 9 commits into from
May 17, 2017
Merged

Conversation

zhusee2
Copy link
Contributor

@zhusee2 zhusee2 commented May 16, 2017

Purpose

Create a <Text> element with an <input> inside.
This visual element is supposed to be used within <TextInput>.

Implement

  1. The <EditableText> is a special version of <Text>, which places an <input> absolutely above the basic row. The <input> stays transparent until it gets focused.
  2. The border mentioned in design spec is set on the basic row <div> instead of <input>, because the border should be placed below the basic row and thus will be hidden by overflow: hidden rule.
  3. When the <input> is focused, itself will be visible while the basic text under it will be hidden. Because the basic text and the <input> should look the same, the only difference on screen should be only the border line.
  4. All status-related elements will be hidden from <Text> to prevent distraction in edit mode.

Demo

2017-05-16 6 25 43

…content

Also tweaks to prevent Terminal console being flooded by chunk modules.
However the settings will not be taken into middleware config at this version
of Storybook (2.35.3)

It's expected to be effective on next relase of Storybook.
@zhusee2 zhusee2 requested a review from cjies May 16, 2017 10:35
@zhusee2 zhusee2 force-pushed the feature/zhusee_editable_text branch from 57ed92d to 4b1d521 Compare May 16, 2017 10:36
@zhusee2 zhusee2 merged commit 36077db into develop May 17, 2017
@zhusee2 zhusee2 deleted the feature/zhusee_editable_text branch May 17, 2017 07:50
This was referenced May 24, 2017
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.

2 participants