kage-editor icon indicating copy to clipboard operation
kage-editor copied to clipboard

Update kage-editor

Open graphemecluster opened this issue 4 years ago • 4 comments

変更履歴

  • 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

graphemecluster avatar Apr 14 '21 07:04 graphemecluster

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

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

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

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

kurgm avatar Sep 10 '23 11:09 kurgm

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

graphemecluster avatar Sep 10 '23 14:09 graphemecluster

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

  • #40
  • #41
  • #42
  • #43
  • #44
  • #45
  • #46
  • #47
  • #48
  • #49
  • #50
  • #51
    • #52
  • #53
  • #54

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

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

  • #33
  • #34
  • #35
  • #36
  • #37
  • #38
  • #39

kurgm avatar Sep 18 '23 02:09 kurgm

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

graphemecluster avatar Sep 19 '23 01:09 graphemecluster