Upgrade to KDS v3.0.0 and reference npm package

kolibri/core/assets/src/views/UserTable/index.vue

@@ -107,7 +107,7 @@
               <KRadioButton
                 v-else-if="selectable && !enableMultipleSelection"
                 :disabled="disabled"
-                :value="user.id"
+                :buttonValue="user.id"
                 :currentValue="firstSelectedUser"
                 :label="''"
                 data-test="userRadioButton"

kolibri/core/assets/src/views/language-switcher/LanguageSwitcherModal.vue

@@ -26,7 +26,7 @@
             :key="language.id"
             ref="languageItem"
             v-model="selectedLanguage"
-            :value="language.id"
+            :buttonValue="language.id"
             :label="language.lang_name"
             :title="language.english_name"
             class="language-name"

kolibri/core/assets/src/views/sync/RadioButtonGroup.vue

@@ -4,7 +4,7 @@
     <KRadioButton
       v-for="(item, idx) in items"
       :key="idx"
-      :value="itemValue(item)"
+      :buttonValue="itemValue(item)"
       :currentValue="currentValue"
       :label="itemLabel(item)"
       :description="description(item)"

kolibri/core/assets/src/views/sync/SelectDeviceModalGroup/SelectDeviceForm.vue

@@ -38,7 +38,7 @@
             :key="idx"
             v-model="selectedDeviceId"
             class="radio-button"
-            :value="d.id"
+            :buttonValue="d.id"
             :label="d.nickname"
             :description="d.base_url"
             :disabled="formDisabled || !isDeviceAvailable(d.id)"
@@ -65,7 +65,7 @@
             :key="d.id"
             v-model="selectedDeviceId"
             class="radio-button"
-            :value="d.instance_id"
+            :buttonValue="d.instance_id"
             :label="formatNameAndId(d.device_name, d.id)"
             :description="formatBaseDevice(d)"
             :disabled="formDisabled || fetchFailed || !isDeviceAvailable(d.id)"

kolibri/core/assets/src/views/sync/SelectSyncSourceModal.vue

@@ -9,15 +9,15 @@
     <KRadioButton
       v-model="source"
       :label="$tr('dataPortalLabel')"
-      :value="SyncSources.PORTAL"
+      :buttonValue="SyncSources.PORTAL"
       :disabled="portalIsOffline || formIsDisabled"
       :autofocus="!portalIsOffline"
       :description="$tr('dataPortalDescription')"
     />
     <KRadioButton
       v-model="source"
       :label="$tr('localNetworkLabel')"
-      :value="SyncSources.PEER"
+      :buttonValue="SyncSources.PEER"
       :disabled="formIsDisabled"
       :description="$tr('localNetworkDescription')"
     />

kolibri/core/package.json

@@ -21,7 +21,7 @@
     "js-cookie": "^3.0.5",
     "knuth-shuffle-seeded": "^1.0.6",
     "kolibri-constants": "0.2.0",
-    "kolibri-design-system": "https://github.com/learningequality/kolibri-design-system#0ed2f274b1bc3808218a4d3f526c181b96b32c6d",
+    "kolibri-design-system": "~3.0.0",
     "lockr": "0.8.5",
     "lodash": "^4.17.21",
     "loglevel": "^1.8.1",

kolibri/plugins/coach/assets/src/views/plan/CreateExamPage/CreateExamPreview.vue

@@ -78,13 +78,13 @@
           v-model="fixedOrder"
           :label="coachString('orderRandomLabel')"
           :description="coachString('orderRandomDescription')"
-          :value="false"
+          :buttonValue="false"
         />
         <KRadioButton
           v-model="fixedOrder"
           :label="coachString('orderFixedLabel')"
           :description="coachString('orderFixedDescription')"
-          :value="true"
+          :buttonValue="true"
         />
       </div>
 

kolibri/plugins/coach/assets/src/views/plan/CreateExamPage/SectionEditor.vue

@@ -86,7 +86,7 @@
           <KRadioButton
             v-model="selectedQuestionOrder"
             :label="randomizedLabel$()"
-            :value="true"
+            :buttonValue="true"
             :description="randomizedOptionDescription$()"
           />
         </KGridItem>
@@ -98,7 +98,7 @@
           <KRadioButton
             v-model="selectedQuestionOrder"
             :label="fixedLabel$()"
-            :value="false"
+            :buttonValue="false"
             :description="fixedOptionDescription$()"
           />
         </KGridItem>

kolibri/plugins/coach/assets/src/views/plan/assignments/AssignmentCopyModal.vue

@@ -14,7 +14,7 @@
         :key="classroom.id"
         v-model="selectedClassroomId"
         :label="classroomLabel(classroom)"
-        :value="classroom.id"
+        :buttonValue="classroom.id"
         data-test="radio-button"
       />
     </div>

kolibri/plugins/coach/assets/src/views/plan/assignments/RecipientSelector.vue

@@ -3,7 +3,7 @@
   <div>
     <!-- Entire class -->
     <KRadioButton
-      :value="true"
+      :buttonValue="true"
       :currentValue="entireClassIsSelected"
       :disabled="disabled"
       @change="selectEntireClass()"

kolibri/plugins/device/assets/src/views/DeviceSettingsPage/PrimaryStorageLocationModal.vue

@@ -14,7 +14,7 @@
       v-for="path in storageLocations"
       :key="path.index"
       v-model="selectedPath"
-      :value="path.path"
+      :buttonValue="path.path"
       :label="path.path"
     />
   </KModal>

kolibri/plugins/device/assets/src/views/DeviceSettingsPage/RemoveStorageLocationModal.vue

@@ -15,7 +15,7 @@
       v-for="path in storageLocations"
       :key="path.index"
       v-model="selectedPath"
-      :value="path"
+      :buttonValue="path"
       :label="path"
     />
   </KModal>

kolibri/plugins/device/assets/src/views/DeviceSettingsPage/__test__/DeviceSettingsPage.spec.js

@@ -126,10 +126,10 @@ describe('DeviceSettingsPage', () => {
   function assertIsSelected(button, expected) {
     /*
      * HACK(kds-test) The only way to tell it's checked in the DOM
-     * is to check if "value" and "currentValue" props have the same value
+     * is to check if "buttonValue" and "currentValue" props have the same value
      */
     const buttonProps = button.props();
-    const checked = buttonProps.value === buttonProps.currentValue;
+    const checked = buttonProps.buttonValue === buttonProps.currentValue;
     expect(checked).toBe(expected);
   }
 

kolibri/plugins/device/assets/src/views/DeviceSettingsPage/index.vue

@@ -70,38 +70,38 @@
           <KRadioButton
             data-test="landingPageButton"
             :label="$tr('learnerAppPageChoice')"
-            :value="landingPageChoices.LEARN"
+            :buttonValue="landingPageChoices.LEARN"
             :currentValue="landingPage"
             @input="handleLandingPageChange"
           />
           <KRadioButton
             data-test="signInPageButton"
             :label="$tr('signInPageChoice')"
-            :value="landingPageChoices.SIGN_IN"
+            :buttonValue="landingPageChoices.SIGN_IN"
             :currentValue="landingPage"
             @input="handleLandingPageChange"
           />
           <div class="fieldset" style="margin-left: 32px">
             <KRadioButton
               data-test="allowGuestAccessButton"
               :label="$tr('allowGuestAccess')"
-              :value="SignInPageOptions.ALLOW_GUEST_ACCESS"
+              :buttonValue="SignInPageOptions.ALLOW_GUEST_ACCESS"
               :currentValue="signInPageOption"
               :disabled="disableSignInPageOptions"
               @input="handleSignInPageChange"
             />
             <KRadioButton
               data-test="disallowGuestAccessButton"
               :label="$tr('disallowGuestAccess')"
-              :value="SignInPageOptions.DISALLOW_GUEST_ACCESS"
+              :buttonValue="SignInPageOptions.DISALLOW_GUEST_ACCESS"
               :currentValue="signInPageOption"
               :disabled="disableSignInPageOptions"
               @input="handleSignInPageChange"
             />
             <KRadioButton
               data-test="lockedContentButton"
               :label="$tr('lockedContent')"
-              :value="SignInPageOptions.LOCKED_CONTENT"
+              :buttonValue="SignInPageOptions.LOCKED_CONTENT"
               :currentValue="signInPageOption"
               :disabled="disableSignInPageOptions"
               @input="handleSignInPageChange"
@@ -118,13 +118,13 @@
           </p>
           <KRadioButton
             :label="$tr('doNotAllowDownload')"
-            :value="meteredConnectionDownloadOptions.DISALLOW_DOWNLOAD_ON_METERED_CONNECTION"
+            :buttonValue="meteredConnectionDownloadOptions.DISALLOW_DOWNLOAD_ON_METERED_CONNECTION"
             :currentValue="meteredConnectionDownloadOption"
             @input="handleMeteredConnectionDownloadChange"
           />
           <KRadioButton
             :label="$tr('allowDownload')"
-            :value="meteredConnectionDownloadOptions.ALLOW_DOWNLOAD_ON_METERED_CONNECTION"
+            :buttonValue="meteredConnectionDownloadOptions.ALLOW_DOWNLOAD_ON_METERED_CONNECTION"
             :currentValue="meteredConnectionDownloadOption"
             @input="handleMeteredConnectionDownloadChange"
           />

kolibri/plugins/device/assets/src/views/FacilitiesPage/CreateNewFacilityModal.vue

@@ -21,14 +21,14 @@
     <KRadioButton
       v-model="preset"
       class="permission-preset-radio-button"
-      :value="Presets.NONFORMAL"
+      :buttonValue="Presets.NONFORMAL"
       :label="$tr('nonFormalLabel')"
       :description="$tr('nonFormalDescription')"
     />
     <KRadioButton
       v-model="preset"
       class="permission-preset-radio-button"
-      :value="Presets.FORMAL"
+      :buttonValue="Presets.FORMAL"
       :label="$tr('formalLabel')"
       :description="$tr('formalDescription')"
     />

kolibri/plugins/device/assets/src/views/ManageContentPage/SelectTransferSourceModal/DriveList.vue

@@ -16,7 +16,7 @@
         v-for="drive in drives"
         :key="drive.id"
         :label="enabledDriveLabel(drive)"
-        :value="drive.id"
+        :buttonValue="drive.id"
         :currentValue="value"
         @change="$emit('input', drive.id)"
       />

kolibri/plugins/device/assets/src/views/ManageContentPage/SelectTransferSourceModal/SelectImportSourceModal.vue

@@ -9,22 +9,22 @@
     <KRadioButton
       v-model="source"
       :label="$tr('network')"
-      :value="ContentSources.KOLIBRI_STUDIO"
+      :buttonValue="ContentSources.KOLIBRI_STUDIO"
       :disabled="kolibriStudioIsOffline || formIsDisabled"
       :autofocus="!kolibriStudioIsOffline"
       :description="$tr('studioDescription')"
     />
     <KRadioButton
       v-model="source"
       :label="$tr('localNetworkOrInternet')"
-      :value="ContentSources.PEER_KOLIBRI_SERVER"
+      :buttonValue="ContentSources.PEER_KOLIBRI_SERVER"
       :disabled="formIsDisabled"
       :description="$tr('networkDescription')"
     />
     <KRadioButton
       v-model="source"
       :label="$tr('localDrives')"
-      :value="ContentSources.LOCAL_DRIVE"
+      :buttonValue="ContentSources.LOCAL_DRIVE"
       :disabled="formIsDisabled"
       :description="$tr('localDescription')"
     />

kolibri/plugins/facility/assets/src/views/UserCreatePage.vue

@@ -54,14 +54,14 @@
               :disabled="busy"
               :label="coreString('classCoachLabel')"
               :description="coreString('classCoachDescription')"
-              :value="true"
+              :buttonValue="true"
             />
             <KRadioButton
               v-model="classCoachIsSelected"
               :disabled="busy"
               :label="coreString('facilityCoachLabel')"
               :description="coreString('facilityCoachDescription')"
-              :value="false"
+              :buttonValue="false"
             />
           </fieldset>
 

kolibri/plugins/facility/assets/src/views/UserEditPage.vue

@@ -64,14 +64,14 @@
                 :disabled="formDisabled"
                 :label="coreString('classCoachLabel')"
                 :description="coreString('classCoachDescription')"
-                :value="true"
+                :buttonValue="true"
               />
               <KRadioButton
                 v-model="classCoachIsSelected"
                 :disabled="formDisabled"
                 :label="coreString('facilityCoachLabel')"
                 :description="coreString('facilityCoachDescription')"
-                :value="false"
+                :buttonValue="false"
               />
             </fieldset>
           </template>

kolibri/plugins/media_player/assets/src/views/MediaPlayerLanguages/LanguagesMenuItem.vue

@@ -5,7 +5,7 @@
       ref="kRadioButton"
       name="languages-menu-item"
       :label="label"
-      :value="1"
+      :buttonValue="1"
       :currentValue="selected ? 1 : 0"
       @change="setLanguage(value)"
       @keydown.enter="setLanguage(value), $emit('hide')"

kolibri/plugins/setup_wizard/assets/src/views/JoinOrNewLOD.vue

@@ -8,14 +8,14 @@
     <KRadioButton
       v-model="selected"
       :label="$tr('joinFacilityLabel')"
-      :value="Options.JOIN"
+      :buttonValue="Options.JOIN"
       class="radio-button"
       :autofocus="isJoinSetup"
     />
     <KRadioButton
       v-model="selected"
       :label="$tr('importFromFacilityLabel')"
-      :value="Options.IMPORT"
+      :buttonValue="Options.IMPORT"
       class="radio-button"
       :autofocus="isImportSetup"
     />

kolibri/plugins/setup_wizard/assets/src/views/onboarding-forms/CreateLearnerAccountForm.vue

@@ -14,15 +14,15 @@
       v-model="setting"
       class="radio-button"
       :label="$tr('yesOptionLabel')"
-      :value="true"
+      :buttonValue="true"
       :autofocus="setting"
     />
     <KRadioButton
       ref="noRadio"
       v-model="setting"
       class="radio-button"
       :label="$tr('noOptionLabel')"
-      :value="false"
+      :buttonValue="false"
       :autofocus="!setting"
     />
     <p class="description">

kolibri/plugins/setup_wizard/assets/src/views/onboarding-forms/FacilityPermissionsForm.vue

@@ -11,7 +11,7 @@
       ref="first-button"
       v-model="selected"
       class="permission-preset-radio-button"
-      :value="Presets.NONFORMAL"
+      :buttonValue="Presets.NONFORMAL"
       :label="$tr('nonFormalLabel')"
       :description="$tr('nonFormalDescription')"
     />
@@ -24,7 +24,7 @@
     <KRadioButton
       v-model="selected"
       class="permission-preset-radio-button"
-      :value="Presets.FORMAL"
+      :buttonValue="Presets.FORMAL"
       :label="$tr('formalLabel')"
       :description="$tr('formalDescription')"
     />

kolibri/plugins/setup_wizard/assets/src/views/onboarding-forms/FullOrLearnOnlyDeviceForm.vue

@@ -7,14 +7,14 @@
     <KRadioButton
       v-model="selected"
       :label="$tr('fullDeviceLabel')"
-      :value="Options.FULL"
+      :buttonValue="Options.FULL"
       :description="$tr('fullDeviceDescription')"
       :autofocus="isFullSetup"
     />
     <KRadioButton
       v-model="selected"
       :label="$tr('learnOnlyDeviceLabel')"
-      :value="Options.LOD"
+      :buttonValue="Options.LOD"
       :description="$tr('learnOnlyDeviceDescription')"
       :autofocus="isLODSetup"
     />

kolibri/plugins/setup_wizard/assets/src/views/onboarding-forms/GettingStartedForm.vue

@@ -6,13 +6,13 @@
   >
     <KRadioButton
       v-model="selected"
-      :value="Options.PERSONAL"
+      :buttonValue="Options.PERSONAL"
       :label="$tr('quickStartLabel')"
       :description="getCommonSyncString('onMyOwn')"
     />
     <KRadioButton
       v-model="selected"
-      :value="Options.PUBLIC"
+      :buttonValue="Options.PUBLIC"
       :label="$tr('advancedSetupLabel')"
       :description="$tr('advancedSetupDescription')"
     />