Skip to content

見出しに anchor を付けます#159

Merged
kmyk merged 3 commits intogh-pagesfrom
midashi-anchor
Apr 5, 2021
Merged

見出しに anchor を付けます#159
kmyk merged 3 commits intogh-pagesfrom
midashi-anchor

Conversation

@kmyk
Copy link
Copy Markdown
Collaborator

@kmyk kmyk commented Apr 5, 2021

ページ本体がないアルゴリズムについてもリンクを張って言及しやすくなるのでうれしい

プレビュー: https://kmyk.github.io/algorithm-encyclopedia-staging/#noredirect

ページ本体がないアルゴリズムについてもリンクを張って言及しやすくなるのでうれしい
@kmyk
Copy link
Copy Markdown
Collaborator Author

kmyk commented Apr 5, 2021

ないよりはある方がいいと思うんだけど、目が滑る感じがする。マウスオーバー時のみ表示とかにすべきかな

常に表示されてると目が滑る感じがするので
@kmyk
Copy link
Copy Markdown
Collaborator Author

kmyk commented Apr 5, 2021

マウスカーソルを見出しの上に乗せたときのみ表示するようにしました。
@noshi91 @MiSawa プレビューを見て問題なさそうならマージお願いします。

@MiSawa
Copy link
Copy Markdown
Collaborator

MiSawa commented Apr 5, 2021

draft-link-external に左からマウスを近づけたときに逃げるのがちょっとアレなので、none じゃなくて hidden のほうが良いような気もしなくはないです。

@kmyk
Copy link
Copy Markdown
Collaborator Author

kmyk commented Apr 5, 2021

draft-link-external に左からマウスを近づけたときに逃げるのがちょっとアレなので、none じゃなくて hidden のほうが良いような気もしなくはないです。

これ分かって、しかし単純に visiblity: hidden だと外部リンクのアイコンとの間に不自然な空白が空いてなんとなく嫌だったんですよね。アンカーのアイコンを外部リンクのアイコンの右に表示するようにしてしてみましたが、どうでしょうか?

@MiSawa
Copy link
Copy Markdown
Collaborator

MiSawa commented Apr 5, 2021

draft-link-external に左からマウスを近づけたときに逃げるのがちょっとアレなので、none じゃなくて hidden のほうが良いような気もしなくはないです。

これ分かって、しかし単純に visiblity: hidden だと外部リンクのアイコンとの間に不自然な空白が空いてなんとなく嫌だったんですよね。アンカーのアイコンを外部リンクのアイコンの右に表示するようにしてしてみましたが、どうでしょうか?

よいと思います!!

@kmyk kmyk merged commit dc9186c into gh-pages Apr 5, 2021
@kmyk kmyk deleted the midashi-anchor branch April 5, 2021 14:39
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