デフォルトスキンのAAポップアップ表示に関する不具合修正 #353
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
1b42bac の修正が実際には効いていませんでした 通常のレスとポップアップ両方に適用するようにして、記述自体を適切な位置へ移動しました
83d3640 をチェックアウトして出力されるhtmlを調べたところ、aaRes は popupInner の 直子要素ではありませんでした。その時から正しく表示できていなかったものと思われます。
AAレスポップアップの表示を white-space:pre にしたことで、 縦スクロールバーが現れる時はいつも横スクロールバーも一緒に現れるようになった。 縦スクロールバーが出るとその分の横幅が狭まって内容が収まりきらなくなり 横スクロールバーも出るということなので、縦スクロールバーが現れる領域に内容領域が かからないように、スクロールバーの幅相当分として padding-right:18px に固定した。 実測では Windows で 17px 、GNOME で 14px だったが、これでOKではないかと。
AAが折り返しで崩れて表示されるよりは、横スクロールバーが現れる方が望ましい動作でしょう
巨大なAAをポップアップで表示したとき、表示領域がちょっと窮屈な感じがしたので、 少しだけ拡げてみました。もっと広くても良いかもしれない。
|
普段 AA が多く投稿されるスレをあまり見ないもので，修正助かります． |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
2ch.netのchaikaスレに、デフォルトスキンのAAポップアップ表示に関する不具合修正パッチが
上げられていたので、それを元にさらに自分で改良したものです。
ひととおりテスト済みです。どれも細かな修正ですが、どんなものでしょうか。