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

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

Open
uhooi opened this issue Jul 23, 2022 · 8 comments
Open

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

uhooi opened this issue Jul 23, 2022 · 8 comments
Labels
bug Something isn't working

Comments

@uhooi
Copy link
Member

uhooi commented Jul 23, 2022

不具合

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

- ソースを壊す変更は含まれていません
- 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: 7d5deb6

@uhooi uhooi added the bug Something isn't working label Jul 23, 2022
@uhooi
Copy link
Member Author

uhooi commented Jul 23, 2022

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

@uhooi
Copy link
Member Author

uhooi commented Jul 23, 2022

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

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

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

@nnabeyang
Copy link
Contributor

@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/)

@uhooi
Copy link
Member Author

uhooi commented Jul 23, 2022

@nnabeyang ありがとうございますー!
まさにその回避策をしていたのですが、Markdown は私以外の人も書くのでルールを増やしたくなく、↓の回避策を取り入れる方向でいこうと思います!
#83 (comment)

@shimastripe
Copy link

shimastripe commented Jul 23, 2022

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

@uhooi
Copy link
Member Author

uhooi commented Jul 23, 2022

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

@nnabeyang
Copy link
Contributor

nnabeyang commented Jul 23, 2022

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

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

@uhooi
Copy link
Member Author

uhooi commented Jul 23, 2022

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

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

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

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants