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

Fix #308034: Add user line spacing to text inspector #6347

Merged
merged 1 commit into from
Nov 17, 2020

Conversation

elerouxx
Copy link
Contributor

@elerouxx elerouxx commented Jul 19, 2020

Resolves https://musescore.org/en/node/308034

Adds custom text line spacing control to Text Inspector and Text Styles

Spacing is in lines (li.)
The new property is a multiplier to the font's natural line spacing, in which 1.0 lines would be the full natural spacing of the font (and MuseScore's current default), 0.5 lines would be half of this space and so on, in a similar way than most word processors deal with line spacing.

This property was added for every Text Style and to the Text inspector.

Image shows the modified inspector and some text elements adjusted to different spacing.

image

Code review, feedback and testing will be greatly appreciated!

  • I signed CLA
  • I made sure the code compiles on my machine
  • I made sure there are no unnecessary changes in the code

@Jojo-Schmitz
Copy link
Contributor

Commit title lacks "fix #308034:", and PR message should include that this resolves https://musescore.org/en/node/308034

@elerouxx elerouxx marked this pull request as draft July 19, 2020 13:09
@elerouxx elerouxx force-pushed the text-user-linesp-inspect-3.x branch from 0e12b1e to a9fbd32 Compare July 19, 2020 23:25
@elerouxx elerouxx changed the title [WIP] Add user line spacing to text inspector [WIP] Fix #308034: Add user line spacing to text inspector Jul 19, 2020
@elerouxx elerouxx force-pushed the text-user-linesp-inspect-3.x branch from a9fbd32 to a31ce63 Compare July 20, 2020 21:23
libmscore/property.h Outdated Show resolved Hide resolved
mscore/editstyle.ui Outdated Show resolved Hide resolved
@elerouxx elerouxx force-pushed the text-user-linesp-inspect-3.x branch 2 times, most recently from 9fc3929 to f6503de Compare July 22, 2020 04:20
@elerouxx elerouxx force-pushed the text-user-linesp-inspect-3.x branch from f6503de to 2e41022 Compare July 23, 2020 04:12
@elerouxx elerouxx changed the title [WIP] Fix #308034: Add user line spacing to text inspector Fix #308034: Add user line spacing to text inspector Jul 23, 2020
@elerouxx elerouxx marked this pull request as ready for review July 23, 2020 04:27
@elerouxx elerouxx force-pushed the text-user-linesp-inspect-3.x branch from 2e41022 to c078a00 Compare August 10, 2020 03:05
@elerouxx elerouxx force-pushed the text-user-linesp-inspect-3.x branch from c078a00 to 294eb68 Compare August 23, 2020 20:11
@vpereverzev vpereverzev merged commit c8a9dd7 into musescore:3.x Nov 17, 2020
igorkorsukov added a commit to igorkorsukov/MuseScore that referenced this pull request Feb 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants