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

Update kage-editor #15

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

Conversation

graphemecluster
Copy link
Collaborator

@graphemecluster graphemecluster commented Apr 14, 2021

変更履歴

  • height: 100vh はスクリーンの高さとは限らないので代わりに position: absolute を使用
  • 選択の際カーソルを crosshair にする
  • 選択長方形に残りの3つの制御点を追加
  • 移動のときカーソルを move にする
  • 選択している筆画の中心線のみ表示するオプションを追加
  • 幅が足りないところ(2ヶ所)を広げる
  • 設定画面のUIを改良
  • 曲線の制御点の間に補助線を表示
  • 翻訳の更新
  • 他、細かいところの改善

新機能のリクエスト

  • IDSの検索に対して一枚目のグリフを非表示にする
  • Shift+リサイズで縦横比を維持
  • Shift+移動で方向を一軸に限る
  • Ctrl+移動でグリフを複写
  • Alt+ドラッグで線や制御点にスナップする
  • マウスの右ボタンでドラッグではなく他の操作を行う
  • 検索エリアからグリフをクリックする他にドラッグ&ドロップで追加
  • Issuesの通り、タッチスクリーンもサポートする

英語(ざっくり翻訳)

  • height: 100vh is not limited to the screen, so position: absolute is used instead
  • Make the cursor crosshair when the background is dragged
  • Add the remaining 3 control points to the selection rectangle
  • Make the cursor move while moving
  • Add an option to show only the center line of the current selected stroke
  • Widen two places
  • Improve UI of the option pop-up
  • Show auxiliary lines between curve control points
  • Update translation
  • Other small improvements

@kurgm kurgm self-requested a review April 23, 2021 04:39
@kurgm kurgm self-assigned this Apr 23, 2021
@graphemecluster graphemecluster marked this pull request as draft March 4, 2022 08:27
@kurgm
Copy link
Owner

kurgm commented Sep 10, 2023

@graphemecluster
PRをずっと放置していて申し訳ありません。
長い時間が空いてしまったのですが、今後、週末などに積極的に時間を取ってこのPRの内容を取り込みたいと思っています。

私が取れる時間の関係で、内容の確認に全量では数週間〜数ヶ月程度かかる可能性があります。そのため、管理上の都合で申し訳ありませんが、このPRを機能ごとに小さく分割して、優先度の高いものから順に確認して少しずつ取り込んでいく形にしたいと思っています。

特に問題がなければ、分割は私の方で行なおうと思っています。その際に、 graphemecluster さんの authorship は保つようにいたします。

最後になりますが、このPRを contribute していただいていることに感謝します。ありがとうございます!

@graphemecluster
Copy link
Collaborator Author

すごく古いプルリクなのにわざわざご対応いただきありがとうございます。
これは当時 PR は小さく分割すべきという業界の暗黙の礼儀も知らず IDE なしで弄っていたものなのでレビューの際大目に見ていただければ幸いです m(_ _)m

@kurgm
Copy link
Owner

kurgm commented Sep 18, 2023

コミットを分割し,PRも分割しました。

@graphemecluster
Collaboratorとして招待していますので、もし必要であれば上記PRのブランチで内容を修正などしていただいても構いません 🙇

また,記載いただいた新機能のリクエストも,(まだissueになっていないものは)個別のissueを立てました。

@graphemecluster
Copy link
Collaborator Author

こんなにも早く分割してくださりありがとうございます!とりあえず一通り目を通していくつかのコメントをつけました。修正してほしいところがあれば教えてください。

@kurgm kurgm removed their request for review September 23, 2023 03:05
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