Navigation Menu

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

Use text area to show response #7

Merged
merged 2 commits into from Oct 25, 2014

Conversation

myokoym
Copy link
Member

@myokoym myokoym commented Oct 25, 2014

#6 の続きです。

alertをやめて、レスポンスを表示するためのtextareaを作成しました。
全部終わってから表示するのではなく、1行実行されるたびに追記する感じにしてあります。
レスポンスの区切りはJSONっぽくカンマにしましたが、見づらいかもしれません。

あと、ある程度横幅があるディスプレイではテキストエリアが横に並ぶようにしてみました。

http://getbootstrap.com/css/#grid-options

screenshot from 2014-10-25 16 39 48

狭い場合は縦に並びます。

screenshot from 2014-10-25 16 40 22

@kou
Copy link
Member

kou commented Oct 25, 2014

おぉ!これはすごい!

レスポンスの区切りはJSONっぽくカンマにしましたが、見づらいかもしれません。

単に改行区切りでよいと思います!
1つの大きなJSONではなく、n個のJSONの列なので。

kou added a commit that referenced this pull request Oct 25, 2014
Use text area to show response

Patch by Masafumi Yokoyama. Thanks!!!
@kou kou merged commit 0f1712c into groonga:master Oct 25, 2014
@kou
Copy link
Member

kou commented Oct 25, 2014

デプロイしました!

@kou
Copy link
Member

kou commented Oct 25, 2014

横山さんも直接GitHub/Herokuにpushできるようにしました!

@myokoym myokoym deleted the use-textarea-for-response branch October 25, 2014 08:39
@myokoym
Copy link
Member Author

myokoym commented Oct 25, 2014

マージとpush権ありがとうございます!

単に改行区切りでよいと思います!
1つの大きなJSONではなく、n個のJSONの列なので。

一瞬、複数のレスポンスを1つのJSONとして扱えたら便利な場面が
あるかもしれないなあと思ったのですが、なさそうな気がしてきました!

@kou
Copy link
Member

kou commented Oct 25, 2014

1つのJSONで扱いたいならさらに [...] でくるんでやらないといけないんですよねぇ。

@myokoym
Copy link
Member Author

myokoym commented Oct 25, 2014

あ、そうですね。たぶんコーディングしていたときは、[...]でくるむだけで
JSONになるならカンマもありか…と思っていたのですが、今考えると中途半端ですね。

実際に使ってみてどうだったかも聞いてみたいですが、
とりあえずカンマ区切りは止める方針でいきたいと思います。

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