Fix #20915 - Weird behaviour of "Reset to default" buttons in "Prefer…

…ence | Shortcuts"

The loadDefaultShortcuts() function in mscore/shortcut.cpp was not populating the list it returns.

Also:
- Made labels for "Reset to Default" buttons more explicit.
- Added (temporarily?) support for the <code> tag to loadDefaultShortcuts(), until the patch with "NumPad" support is merged (pull request 315), which makes the <code> tag obsolete.

mscore/prefsdialog.ui

@@ -34,7 +34,7 @@
      <item>
       <widget class="QPushButton" name="resetToDefault">
        <property name="text">
-        <string>Reset to Default</string>
+        <string>Reset All Preferences to Default</string>
        </property>
       </widget>
      </item>
@@ -2661,7 +2661,7 @@
          <item>
           <widget class="QPushButton" name="resetShortcut">
            <property name="text">
-            <string>Reset to Default</string>
+            <string>Reset Shortcut to Default</string>
            </property>
           </widget>
          </item>

mscore/shortcut.cpp

@@ -474,7 +474,6 @@ static QList<Shortcut1*> loadDefaultShortcuts()
             return list;
             }
       XmlReader e(&f);
-      QString key;
       while (e.readNextStartElement()) {
             if (e.name() == "Shortcuts") {
                   while (e.readNextStartElement()) {
@@ -493,6 +492,7 @@ static QList<Shortcut1*> loadDefaultShortcuts()
                                     else
                                           e.unknown();
                                     }
+                              list.append(sc);
                               }
                         else
                               e.unknown();