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

[ENCHANCEMENT] Additional Functionality for Playbars (Assets) #132

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

Trofem
Copy link

@Trofem Trofem commented Feb 26, 2025

This PR changed playbar.xml file. Now it describes changed logic of playbars, accordingly to #4229 Funkin PR changes.

  • Difficulty:
    image 1

  • BPM:
    image 2

  • Note Snap
    image 3

  • Song Remaining Time:
    image 4

For playbarDifficulty and playbarBPM added /close description.
Closing/openning ability added to all other buttons in this order:
playbarNoteSnap - Ctrl+Click -> Note Window.
playbarSongRemaining - Click -> Offsets Window
I think this is optimal!
tooltip="Song Remaining Time/Left Click to open/close Offsets Window"
\n added and etc
i hope this is final change for now.
@Hundrec Hundrec added chart editor PR is related to the operation of the Chart Editor. status: pending triage The bug or PR has not been reviewed yet. size: small A small pull request with 10 or fewer changes labels Feb 26, 2025
@AbnormalPoof AbnormalPoof added type: enhancement Provides an enhancement or new feature. status: pending pull request Has a PR which resolves it that is pending review. and removed status: pending triage The bug or PR has not been reviewed yet. labels Feb 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chart editor PR is related to the operation of the Chart Editor. size: small A small pull request with 10 or fewer changes status: pending pull request Has a PR which resolves it that is pending review. type: enhancement Provides an enhancement or new feature.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants