website icon indicating copy to clipboard operation
website copied to clipboard

箇条書きでインデントを下げると元に戻らない

Open uhooi opened this issue 3 years ago • 8 comments

不具合

箇条書きでインデントを下げると、インデントを戻しても下がったままになってしまう。

- ソースを壊す変更は含まれていません
- Xcode の最小サポートバージョンが 13.3 に上がりました
- 各プラットフォームの最小サポートバージョンが上がりました
  - iOS: 13.0
  - macOS: 10.15
  - watchOS: 6.0
  - tvOS: 13.0
- Swift Structured Concurrency を採用しました
  - 従来の completion ベースの API と同等のフル機能を備えています
- [NukeUI](https://github.com/kean/NukeUI) がメインリポジトリの一部になりました
- ドキュメントが DocC で書き直され、GitHub へホスティングしました
  - Nuke: [https://kean-docs.github.io/nuke/documentation/nuke/](https://kean-docs.github.io/nuke/documentation/nuke/)
  - NukeUI: [https://kean-docs.github.io/nukeui/documentation/nukeui/](https://kean-docs.github.io/nukeui/documentation/nukeui/)
  - NukeExtensions: [https://kean-docs.github.io/nukeextensions/documentation/nukeextensions/](https://kean-docs.github.io/nukeextensions/documentation/nukeextensions/)

スクリーンショット
スクリーンショット 2022-07-23 11 32 10

原因

おそらく私が書いた CSS が悪いと思われる。

.content > ul {
    margin-top: 1.5em;
    margin-bottom: 1.5em;
}

.content ul {
    margin-left: 1.5em;
}

.content li {
    line-height: 1.5em;
}

ref: https://github.com/ios-osushi/website/pull/44/commits/7d5deb641e54478d9437b1e02755dbcc1c2819df

uhooi avatar Jul 23 '22 02:07 uhooi

CSS でなく HTML に問題があることがわかった。 ref: https://twitter.com/nnabeyang/status/1550681500181762048

uhooi avatar Jul 23 '22 03:07 uhooi

Publish を修正しないと、 ul がネストしているのを直せない。 CSS で見た目は回避できるので、こちらを適用したい。

.content ul ul {
    margin-left: 0;
}

ref: https://twitter.com/murokaco/status/1550683974330380288

uhooi avatar Jul 23 '22 03:07 uhooi

@uhooi 対処方法としては、改行を足すだと思います。上にマージンができますが、これはCSSで直せるような気がしますね。

- ソースを壊す変更は含まれていません
- Xcode の最小サポートバージョンが 13.3 に上がりました
- 各プラットフォームの最小サポートバージョンが上がりました
  - iOS: 13.0
  - macOS: 10.15
  - watchOS: 6.0
  - tvOS: 13.0

- Swift Structured Concurrency を採用しました
  - 従来の completion ベースの API と同等のフル機能を備えています

- [NukeUI](https://github.com/kean/NukeUI) がメインリポジトリの一部になりました
- ドキュメントが DocC で書き直され、GitHub へホスティングしました
  - Nuke: [https://kean-docs.github.io/nuke/documentation/nuke/](https://kean-docs.github.io/nuke/documentation/nuke/)
  - NukeUI: [https://kean-docs.github.io/nukeui/documentation/nukeui/](https://kean-docs.github.io/nukeui/documentation/nukeui/)
  - NukeExtensions: [https://kean-docs.github.io/nukeextensions/documentation/nukeextensions/](https://kean-docs.github.io/nukeextensions/documentation/nukeextensions/)

nnabeyang avatar Jul 23 '22 03:07 nnabeyang

@nnabeyang ありがとうございますー! まさにその回避策をしていたのですが、Markdown は私以外の人も書くのでルールを増やしたくなく、↓の回避策を取り入れる方向でいこうと思います! https://github.com/ios-osushi/website/issues/83#issuecomment-1193050116

uhooi avatar Jul 23 '22 03:07 uhooi

Publishが依存しているInk(Markdown Parser ライブラリ)側で上げられているこちらのPRが入ると https://github.com/JohnSundell/Ink/pull/71 HTML構造を直したい本質の問題は解決しそうです。

shimastripe avatar Jul 23 '22 03:07 shimastripe

@shimastripe おおお…ありがとうございます! すでに PR があったのですね。 そして Markdown パーサーは別リポジトリでしたね、、失念していました、、、

uhooi avatar Jul 23 '22 03:07 uhooi

@uhooi 改行足して回避しても、ulが分かれてしまうのでバグですね。 @shimastripe PRすでにあったんですね。まだ今年の3月だから新しい...

いや、コミット自体は2年前か。取り込まれそうにないですね。

nnabeyang avatar Jul 23 '22 03:07 nnabeyang

Publish を修正しないと、 ul がネストしているのを直せない。 CSS で見た目は回避できるので、こちらを適用したい。

.content ul ul {
    margin-left: 0;
}

ref: https://twitter.com/murokaco/status/1550683974330380288

この回避策が効かなかった。 箇条書きの間に空白行を入れて回避することにする。

uhooi avatar Jul 23 '22 06:07 uhooi