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

Form components #89

Merged
merged 2 commits into from
Nov 25, 2020
Merged

Form components #89

merged 2 commits into from
Nov 25, 2020

Conversation

herablog
Copy link
Member

@herablog herablog commented Nov 6, 2020

こちらもプレビュー作るためにドラフトPRします。

Storybook: https://ameba-spindle--pr89-feature-form-compone-dwjafddy.web.app/?path=/docs/form-checkbox--checkbox

  • Checkbox
  • DropDown
  • InputLabel
  • InvalidMessage
  • Radio
  • TextArea
  • TextField @hiloki @syasuda90 [プロパティ名確認][https://github.com/Form components #89#discussion_r525864343)
  • ToggleSwitch

@reg-suit
Copy link

reg-suit bot commented Nov 6, 2020

reg-suit detected visual differences.

Check this report, and review them.

⚪⚪⚪⚪⚪⚪⚪⚪⚪⚪⚪⚪⚪⚪⚪⚪⚪⚪⚪
⚫⚫⚫⚫
🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵

What do the circles mean? The number of circles represent the number of changed images.
🔴 : Changed items, ⚪ : New items, ⚫ : Deleted items, and 🔵 Passed items

How can I change the check status? If reviewers approve this PR, the reg context status will be green automatically.

@github-actions
Copy link
Contributor

github-actions bot commented Nov 6, 2020

Visit the preview URL for this PR (updated for commit 98184c0):

https://ameba-spindle--pr89-feature-form-compone-dwjafddy.web.app

(expires Fri, 25 Dec 2020 04:37:32 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

@herablog herablog force-pushed the feature/form-components branch 10 times, most recently from f94efd3 to 4df0215 Compare November 13, 2020 12:16
@hiloki hiloki self-requested a review November 16, 2020 01:33
@herablog herablog self-assigned this Nov 16, 2020
@herablog
Copy link
Member Author

herablog commented Nov 17, 2020

IEでの確認は #104 がマージされてからが良さそうです。

Copy link

@syasuda90 syasuda90 left a comment

Choose a reason for hiding this comment

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

いくつか気になった箇所コメントさせていただきました。

packages/spindle-ui/src/Form/DropDown.stories.mdx Outdated Show resolved Hide resolved
packages/spindle-ui/src/Form/DropDown.tsx Show resolved Hide resolved
packages/spindle-ui/src/Form/Radio.css Show resolved Hide resolved
packages/spindle-ui/src/Form/TextField.tsx Outdated Show resolved Hide resolved
@herablog herablog force-pushed the feature/form-components branch 2 times, most recently from 865f1ca to e5bede1 Compare November 18, 2020 11:23
Copy link
Contributor

@tatz012 tatz012 left a comment

Choose a reason for hiding this comment

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

五月雨ですみません:pray:

packages/spindle-ui/src/Form/DropDown.stories.mdx Outdated Show resolved Hide resolved
@herablog
Copy link
Member Author

現在のステータスを更新。「Done is better than perfect」の精神で出せるものから出していくか。
#89 (comment)

D7LAGdpU0AE1T1x

@herablog herablog changed the title [WIP] Form components Form components Nov 25, 2020
@herablog
Copy link
Member Author

ありがとうございます〜docのstatus更新と、コミットログの都合上コミットまとめてリリースしてみます。

@herablog herablog marked this pull request as ready for review November 25, 2020 04:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants