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

refactor: [0684] 設定系のサブ関数をグローバル関数に変更 #1466

Merged
merged 3 commits into from Apr 25, 2023

Conversation

cwtickle
Copy link
Owner

@cwtickle cwtickle commented Apr 23, 2023

🔨 変更内容 / Details of Changes

  1. 設定系のサブ関数をグローバル関数に変更しました。
    対象は次の通りです。
    • resetDifWindow ()
    • makeDifList (_difList, _difCover, _targetKey)
    • nextDifficulty (_scrollNum)
    • makeDifBtn (_scrollNum)
    • createDifWindow (_key)
    • changeDifficulty (_num)
    • getScoreBaseData(_scoreId)
    • drawSpeedGraph (_scoreId)
    • drawDensityGraph (_scoreId)
    • drawBaseLine (_context, _resolution)
    • drawLine (_context, _y, _lineType, _fixed)
    • updateScoreDetailLabel(_name, _label, _value, _pos, _labelname)
    • makeDifInfoLabels (_scoreId)
    • makeDifInfo (_scoreId)
    • setReverse (_btn)
    • setReverseView (_btn)
    • setGauge (_scrollNum)
    • gaugeChange (_gaugeNum)
    • setLife (_baseProperty, _setProperty, _magnification)
    • setLifeCategory (_baseObj, { _magInit = 1, _magRcv = 1, _magDmg = 1 }
    • changeLifeMode (_baseObj)
    • gaugeFormat (_mode, _border, _rcv, _dmg, _init, _lifeValFlg)
    • setDifficulty (_initFlg)
  • なお、対象外とした関数は次の通りです。
    • createScoreDetail (_name, _graphUseFlg)
    • setScoreDetail ()

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

  1. ほとんどが上位のcreateOptionWindowに非依存の関数にも関わらず、
    外部から制御したいときに現状は使用できない状況のため。

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

📝 その他コメント / Other Comments

  • 他の画面で使用する場合は、DifficultyやAutoPlayなど対応する名前のdiv要素が必要です。
    主に外部から制御することを目的としています。

@codeclimate
Copy link

codeclimate bot commented Apr 23, 2023

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

View more on Code Climate.

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