-
Notifications
You must be signed in to change notification settings - Fork 10
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
Update kage-editor #15
base: master
Are you sure you want to change the base?
Conversation
@graphemecluster 私が取れる時間の関係で、内容の確認に全量では数週間〜数ヶ月程度かかる可能性があります。そのため、管理上の都合で申し訳ありませんが、このPRを機能ごとに小さく分割して、優先度の高いものから順に確認して少しずつ取り込んでいく形にしたいと思っています。 特に問題がなければ、分割は私の方で行なおうと思っています。その際に、 graphemecluster さんの authorship は保つようにいたします。 最後になりますが、このPRを contribute していただいていることに感謝します。ありがとうございます! |
すごく古いプルリクなのにわざわざご対応いただきありがとうございます。 |
こんなにも早く分割してくださりありがとうございます!とりあえず一通り目を通していくつかのコメントをつけました。修正してほしいところがあれば教えてください。 |
変更履歴
height: 100vh
はスクリーンの高さとは限らないので代わりにposition: absolute
を使用crosshair
にするmove
にする新機能のリクエスト
英語(ざっくり翻訳)
height: 100vh
is not limited to the screen, soposition: absolute
is used insteadcrosshair
when the background is draggedmove
while moving