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

geolonia をクラス属性に持つ div に高さが指定されていないときには警告を出したい #237

Open
champierre opened this issue Sep 14, 2021 · 6 comments

Comments

@champierre
Copy link
Contributor

https://docs.geolonia.com/embed-api/#html-%E3%81%AE%E8%A8%98%E8%BF%B0 には、

geolonia というクラス属性を持つ <div /> 要素を設置してください。この要素には、CSS 等で高さが指定されている必要があります。

とありますが、CSS で height を適用し忘れるのは、良くあるミスだと思います。(自分も一番最初やったし、「geolonia maps を使おう」のイベントのときにもそういうコメントを見ました)
height を適用し忘れているときにはダイアログなどで警告を出せないでしょうか?

@miya0001
Copy link
Member

たとえばクリックしたら地図を表示とかいろいろユースケースがあると思うんですよね。
うちの地図に限らず高さがゼロだと(CSSに限らずコンテンツがなくても)なにも表示されないのはうちの地図の問題というよりはHTMLの仕様だと思うので、僕たちが知らない未知のインターフェースとかも想定すると、これは仕方ないかなと思います。

@keichan34
Copy link
Member

デフォルトの高さを設定しようかなと思いました。どう思いますか?

@champierre
Copy link
Contributor Author

たとえば localhost でテストしているときだけアラートに出すとか、せめて console.log とかに表示するとか出来ないですかねえ。確かに geolonia maps だけの問題でなく、google maps を設置したときも同じ問題でしばらくはまった覚えがあるのですが、ならば geolonia maps では少しでもそれを軽減する他にはない工夫をすることで「かんたんに設置できる」という売りにつながるのではないかと思いました。

@miya0001
Copy link
Member

miya0001 commented Sep 16, 2021

これもう一つ他のフィードバックでもそうなんですけど、実際にサイトに組み込む場合に、高さが「適当でいい」というシチュエーションは、あまり想定できないので、実際のサイトに組み込むはこのミスもあまり起こらないのではないかと推測してます。

geolonia/app.geolonia.com#586

一方で、先日のイベントみたいなハンズオン形式でやる場合は、逆にこういうミスが多発するんだろうなというのも理解できます。
それならイベントのときに、「ここをきをつけてくださいね」で終わる話な気がするんですよね。

@champierre
Copy link
Contributor Author

確かにドキュメントの書き方の工夫で、ミスをいくらかは軽減できるような気もしています。

@miya0001
Copy link
Member

実はドキュメントにも書いてるんですけどね。まあ読まないですよねw

はじめての地図を設置___Geolonia_公式ドキュメント

https://docs.geolonia.com/tutorial/004/

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

No branches or pull requests

3 participants