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

[0441/adjustment-layout] Adjustment周りの見た目部分の調整 #1099

Merged
merged 2 commits into from Sep 4, 2021

Conversation

cwtickle
Copy link
Owner

@cwtickle cwtickle commented Sep 4, 2021

🔨 変更内容 / Details of Changes

  1. Adjustmentの表示について、フレーム数であることがわかるように「f」を付加しました。
    Adjustmentを0.1単位で設定できる機能の追加 #1096 関連の修正となります。
  2. Speed, Adjustmentのオンマウス説明文を見直しました。
    内側、外側のボタン操作の説明を追加しています。

🔖 関連Issue, 変更理由 / Related Issues, Reason for Changes

  1. 過去バージョンとの仕様差異が目でわかるようにするのと、
    数字だけでは単位がわからない(フレームか、ミリ秒か)ため。
  2. 外側、内側のボタンの調整幅についての記載が無かったため。

📷 スクリーンショット / Screenshot

📝 その他コメント / Other Comments

@suzme
Copy link
Contributor

suzme commented Sep 4, 2021

ラベル部分は対応忘れてましたが、setSettingの引数を内側ボタンと同じようにすれば0.5f刻みになります。

// 4924行目~
multiAppend(_obj,
	makeSettingLblCssButton(linkId, `${initName}${g_localStorage[_settingName] === g_stateObj[_settingName] ? ' *' : ''}`, 0,
		_ => setSetting(skipTerm2 > 0 ? skipTerm : 1, _settingName, unitName, roundNum),
		{ cxtFunc: _ => setSetting(skipTerm2 > 0 ? skipTerm * (-1) : -1, _settingName, unitName, roundNum) }),

@cwtickle
Copy link
Owner Author

cwtickle commented Sep 4, 2021

ありがとうございます m(_ _)m
仕組みで直せることは知っていますが、あえてそのままでもいいのかなと思っています。

Adjustmentはプラスマイナスを調整しながら行うため、ラベルボタンを触るケースはそんなに無さそうなのと、
間違って押してもリカバリーできる方法があるのでそこまで問題にならなさそうな気はします。
また、隠し機能とはいえ見た目に影響しない範囲で使えるなら使った方が良いかなと。

あと、前のPRでは特に触れなかったですが、
この方法だと譜面ヘッダー側(blankFrame, adjustment)も小数使えるんですね。

@codeclimate
Copy link

codeclimate bot commented Sep 4, 2021

Code Climate has analyzed commit 7df24dc and detected 0 issues on this pull request.

View more on Code Climate.

@cwtickle cwtickle merged commit 1c38e56 into develop Sep 4, 2021
@cwtickle cwtickle deleted the feature/0441-adjustment-layout branch September 5, 2021 22:16
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.

None yet

2 participants