MuseScore icon indicating copy to clipboard operation
MuseScore copied to clipboard

[MU4 Issue] [UI Finesse] On wide screens, the text in Learn > Classes is too wide to be comfortably readable

Open cbjeukendrup opened this issue 1 year ago • 2 comments

Describe the bug If the MuseScore window is relatively wide, like full screen on a 27" screen, the text in Home > Learn > Classes becomes too wide to be comfortably readably (i.e. there are too many words per line; lines too long).

Expected behavior Text should have a maximum width. Most (accessibility) guidelines say max 50-80 characters per line; so I guess that can be translated to something like 576 pixels.

Screenshots Schermafbeelding 2022-08-04 om 01 12 12

cbjeukendrup avatar Aug 03 '22 23:08 cbjeukendrup

Also on less wide screen, it doesn't scroll. On mine, I can't see the button :-(

MarcSabatella avatar Aug 10 '22 17:08 MarcSabatella

@jessjwilliamson Can you look at this issue? How can we best solve the problems of Casper and Marс?

@iwoithe Let's wait for Jessica's answer

Eism avatar Aug 11 '22 09:08 Eism

@jessjwilliamson Can you look at this issue? How can we best solve the problems of Casper and Marс?

@iwoithe Let's wait for Jessica's answer

Hi! Thank you for reaching out to me. I can suggest a few things that might solve this issue, with screenshots I provided of new design. This is also updated on Figma in HOME_TAB. One thing I'd also add is I don't believe there is a point of having a search box in this specific Classes page. Get started and Advanced should still have search box but not Classes. :) So...

  • The BG1 rounded box behind text should be maximum 1000 pixels wide, which means the text box (in Figma anyway) would be reduced to 928 pixels, so it can fit within the padding and BG1 box.
  • To make text more readable I'd suggest create another paragraph on sentence "In addition," etc.
  • If it's possible, I did this in Figma but paragraph spacing between text lines I've made a bit taller to 6.

Here's what it would look like in different screens sizes, and yes this page should be scrollable for small screens if this section is too big.

image image image

jessjwilliamson avatar Aug 12 '22 10:08 jessjwilliamson

is this issue avalable now? I am a beginner in C++ and want to contribute if possible

aritroCoder avatar Sep 10 '22 08:09 aritroCoder

Hello! Welcome to MuseScore!

This issue already has a PR by @iwoithe . However, I do not think that PR has the updated specs @jessjwilliamson provided

@iwoithe Are you still working on this?

HemantAntony avatar Sep 10 '22 08:09 HemantAntony

@iwoithe Are you still working on this?

Yeah, sorry for the inactivity. My life has picked up quite significantly in the last few weeks and will probably stay that way for another few weeks.

@aritroCoder Go for it. :slightly_smiling_face:

iwoithe avatar Sep 10 '22 09:09 iwoithe

image I am getting this warning in ubuntu, should I go ahead and install it, or there is something else i need to do?

aritroCoder avatar Sep 11 '22 07:09 aritroCoder

This is the wiki for building projects: https://github.com/musescore/MuseScore/wiki/Set-up-developer-environment

It should tell you how to install cmake and git

It also tells you how to build and run the project

HemantAntony avatar Sep 11 '22 08:09 HemantAntony

Hey I wish to contribute if the issue is still not resolved can I do it to you?

GargDivanshu avatar Sep 13 '22 16:09 GargDivanshu

@aritroCoder might be working on this. If you aren't, can you say so?

HemantAntony avatar Sep 13 '22 16:09 HemantAntony

Sorry, I was not able to respond for a while, yes I am working on it, already set up the development environment, and now trying to understand the source a little more

aritroCoder avatar Sep 13 '22 16:09 aritroCoder

I have followed the steps as given in the doc till https://github.com/musescore/MuseScore/wiki/Compile-on-the-command-line But now I am getting error while trying to run ./build.cmake image

Please assist me to identify why I get this error as I did all the steps mentioned correctly

aritroCoder avatar Sep 13 '22 18:09 aritroCoder

I would like to work on this, can someone assign me this issue?

octopols avatar Jan 26 '23 20:01 octopols

@jessjwilliamson Can you look at this issue? How can we best solve the problems of Casper and Marс? @iwoithe Let's wait for Jessica's answer

Hi! Thank you for reaching out to me. I can suggest a few things that might solve this issue, with screenshots I provided of new design. This is also updated on Figma in HOME_TAB. One thing I'd also add is I don't believe there is a point of having a search box in this specific Classes page. Get started and Advanced should still have search box but not Classes. :) So...

  • The BG1 rounded box behind text should be maximum 1000 pixels wide, which means the text box (in Figma anyway) would be reduced to 928 pixels, so it can fit within the padding and BG1 box.
  • To make text more readable I'd suggest create another paragraph on sentence "In addition," etc.
  • If it's possible, I did this in Figma but paragraph spacing between text lines I've made a bit taller to 6.

Here's what it would look like in different screens sizes, and yes this page should be scrollable for small screens if this section is too big.

image image image

I was not able to remove the search box only from the class tab in learn. In the PR the search box is gone from the learn page itself.

octopols avatar Jan 26 '23 21:01 octopols

max_1000_px scrollable_can_go_under_1000

octopols avatar Jan 26 '23 21:01 octopols

@octopols Thank you for jumping in! For search bar don't worry about removing it at all then if it's causing issues. We want to especially use it in the Get started tab! Your screenshots look good - is there a PR I can test? :)

jessjwilliamson avatar Jan 27 '23 09:01 jessjwilliamson

@octopols Thank you for jumping in! For search bar don't worry about removing it at all then if it's causing issues. We want to especially use it in the Get started tab! Your screenshots look good - is there a PR I can test? :)

I was able to fix the search box thing in the second commit of the PR #16087. Now it's visible in the Get started tab and hidden in the classes tab.

octopols avatar Jan 27 '23 14:01 octopols