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

feat: エディター -> プレビューへの片方向スクロール同期 #115

Merged
merged 2 commits into from
Aug 21, 2024

Conversation

senkenn
Copy link
Contributor

@senkenn senkenn commented Aug 7, 2024

📑 Summary

  • エディター -> プレビューのスクロール同期を実装しました。
  • 以下が VS Code の標準の Markdown Preview のスクロール同期機能と異なる点です。
    今回はこのスクロール同期の最初の実装ということもあり、実装はなるべくシンプルにしました。
観点 VS Code このPR
スクロール同期の方向 エディター <-> プレビューの双方向 エディター -> プレビューの片方向のみ
ソース行のオフセットなど微調整 あり なし
スクロール同期する・しないの設定 あり なし(常に同期)
Screencast.from.08-08-2024.12.33.16.AM.webm

Related #1

📋 Tasks

  • 📖 Contribution Guide を読んだ
  • 👩‍💻 canary ブランチに対するプルリクエストである
  • 実行して正しく動作しているか確認する
  • 不要なコードが含まれていないか( コメントやログの消し忘れに注意 )
  • XSS になるようなコードが含まれていないか
  • Pull Reuqest の内容は妥当か( 膨らみすぎてないか )

@senkenn senkenn changed the title Senkenn/issue#1 エディター -> プレビューへの片方向スクロール同期 Aug 7, 2024
@senkenn senkenn changed the title エディター -> プレビューへの片方向スクロール同期 [WIP] エディター -> プレビューへの片方向スクロール同期 Aug 7, 2024
@cm-igarashi-ryosuke cm-igarashi-ryosuke self-requested a review August 14, 2024 00:14
Copy link
Member

@cm-igarashi-ryosuke cm-igarashi-ryosuke left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PR確認しました!LGTMです!!!

正直、何も知識がない状態だったのですべてが学びでした 🙏
ソースマップ追加後に動作の最終チェックをさせてください。

@cm-igarashi-ryosuke
Copy link
Member

@senkenn お待たせいたしました。

zenn-dev/zenn-editor#504 をマージし、バージョン 0.1.155 としてリリースしました。

package.jsonのdependenciesのzenn-*パッケージのバージョンを0.1.155に更新をお願いいたします!

@senkenn senkenn marked this pull request as ready for review August 20, 2024 11:22
@senkenn
Copy link
Contributor Author

senkenn commented Aug 20, 2024

@cm-igarashi-ryosuke ありがとうございます!更新いたしました。一応再度動作確認して期待通りの挙動であることを確認しました。

@senkenn senkenn changed the title [WIP] エディター -> プレビューへの片方向スクロール同期 エディター -> プレビューへの片方向スクロール同期 Aug 20, 2024
@senkenn senkenn changed the title エディター -> プレビューへの片方向スクロール同期 feat: エディター -> プレビューへの片方向スクロール同期 Aug 20, 2024
@cm-igarashi-ryosuke
Copy link
Member

cm-igarashi-ryosuke commented Aug 21, 2024

@senkenn ご対応ありがとうございました!LGTMですのでマージします!

リリースまでもう少しお待ち下さい。
リリース案内にはContributorとしてぜひお名前を掲載させてください。ありがとうございました!

@cm-igarashi-ryosuke cm-igarashi-ryosuke merged commit fdce937 into zenn-dev:canary Aug 21, 2024
1 check passed
@senkenn senkenn deleted the senkenn/issue#1 branch August 21, 2024 03:50
@cm-igarashi-ryosuke
Copy link
Member

@senkenn お待たせいたしました。

バージョン0.2.0で機能をリリースいたしました! 🎉

また、リリース案内にもお名前を掲載させていただきました!
https://info.zenn.dev/2024-08-22-vscode-ext-supports-scroll-sync

多大なる貢献をありがとうございました!

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.

2 participants