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: [0581] カラーセットのキー別一括保存対応 #1309

Merged
merged 5 commits into from Oct 1, 2022

Conversation

cwtickle
Copy link
Owner

@cwtickle cwtickle commented Oct 1, 2022

🔨 変更内容 / Details of Changes

  1. これまで画面を開いているときだけ有効になっていた、カラーセットをキー別に丸ごと保存するようにしました。
    Type1~4を選択している場合に有効で、保存後はTypeSという表記に変わります。
    Default/Type0についてはこれまで通りです。
    なお、ColorTypeの設定はこれまで通り作品別に保存されます。

  1. カラーグループ、シャッフルグループの保存に対応しました。
    • カラーグループ、シャッフルグループが保存された場合は「S」と表記されます。

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

  1. これまでは必要時に毎回再設定が必要になっていたため。
  2. 1.と同じ理由です。

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

📝 その他コメント / Other Comments

  • 別キーに切替時、ColorType = TypeSが存在しない場合は ColorType = Default に切り替わります。

js/danoni_main.js Outdated Show resolved Hide resolved
js/danoni_main.js Outdated Show resolved Hide resolved
if (g_keyObj[`${_type}${keyCtrlPtn}_1`] !== undefined) {
document.getElementById(`lnk${toCapitalize(_type)}Group`).textContent =
getStgDetailName(`${g_keycons[`${_type}GroupNum`] + 1}`);
const num = g_keycons[`${_type}GroupNum`] === -1 ? g_keycons.groupSelf : g_keycons[`${_type}GroupNum`] + 1;
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Identical blocks of code found in 2 locations. Consider refactoring.

const typeName = toCapitalize(_type);
const num = g_keycons[`${_type}GroupNum`] === -1 ? g_keycons.groupSelf : g_keycons[`${_type}GroupNum`] + 1;
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Identical blocks of code found in 2 locations. Consider refactoring.

js/danoni_main.js Outdated Show resolved Hide resolved
js/danoni_main.js Outdated Show resolved Hide resolved
@cwtickle cwtickle marked this pull request as ready for review October 1, 2022 15:00
@codeclimate
Copy link

codeclimate bot commented Oct 1, 2022

Code Climate has analyzed commit bbc3616 and detected 2 issues on this pull request.

Here's the issue category breakdown:

Category Count
Duplication 2

View more on Code Climate.

@cwtickle cwtickle merged commit c8fbac1 into develop Oct 1, 2022
@cwtickle cwtickle deleted the feature/0581-store-colorlist branch October 2, 2022 05:25
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

1 participant