karaoke icon indicating copy to clipboard operation
karaoke copied to clipboard

Remove `BeatmapEditorRoundedScreen`

Open Joehuu opened this issue 1 year ago • 4 comments

  • Resolves https://github.com/karaoke-dev/karaoke/issues/1341

Joehuu avatar Aug 27 '23 00:08 Joehuu

image

I guess i should apprvoe this PR until official editor is more complete. Because some of the editor looks weird if display in the full-screen. It's hard for me to point-out the reason and find a way to adjust that 😢

andy840119 avatar Aug 27 '23 01:08 andy840119

And if you are interested in contributing to this project, i can filter the issue and pick-up the suitable one for you(because some of the issues is a memo for me and might be hard to see the priority)

andy840119 avatar Aug 27 '23 01:08 andy840119

Because some of the editor looks weird if display in the full-screen. It's hard for me to point-out the reason and find a way to adjust that 😢

The change is to mostly match the current design state of the official editor. I do agree that some screens look weird because they are mostly blank / have no content yet (even in the official editor), and making it fullscreen (and hiding the beatmap background) doesn't make it better.

Can wait, it's just a cosmetic change.

And if you are interested in contributing to this project, i can filter the issue and pick-up the suitable one for you(because some of the issues is a memo for me and might be hard to see the priority)

I just want to clean up the issues right now by targeting the easier ones first (even if it's low priority). That would be code quality, design, bug/crashes, etc.

I don't want to be recommended an issue up front, but I would appreciate priority:1|2|3 or similar labels like osu! so I can get a sense of what the current goals are and contribute to something bigger when I get the chance.

Joehuu avatar Aug 27 '23 03:08 Joehuu

I don't want to be recommended an issue up front, but I would appreciate priority:1|2|3 or similar labels like osu! so I can get a sense of what the current goals are and contribute to something bigger when I get the chance.

I'm not sure using priority flag is a good idea because:

  1. the main contributor for this project is only me ;_;
  2. this project does not have much users right now(means there's no s1 issue), so priority is decided by what i'm doing right now and might be changed frequently.

I guess using https://github.com/karaoke-dev/karaoke/labels/Good%20for%20contributor flag might be better right now, you can pick-up the issue with this label, and feel free to ask me if have any question.

andy840119 avatar Aug 27 '23 15:08 andy840119