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:ダイス固定・固定解除の追加 #67 #71

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

Conversation

NktAts
Copy link
Contributor

@NktAts NktAts commented Jun 26, 2019

概要

ダイスのメニューから、ダイスの出目を固定・固定解除する機能を追加しました。

詳細

  1. ダイスのプロパティに「isLock」を追加(デフォルト:false)
  2. isLock=falseの時はメニューに「ダイス目を固定」が表示され、選択するとisLock=trueに更新
  3. isLock=trueの間は、ダブルクリック・「ダイスを振る」・「ダイス目を設定」を実行できなくする
  4. isLock=trueの時はメニューに「ダイス目を固定解除」が表示され、選択するとisLock=falseに更新

懸念点

  • ダイスについて #67 に記載のうち、ダブルクリックでダイス目が変わってしまう問題に対処しました。メッセージに関しては非対応です。

  • 見た目の変更は現時点ではありません。(マップマスクと同じく、ダイスの上に鍵マークをつけようと思いましたが、うまくダイスにかぶるよう配置ができませんでした)

  • ダイス目を固定した状態でダイスのメニューを開いたときに、罫線が2つ連続で表示されます。

その他

プログラミングやGithubの利用に慣れていないため、不自然な箇所あるかもしれません。
その際はコメントいただけると助かります。

@NktAts
Copy link
Contributor Author

NktAts commented Jun 26, 2019

プルリクエスト送信後に気づきましたが、
Issue #67 投稿者はシノビガミのプロットで使用することを想定しているようなので、
「ダイス目を設定」はロック中でも実行できたほうがよいかもしれません。
どういう仕様にするかはお任せします。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants