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

Update theme for smartphone #76

Merged
merged 6 commits into from
Nov 20, 2019
Merged

Conversation

hanachin
Copy link
Member

動機

SEO

Rubyでメソッドの使い方を検索したとき、公式ではない公式風APIドキュメントがしれっと公式よりも上に出てくる問題をなんとかしたい。
いや、サイトを運営している人が必ずしも悪いわけじゃないけど、Ruby 1.9時代の情報っぽいので、初心者さん殺しの罠になりかねないんだよね・・・。

— Junichi Ito (伊藤淳一) (@jnchito) November 15, 2019

https://docs.ruby-lang.org/ja/latest/doc/index.html をChrome拡張のLighthouseでテストしました。
https://chrome.google.com/webstore/detail/lighthouse/blipmdconlkpinefehnmjammfjpmpbjk?hl=ja

SEOについては以下のような感じでした

blob_chrome-extension___blipmdconlkpinefehnmjammfjpmpbjk_7934792a-1d48-4331-b9a2-b490049a9ede
Lighthouse Report.pdf

改善したこと

viewportのmetaタグを追加しました

Many search engines rank pages based on how mobile-friendly they are. Without a viewport meta tag, mobile devices render pages at typical desktop screen widths and then scale the pages down, making them difficult to read.
https://web.dev/viewport/

コードがはみ出たときにスクロールバーを表示するようにしました

viewportの設定をしてはみ出すようになったので、スクロールバーを出しました

ライブラリ等のページの一覧のテーブル表示をやめて継承に基づいたインデントをなくした

テーブル表示はモバイルだと横にはみ出て読みづらくなるのでやめてみました。
テーブル表示をやめると継承に基づいたインデントの表示の意味がなくなるのでやめました。

リンクがはみ出るのを直しました

viewportの設定をしてはみ出すようになったので、適切な位置で改行するようにしました

結果

localhost_9292_latest_library__builtin html(iPhone X)
localhost_9292_latest_class_BasicObject html(iPhone X)

@hanachin hanachin marked this pull request as ready for review November 17, 2019 16:37
@elim
Copy link

elim commented Nov 17, 2019

おお。 スクリーンショットを見た感じ、 Issue #55 も解決していそうですね(変更内容も #55 での提案と同様っぽい)

@hanachin
Copy link
Member Author

ありがとうございます! ( #55 知らなかった 👍 )、解決されてますね 💯

@znz znz merged commit 700776c into rurema:master Nov 20, 2019
@znz
Copy link
Member

znz commented Nov 20, 2019

ありがとうございます。
入れてしまって、また何かあれば別途ということで。

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.

3 participants