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

[MU4 Task] Improvements to "layout" palette #10451

Closed
bkunda opened this issue Feb 4, 2022 · 10 comments · Fixed by #12865
Closed

[MU4 Task] Improvements to "layout" palette #10451

bkunda opened this issue Feb 4, 2022 · 10 comments · Fixed by #12865
Assignees
Labels
P3 Priority: Low

Comments

@bkunda
Copy link

bkunda commented Feb 4, 2022

Task description
This task aims to improve the usability and visual aesthetic of the layout palette.

The key tasks are:

  1. Render all icons in the same font colour (Font_Primary_Colour)
  2. Implement scalability of icons generated from musescoreicon (FIXED ELSEWHERE)
  3. Replace icon "Insert one measure before selection" with new icon U+F432 (FIXED ELSEWHERE)
  4. Replace icon "Staff type change" with new icon U+F433
  5. Replace current tooltip for "Group measures" with "Keep measures together"

Screenshot 2022-02-05 at 12 31 42 am

Thanks!

@bkunda bkunda added the P2 Priority: Medium label Feb 4, 2022
@bkunda bkunda added this to Needs triage in [MU4.0 ALPHA2] via automation Feb 4, 2022
@duanjr
Copy link
Contributor

duanjr commented Mar 16, 2022

Looks this issue is not active.
I've done some job research.
My current palette looks like this:
image
One thing to notice is that "Staff type change" is not an anction, it needs more change to show U+F433 than just change the icon code, which may not worth it.(looks we only set icon for actions)

@HemantAntony
Copy link
Contributor

Looks this issue is not active. I've done some job research. My current palette looks like this: image One thing to notice is that "Staff type change" is not an anction, it needs more change to show U+F433 than just change the icon code, which may not worth it.(looks we only set icon for actions)

Yeah, I had tried to change that icon. I don't think its actually displayed as an icon. It seems like its being "drawn" or "painted" with lines and not as a pre-existing icon. I remember the code for that being in engraving folder in stafftypechange.cpp. Maybe the draw() function. Its not direct like the other icons

@duanjr
Copy link
Contributor

duanjr commented Mar 16, 2022

Looks this issue is not active. I've done some job research. My current palette looks like this: image One thing to notice is that "Staff type change" is not an anction, it needs more change to show U+F433 than just change the icon code, which may not worth it.(looks we only set icon for actions)

Yeah, I had tried to change that icon. I don't think its actually displayed as an icon. It seems like its being "drawn" or "painted" with lines and not as a pre-existing icon. I remember the code for that being in engraving folder in stafftypechange.cpp. Maybe the draw() function. Its not direct like the other icons

Yeah, and those 'break' and 'spacer' are neither icons. You remember right about the code, I rewrite draw() function, but disrupted part of the existing code structure about painting

@duanjr
Copy link
Contributor

duanjr commented Mar 16, 2022

I suggest not using U+F433 or we can "draw" a new icon with painter for consistency with existing code.(I can't find a palette cell which is not action but uses icons)

@HemantAntony
Copy link
Contributor

HemantAntony commented Mar 16, 2022

Yeah, and those 'break' and 'spacer' are neither icons. You remember right about the code, I rewrite draw() function, but disrupted part of the existing code structure about painting

I'm not really sure about this, but I think you shouldn't just change the "icon" by rewriting the draw() because unless I am mistaken that function gets called when you add a StaffTypeChange to the score.

@bkunda
Copy link
Author

bkunda commented Mar 16, 2022

@vpereverzev @igorkorsukov @Eism can one of you please weigh in on this when you get a minute? I seem to remember a discussion about leaving some of these as musescoreicon and making them scalable like the "drawn" icons. I don't have an opinion about the best way forward here, but I think you guys should perhaps advise what the best path is for this issue 🙂

@duanjr
Copy link
Contributor

duanjr commented Mar 16, 2022

I'm not really sure about this, but I think you shouldn't just change the "icon" by rewriting the draw() because unless I am mistaken that function gets called when you add a StaffTypeChange to the score.

Yes and it will be about only half the size as expected. And more change is needed to fix this.

@vpereverzev vpereverzev moved this from Needs triage to Low priority in [MU4.0 ALPHA2] Apr 8, 2022
@vpereverzev vpereverzev removed this from Low priority in [MU4.0 ALPHA2] Apr 8, 2022
@vpereverzev vpereverzev added this to To do in 4.x SHORTLIST via automation Apr 8, 2022
@bkunda bkunda added P3 Priority: Low P2 Priority: Medium and removed P2 Priority: Medium labels May 30, 2022
@bkunda bkunda removed this from To do in 4.x SHORTLIST May 30, 2022
@bkunda bkunda added this to Needs triage in [MU4.0 ALPHA2] via automation May 30, 2022
@RomanPudashkin RomanPudashkin removed this from Needs triage in [MU4.0 ALPHA2] Jun 6, 2022
@RomanPudashkin RomanPudashkin added this to Needs triage in [MU4.0 BETA1] via automation Jun 6, 2022
@RomanPudashkin RomanPudashkin added this to To do in [MU4.0 - PALETTES] via automation Jun 6, 2022
HemantAntony added a commit to HemantAntony/MuseScore that referenced this issue Aug 15, 2022
@HemantAntony
Copy link
Contributor

@bkunda I've managed to fix this after a couple of months :) Check #12865

HemantAntony added a commit to HemantAntony/MuseScore that referenced this issue Aug 15, 2022
HemantAntony added a commit to HemantAntony/MuseScore that referenced this issue Aug 15, 2022
HemantAntony added a commit to HemantAntony/MuseScore that referenced this issue Aug 19, 2022
HemantAntony added a commit to HemantAntony/MuseScore that referenced this issue Aug 19, 2022
HemantAntony added a commit to HemantAntony/MuseScore that referenced this issue Aug 20, 2022
HemantAntony added a commit to HemantAntony/MuseScore that referenced this issue Sep 5, 2022
@vpereverzev vpereverzev moved this from Needs triage to In progress in [MU4.0 BETA1] Sep 12, 2022
HemantAntony added a commit to HemantAntony/MuseScore that referenced this issue Sep 18, 2022
@Tantacrul Tantacrul removed this from In progress in [MU4.0 BETA1] Oct 18, 2022
@Tantacrul Tantacrul added this to To triage in [MU4.0 RELEASE] via automation Oct 18, 2022
@Tantacrul Tantacrul moved this from To triage to In progress in [MU4.0 RELEASE] Oct 18, 2022
@vpereverzev vpereverzev removed this from In progress in [MU4.0 RELEASE] Dec 1, 2022
@vpereverzev vpereverzev added this to To do in 4.x SHORTLIST via automation Dec 1, 2022
@jeetee
Copy link
Contributor

jeetee commented Dec 20, 2022

Addendum to this task by forum confusion reports.
It seems that many users expect the "keep together" element to work as a "force and squeeze and magic together" element rather than what it does.

I propose to have this, due to its more niche and specific use case, hidden behind the "more" button of the layout palette by default.

@oMrSmith
Copy link

oMrSmith commented Dec 20, 2022

Maybe so?

Example

(Inserterting a measure is not actually a layout thingy, so I removed it too, which makes everything look very symmetric.)

HemantAntony added a commit to HemantAntony/MuseScore that referenced this issue Mar 4, 2023
[MU4.0 - PALETTES] automation moved this from To do to Done May 9, 2023
4.x SHORTLIST automation moved this from To do to Done May 9, 2023
@oktophonie oktophonie removed this from Done in 4.x SHORTLIST Jul 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P3 Priority: Low
Projects
No open projects
Development

Successfully merging a pull request may close this issue.

7 participants