Skip to content

Commit 3d63d72

Browse files
Jojo-SchmitzJojo-Schmitz
committed
Rename square to rectangle
as that is what it really is
1 parent e60691c commit 3d63d72

File tree

10 files changed

+22
-22
lines changed

10 files changed

+22
-22
lines changed

‎src/engraving/api/v1/apitypes.h‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -586,9 +586,9 @@ enum class HDuration {
586586
Q_ENUM_NS(HDuration);
587587

588588
enum class FrameType {
589-
NO_FRAME = int(mu::engraving::FrameType::NO_FRAME),
590-
SQUARE = int(mu::engraving::FrameType::SQUARE),
591-
CIRCLE = int(mu::engraving::FrameType::CIRCLE),
589+
NO_FRAME = int(mu::engraving::FrameType::NO_FRAME),
590+
RECTANGLE = int(mu::engraving::FrameType::RECTANGLE),
591+
CIRCLE = int(mu::engraving::FrameType::CIRCLE),
592592
};
593593
Q_ENUM_NS(FrameType);
594594

‎src/engraving/dom/textbase.cpp‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1846,7 +1846,7 @@ void TextBase::layoutFrame(LayoutData* ldata) const
18461846
ldata->frame = ldata->bbox();
18471847
}
18481848

1849-
if (square()) {
1849+
if (rectangle()) {
18501850
// make sure width >= height
18511851
if (ldata->frame.height() > ldata->frame.width()) {
18521852
double w = ldata->frame.height() - ldata->frame.width();

‎src/engraving/dom/textbase.h‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@ static constexpr double SUPERSCRIPT_OFFSET = -0.9; // of x-height
4646
//---------------------------------------------------------
4747

4848
enum class FrameType : char {
49-
NO_FRAME, SQUARE, CIRCLE
49+
NO_FRAME, RECTANGLE, CIRCLE
5050
};
5151

5252
//---------------------------------------------------------
@@ -414,7 +414,7 @@ class TextBase : public EngravingItem
414414
// helper functions
415415
bool hasFrame() const { return m_frameType != FrameType::NO_FRAME; }
416416
bool circle() const { return m_frameType == FrameType::CIRCLE; }
417-
bool square() const { return m_frameType == FrameType::SQUARE; }
417+
bool rectangle() const { return m_frameType == FrameType::RECTANGLE; }
418418

419419
TextStyleType textStyleType() const { return m_textStyleType; }
420420
void setTextStyleType(TextStyleType id) { m_textStyleType = id; }

‎src/engraving/rw/read114/read114.cpp‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -276,7 +276,7 @@ static bool readTextProperties(XmlReader& e, ReadContext& ctx, TextBase* t, Engr
276276
} else if (tag == "foregroundColor") { // same as "color" ?
277277
e.skipCurrentElement();
278278
} else if (tag == "frame") {
279-
t->setFrameType(e.readBool() ? FrameType::SQUARE : FrameType::NO_FRAME);
279+
t->setFrameType(e.readBool() ? FrameType::RECTANGLE : FrameType::NO_FRAME);
280280
t->setPropertyFlags(Pid::FRAME_TYPE, PropertyFlags::UNSTYLED);
281281
} else if (tag == "halign") {
282282
Align align = t->align();
@@ -509,10 +509,10 @@ static void readFingering114(XmlReader& e, Fingering* fing)
509509
} else if (tag == "frame") {
510510
auto frame = e.readInt();
511511
if (frame) {
512-
if (isStringNumber) { //default value is circle for stringnumber, square is set in tag circle
512+
if (isStringNumber) { //default value is circle for stringnumber, rectangle is set in tag circle
513513
fing->setFrameType(FrameType::CIRCLE);
514514
} else { //default value is square for stringnumber, circle is set in tag circle
515-
fing->setFrameType(FrameType::SQUARE);
515+
fing->setFrameType(FrameType::RECTANGLE);
516516
}
517517
} else {
518518
fing->setFrameType(FrameType::NO_FRAME);
@@ -522,7 +522,7 @@ static void readFingering114(XmlReader& e, Fingering* fing)
522522
if (circle) {
523523
fing->setFrameType(FrameType::CIRCLE);
524524
} else {
525-
fing->setFrameType(FrameType::SQUARE);
525+
fing->setFrameType(FrameType::RECTANGLE);
526526
}
527527
} else {
528528
e.skipCurrentElement();

‎src/engraving/rw/read206/read206.cpp‎

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -221,13 +221,13 @@ void Read206::readTextStyle206(MStyle* style, XmlReader& e, ReadContext& ctx, st
221221
} else if (tag == "sizeIsSpatiumDependent" || tag == "spatiumSizeDependent") {
222222
sizeIsSpatiumDependent = e.readInt();
223223
} else if (tag == "frameWidth") { // obsolete
224-
frameType = FrameType::SQUARE;
224+
frameType = FrameType::RECTANGLE;
225225
/*frameWidthMM =*/ e.readDouble();
226226
} else if (tag == "frameWidthS") {
227-
frameType = FrameType::SQUARE;
227+
frameType = FrameType::RECTANGLE;
228228
frameWidth = Spatium(e.readDouble());
229229
} else if (tag == "frame") {
230-
frameType = e.readInt() ? FrameType::SQUARE : FrameType::NO_FRAME;
230+
frameType = e.readInt() ? FrameType::RECTANGLE : FrameType::NO_FRAME;
231231
} else if (tag == "paddingWidth") { // obsolete
232232
/*paddingWidthMM =*/
233233
e.readDouble();
@@ -1334,7 +1334,7 @@ static bool readTextProperties206(XmlReader& e, ReadContext& ctx, TextBase* t)
13341334
} else if (tag == "foregroundColor") { // same as "color" ?
13351335
e.skipCurrentElement();
13361336
} else if (tag == "frame") {
1337-
t->setFrameType(e.readBool() ? FrameType::SQUARE : FrameType::NO_FRAME);
1337+
t->setFrameType(e.readBool() ? FrameType::RECTANGLE : FrameType::NO_FRAME);
13381338
t->setPropertyFlags(Pid::FRAME_TYPE, PropertyFlags::UNSTYLED);
13391339
} else if (tag == "frameRound") {
13401340
read400::TRead::readProperty(t, e, ctx, Pid::FRAME_ROUND);
@@ -1343,7 +1343,7 @@ static bool readTextProperties206(XmlReader& e, ReadContext& ctx, TextBase* t)
13431343
t->setFrameType(FrameType::CIRCLE);
13441344
} else {
13451345
if (t->circle()) {
1346-
t->setFrameType(FrameType::SQUARE);
1346+
t->setFrameType(FrameType::RECTANGLE);
13471347
}
13481348
}
13491349
t->setPropertyFlags(Pid::FRAME_TYPE, PropertyFlags::UNSTYLED);

‎src/engraving/style/styledef.cpp‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1236,7 +1236,7 @@ const std::array<StyleDef::StyleValue, size_t(Sid::STYLES)> StyleDef::styleValue
12361236
styleDef(measureNumberAlternateAlign, Align(AlignH::LEFT, AlignV::BASELINE)),
12371237
styleDef(measureNumberAlternatePosAbove, PointF(0.0, -4.0)),
12381238
styleDef(measureNumberAlternatePosBelow, PointF(0.0, 4.0)),
1239-
styleDef(measureNumberAlternateFrameType, int(FrameType::SQUARE)),
1239+
styleDef(measureNumberAlternateFrameType, int(FrameType::RECTANGLE)),
12401240
styleDef(measureNumberAlternateFramePadding, 0.3),
12411241
styleDef(measureNumberAlternateFrameWidth, 0.1),
12421242
styleDef(measureNumberAlternateFrameRound, 0),
@@ -1363,7 +1363,7 @@ const std::array<StyleDef::StyleValue, size_t(Sid::STYLES)> StyleDef::styleValue
13631363
styleDef(rehearsalMarkFontStyle, int(FontStyle::Bold)),
13641364
styleDef(rehearsalMarkColor, PropertyValue::fromValue(Color::BLACK)),
13651365
styleDef(rehearsalMarkAlign, Align(AlignH::HCENTER, AlignV::BASELINE)),
1366-
styleDef(rehearsalMarkFrameType, int(FrameType::SQUARE)),
1366+
styleDef(rehearsalMarkFrameType, int(FrameType::RECTANGLE)),
13671367
styleDef(rehearsalMarkFramePadding, 0.5),
13681368
styleDef(rehearsalMarkFrameWidth, 0.16),
13691369
styleDef(rehearsalMarkFrameRound, 0),

‎src/importexport/guitarpro/internal/importptb.cpp‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -844,7 +844,7 @@ void PowerTab::addToScore(ptSection& sec)
844844
lastPart = sec.partMarker;
845845
auto seg = measure->getSegment(SegmentType::ChordRest, measure->tick());
846846
RehearsalMark* t = new RehearsalMark(seg);
847-
t->setFrameType(FrameType::SQUARE);
847+
t->setFrameType(FrameType::RECTANGLE);
848848
t->setPlainText(String(Char::fromAscii(sec.partMarker)));
849849
t->setTrack(0);
850850
seg->add(t);

‎src/importexport/musicxml/internal/musicxml/export/exportmusicxml.cpp‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -745,7 +745,7 @@ static String frame2xml(const TextBase* el)
745745
switch (el->frameType()) {
746746
case FrameType::CIRCLE:
747747
return u" enclosure=\"circle\"";
748-
case FrameType::SQUARE:
748+
case FrameType::RECTANGLE:
749749
return u" enclosure=\"rectangle\"";
750750
default:
751751
return String();

‎src/importexport/musicxml/internal/musicxml/import/importmusicxmlpass2.cpp‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3593,7 +3593,7 @@ void MusicXmlParserDirection::direction(const String& partId,
35933593
} else if (m_enclosure == "none") {
35943594
t->setFrameType(FrameType::NO_FRAME);
35953595
} else if (m_enclosure == "rectangle") {
3596-
t->setFrameType(FrameType::SQUARE);
3596+
t->setFrameType(FrameType::RECTANGLE);
35973597
t->setFrameRound(0);
35983598
}
35993599

@@ -3691,7 +3691,7 @@ void MusicXmlParserDirection::direction(const String& partId,
36913691
} else if (m_enclosure == "none") {
36923692
dynamic->setFrameType(FrameType::NO_FRAME);
36933693
} else if (m_enclosure == "rectangle") {
3694-
dynamic->setFrameType(FrameType::SQUARE);
3694+
dynamic->setFrameType(FrameType::RECTANGLE);
36953695
dynamic->setFrameRound(0);
36963696
}
36973697

‎src/notation/view/widgets/editstyle.cpp‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1074,7 +1074,7 @@ EditStyle::EditStyle(QWidget* parent)
10741074

10751075
textStyleFrameType->clear();
10761076
textStyleFrameType->addItem(muse::qtrc("notation/editstyle", "None", "no frame for text"), int(FrameType::NO_FRAME));
1077-
textStyleFrameType->addItem(muse::qtrc("notation/editstyle", "Rectangle"), int(FrameType::SQUARE));
1077+
textStyleFrameType->addItem(muse::qtrc("notation/editstyle", "Rectangle"), int(FrameType::RECTANGLE));
10781078
textStyleFrameType->addItem(muse::qtrc("notation/editstyle", "Circle"), int(FrameType::CIRCLE));
10791079

10801080
connect(dynamicsAndHairpinPos, &QComboBox::currentIndexChanged, dynamicsAndHairpinPosDescription, [=]() {

0 commit comments

Comments
 (0)