MuseScore icon indicating copy to clipboard operation
MuseScore copied to clipboard

[MU4 Task] Improvements to "layout" palette

Open bkunda opened this issue 3 years ago • 10 comments
trafficstars

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 avatar Feb 04 '22 13:02 bkunda

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)

duanjr avatar Mar 16 '22 04:03 duanjr

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

HemantAntony avatar Mar 16 '22 04:03 HemantAntony

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 avatar Mar 16 '22 05:03 duanjr

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)

duanjr avatar Mar 16 '22 05:03 duanjr

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.

HemantAntony avatar Mar 16 '22 05:03 HemantAntony

@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 🙂

bkunda avatar Mar 16 '22 06:03 bkunda

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.

duanjr avatar Mar 16 '22 06:03 duanjr

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

HemantAntony avatar Aug 15 '22 12:08 HemantAntony

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.

jeetee avatar Dec 20 '22 11:12 jeetee

Maybe so?

Example

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

oMrSmith avatar Dec 20 '22 12:12 oMrSmith