Skip to content

feat: Three new mods#626

Merged
Mivik merged 18 commits intoTeamFlos:mainfrom
IcedDog:new-mods
Mar 20, 2026
Merged

feat: Three new mods#626
Mivik merged 18 commits intoTeamFlos:mainfrom
IcedDog:new-mods

Conversation

@IcedDog
Copy link
Copy Markdown
Contributor

@IcedDog IcedDog commented Dec 4, 2025

Added three new mods.

Copy link
Copy Markdown
Member

@liquidhelium liquidhelium left a comment

Choose a reason for hiding this comment

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

需要改一下高速模式的代码,其他看着没问题
(如果说其在设计上就是为了强制玩家使用1.5速,那这个名字不太合适也许?因为对于2速玩家反而是低速)

Copy link
Copy Markdown
Contributor

@Mivik Mivik left a comment

Choose a reason for hiding this comment

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

Run cargo fmt plz

@IcedDog
Copy link
Copy Markdown
Contributor Author

IcedDog commented Dec 11, 2025

需要改一下高速模式的代码,其他看着没问题 (如果说其在设计上就是为了强制玩家使用1.5速,那这个名字不太合适也许?因为对于2速玩家反而是低速)

我改成了在原来速度的基础上乘1.5。但是我是希望设置和模组是区分开来的,后面结算界面上应当正常显示设置的速度,并在(将来可能有的)开启的模组列表显示里面显示开启了Nightcore模组。

@Mivik
Copy link
Copy Markdown
Contributor

Mivik commented Mar 20, 2026

LGTM, thanks for your contribution!

@Mivik Mivik dismissed liquidhelium’s stale review March 20, 2026 03:59

Already resolved

@Mivik Mivik merged commit 6fdd61a into TeamFlos:main Mar 20, 2026
3 of 5 checks passed
@YuevUwU
Copy link
Copy Markdown
Contributor

YuevUwU commented Apr 5, 2026

Behavior Changes

  • Add mod icons, displaying on ending scene
  • "Fade Out" mod is now really "Fade" (by changing alpha) instead of hiding immediately (Still apply alpha to the whole note)
  • (And Later...) "Fade Out" mod effects no longer apply on fake note (#720)

New

  • Add 3 mods:
    • Fade In: (mutually exclusive with "Fade Out" mod)
    • Nightcore: Speed of chart and music will be 1.5 * [Settings > Chart > Speed] (with increasing pitch as usual)
    • Rainbow: Note and Judgeline will have hue changes

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants