@instructure/quiz-core 20.31.2-snapshot.22 → 20.31.2-snapshot.24

package/es/banks/components/AsyncSearch/index.js

@@ -153,7 +153,7 @@ export var AsyncSearch = /*#__PURE__*/function (_Component) {
           });
         });
       }
-      if (_this.props.improvedBankSharingEnabled && shouldSearchCourses) {
+      if (shouldSearchCourses) {
         _this.props.getShareableCourses(value).then(function (shareableCourses) {
           var filteredCourses = _this.filterOptions(shareableCourses);
           _this.setState({
@@ -357,7 +357,6 @@ export var AsyncSearch = /*#__PURE__*/function (_Component) {
 }(Component);
 AsyncSearch.propTypes = {
   debounceDelay: PropTypes.number,
-  improvedBankSharingEnabled: PropTypes.bool.isRequired,
   minChars: PropTypes.number,
   onSelectionChange: PropTypes.func.isRequired,
   getShareableCourses: PropTypes.func,

package/es/banks/components/Banks/index.js

@@ -18,7 +18,6 @@ function mapStateToProps(state, props) {
   var rootAccountUuid = state.getIn(['outcomes', 'externalAccountUuid']);
   return {
     itemBankSearchTableEnabled: featureOn('item_bank_search_table'),
-    improvedBankSharingEnabled: featureOn('improved_bank_sharing'),
     usedByCourseEnabled: featureOn('filter_item_banks_used_by_course'),
     subAccountBankSharingEnabled: featureOn('sub_account_bank_sharing'),
     scopeType: scopeType,

package/es/banks/components/Banks/presenter.js

@@ -308,7 +308,6 @@ export var Banks = (_dec = withStyle(generateStyle, generateComponentTheme), _de
         width: 4
       }, jsx(AsyncSearch, {
         getShareableSubaccounts: this.props.getShareableSubaccounts,
-        improvedBankSharingEnabled: this.props.improvedBankSharingEnabled,
         minChars: this.props.minChars,
         onSelectionChange: this.handleSearchSelectionChange,
         searchType: SHARED_WITH_ACCOUNT,
@@ -368,7 +367,7 @@ export var Banks = (_dec = withStyle(generateStyle, generateComponentTheme), _de
     key: "renderHeader",
     value: function renderHeader() {
       return jsx("div", {
-        css: [this.props.styles.header, !this.props.improvedBankSharingEnabled ? this.props.styles.borderedBottom : {}]
+        css: [this.props.styles.header]
       }, jsx(Heading, {
         as: this.props.isCompact ? 'h2' : 'h1',
         level: "h1"
@@ -423,7 +422,7 @@ export var Banks = (_dec = withStyle(generateStyle, generateComponentTheme), _de
         returnToUrl: this.props.returnToUrl,
         scope: this.props.scope,
         navigateToBuild: this.props.navigateToBuild
-      })), this.renderHeader(), this.props.improvedBankSharingEnabled && this.renderFilters(), this.renderBanksSearchResults(), jsx(CreateBankModal, {
+      })), this.renderHeader(), this.renderFilters(), this.renderBanksSearchResults(), jsx(CreateBankModal, {
         itemBanksScope: this.props.scope
       })));
     }
@@ -444,7 +443,6 @@ export var Banks = (_dec = withStyle(generateStyle, generateComponentTheme), _de
   }),
   // from consumer
   clear: PropTypes.func.isRequired,
-  improvedBankSharingEnabled: PropTypes.bool,
   getBanks: PropTypes.func.isRequired,
   getCourseInfo: PropTypes.func,
   // from consumer
@@ -512,7 +510,6 @@ export var Banks = (_dec = withStyle(generateStyle, generateComponentTheme), _de
   onShare: function onShare() {},
   scope: {},
   returnToUrl: null,
-  improvedBankSharingEnabled: false,
   scopeType: 'user',
   sortBy: 'name',
   sortOrder: 'ascending',

package/es/banks/components/SharingModal/ShareList/index.js

@@ -78,7 +78,7 @@ export var ShareList = (_dec = withStyle(generateStyle, null), _dec(_class = (_S
       })), jsx(Grid.Col, null, jsx("span", {
         css: isEntityRemoved ? _this.props.styles.labelRemoved : null,
         "data-automation": isEntityRemoved ? 'sdk-shared-entity-removed' : null
-      }, entity.get('title'))), _this.props.improvedBankSharingEnabled && jsx(Grid.Col, {
+      }, entity.get('title'))), jsx(Grid.Col, {
         width: "auto"
       }, jsx(SimpleSelect, {
         onChange: _this.handlePermissionChange(entity),
@@ -95,7 +95,7 @@ export var ShareList = (_dec = withStyle(generateStyle, null), _dec(_class = (_S
         value: CAN_EDIT
       }, t('Can edit')))), jsx(Grid.Col, {
         width: "auto"
-      }, _this.props.canEdit && (_this.props.improvedBankSharingEnabled || !isCourse) && _this.renderRemoveButton(entity, isEntityRemoved)));
+      }, _this.props.canEdit && _this.renderRemoveButton(entity, isEntityRemoved)));
     };
     return _this;
   }
@@ -117,7 +117,6 @@ export var ShareList = (_dec = withStyle(generateStyle, null), _dec(_class = (_S
     permission: PropTypes.string.isRequired,
     id: PropTypes.string.isRequired
   })).isRequired,
-  improvedBankSharingEnabled: PropTypes.bool.isRequired,
   onPermissionChange: PropTypes.func.isRequired,
   onEntityRemove: PropTypes.func.isRequired,
   sharedBanks: ImmutablePropTypes.list,

package/es/banks/components/SharingModal/index.js

@@ -11,7 +11,6 @@ function mapStateToProps(state, ownProps) {
     return sharedBank.get('bankId') === ownProps.bank.id;
   }).toList();
   return {
-    improvedBankSharingEnabled: featureOn('improved_bank_sharing'),
     subAccountBankSharingEnabled: featureOn('sub_account_bank_sharing'),
     sharedBanks: sharedBanks
   };

package/es/banks/components/SharingModal/presenter.js

@@ -325,13 +325,12 @@ export var SharingModalPresenter = (_dec = withStyle(generateStyle, generateComp
           getShareableCourses: _this.props.getShareableCourses,
           getShareableUsers: _this.props.getShareableUsers,
           getShareableSubaccounts: _this.props.getShareableSubaccounts,
-          improvedBankSharingEnabled: _this.props.improvedBankSharingEnabled,
           minChars: _this.props.minChars,
           onSelectionChange: _this.handleSearchSelectionChange,
           searchType: searchType,
           selectedOption: _this.state.searchSelection,
           selectionFilter: _this.handleSearchFilter
-        })), _this.props.improvedBankSharingEnabled && jsx(Grid.Col, {
+        })), jsx(Grid.Col, {
           width: "auto"
         }, jsx(SimpleSelect, {
           onChange: _this.handleSearchPermissionChange,
@@ -359,7 +358,6 @@ export var SharingModalPresenter = (_dec = withStyle(generateStyle, generateComp
       return jsx(ShareList, {
         canEdit: _this.props.bank.canEdit(),
         entities: entities,
-        improvedBankSharingEnabled: _this.props.improvedBankSharingEnabled,
         onPermissionChange: _this.handleEntityPermissionChange,
         onEntityRemove: _this.handleEntityRemove,
         sharedBanks: _this.props.sharedBanks
@@ -574,7 +572,6 @@ export var SharingModalPresenter = (_dec = withStyle(generateStyle, generateComp
   getUserInfo: PropTypes.func.isRequired,
   getShareableSubaccounts: PropTypes.func.isRequired,
   getSubaccountInfo: PropTypes.func.isRequired,
-  improvedBankSharingEnabled: PropTypes.bool,
   isOpen: PropTypes.bool,
   minChars: PropTypes.number,
   modalTrigger: PropTypes.node,
@@ -593,7 +590,6 @@ export var SharingModalPresenter = (_dec = withStyle(generateStyle, generateComp
 }, _SharingModalPresenter.defaultProps = {
   bank: null,
   currentUser: null,
-  improvedBankSharingEnabled: false,
   isOpen: false,
   minChars: void 0,
   modalTrigger: null,

package/es/common/components/layout/Page/index.js

@@ -9,6 +9,7 @@ import PropTypes from 'prop-types';
 import { withStyle, jsx } from '@instructure/emotion';
 import generateStyle from "./styles.js";
 import generateComponentTheme from "./theme.js";
+import t from '@instructure/quiz-i18n/es/format-message';
 export var Page = (_dec = withStyle(generateStyle, generateComponentTheme), _dec(_class = (_Page = /*#__PURE__*/function (_Component) {
   _inherits(Page, _Component);
   var _super = _createSuper(Page);
@@ -50,7 +51,8 @@ export var Page = (_dec = withStyle(generateStyle, generateComponentTheme), _dec
         tabIndex: "-1",
         ref: this.props.childrenWrapperRef,
         role: this.props.contentRole,
-        css: this.props.styles.childrenWrapper
+        css: this.props.styles.childrenWrapper,
+        "aria-label": t('Quiz content')
       }, this.props.children));
     }
   }]);
@@ -68,7 +70,7 @@ export var Page = (_dec = withStyle(generateStyle, generateComponentTheme), _dec
   makeStyles: PropTypes.func
 }, _Page.defaultProps = {
   childrenWrapperRef: function childrenWrapperRef() {},
-  contentRole: 'main',
+  contentRole: 'region',
   sidebarComponent: null,
   sidebarOpen: null,
   sidebarRole: 'navigation',

package/es/common/components/layout/sidebar/Sidebar/index.js

@@ -290,7 +290,8 @@ export var Sidebar = (_dec = withStyle(generateStyle, generateComponentTheme), _
         // take the click and scroll to itself when the nav is clicked by SR
         ,
         onClick: noop,
-        id: SIDEBAR_NAV_ID
+        id: SIDEBAR_NAV_ID,
+        "aria-label": t('Question Navigator')
       }, this.renderSidebarHeader(), this.renderTotalPointsPossible(), this.renderPinned(), this.renderTitle(), this.renderInstructions(), this.renderResults(), jsx("div", {
         css: this.props.styles.sidebarContent
       }, this.props.children)))));

package/lib/banks/components/AsyncSearch/index.js

@@ -161,7 +161,7 @@ var AsyncSearch = /*#__PURE__*/function (_Component) {
           });
         });
       }
-      if (_this.props.improvedBankSharingEnabled && shouldSearchCourses) {
+      if (shouldSearchCourses) {
         _this.props.getShareableCourses(value).then(function (shareableCourses) {
           var filteredCourses = _this.filterOptions(shareableCourses);
           _this.setState({
@@ -366,7 +366,6 @@ var AsyncSearch = /*#__PURE__*/function (_Component) {
 exports.AsyncSearch = AsyncSearch;
 AsyncSearch.propTypes = {
   debounceDelay: _propTypes.default.number,
-  improvedBankSharingEnabled: _propTypes.default.bool.isRequired,
   minChars: _propTypes.default.number,
   onSelectionChange: _propTypes.default.func.isRequired,
   getShareableCourses: _propTypes.default.func,

package/lib/banks/components/Banks/index.js

@@ -25,7 +25,6 @@ function mapStateToProps(state, props) {
   var rootAccountUuid = state.getIn(['outcomes', 'externalAccountUuid']);
   return {
     itemBankSearchTableEnabled: (0, _featureCheck.featureOn)('item_bank_search_table'),
-    improvedBankSharingEnabled: (0, _featureCheck.featureOn)('improved_bank_sharing'),
     usedByCourseEnabled: (0, _featureCheck.featureOn)('filter_item_banks_used_by_course'),
     subAccountBankSharingEnabled: (0, _featureCheck.featureOn)('sub_account_bank_sharing'),
     scopeType: scopeType,

package/lib/banks/components/Banks/presenter.js

@@ -316,7 +316,6 @@ var Banks = (_dec = (0, _emotion.withStyle)(_styles.default, _theme.default), _d
         width: 4
       }, (0, _emotion.jsx)(_index7.default, {
         getShareableSubaccounts: this.props.getShareableSubaccounts,
-        improvedBankSharingEnabled: this.props.improvedBankSharingEnabled,
         minChars: this.props.minChars,
         onSelectionChange: this.handleSearchSelectionChange,
         searchType: _quizCommon.SHARED_WITH_ACCOUNT,
@@ -376,7 +375,7 @@ var Banks = (_dec = (0, _emotion.withStyle)(_styles.default, _theme.default), _d
     key: "renderHeader",
     value: function renderHeader() {
       return (0, _emotion.jsx)("div", {
-        css: [this.props.styles.header, !this.props.improvedBankSharingEnabled ? this.props.styles.borderedBottom : {}]
+        css: [this.props.styles.header]
       }, (0, _emotion.jsx)(_uiHeading.Heading, {
         as: this.props.isCompact ? 'h2' : 'h1',
         level: "h1"
@@ -431,7 +430,7 @@ var Banks = (_dec = (0, _emotion.withStyle)(_styles.default, _theme.default), _d
         returnToUrl: this.props.returnToUrl,
         scope: this.props.scope,
         navigateToBuild: this.props.navigateToBuild
-      })), this.renderHeader(), this.props.improvedBankSharingEnabled && this.renderFilters(), this.renderBanksSearchResults(), (0, _emotion.jsx)(_index5.default, {
+      })), this.renderHeader(), this.renderFilters(), this.renderBanksSearchResults(), (0, _emotion.jsx)(_index5.default, {
         itemBanksScope: this.props.scope
       })));
     }
@@ -452,7 +451,6 @@ var Banks = (_dec = (0, _emotion.withStyle)(_styles.default, _theme.default), _d
   }),
   // from consumer
   clear: _propTypes.default.func.isRequired,
-  improvedBankSharingEnabled: _propTypes.default.bool,
   getBanks: _propTypes.default.func.isRequired,
   getCourseInfo: _propTypes.default.func,
   // from consumer
@@ -520,7 +518,6 @@ var Banks = (_dec = (0, _emotion.withStyle)(_styles.default, _theme.default), _d
   onShare: function onShare() {},
   scope: {},
   returnToUrl: null,
-  improvedBankSharingEnabled: false,
   scopeType: 'user',
   sortBy: 'name',
   sortOrder: 'ascending',

package/lib/banks/components/SharingModal/ShareList/index.js

@@ -85,7 +85,7 @@ var ShareList = (_dec = (0, _emotion.withStyle)(_styles.default, null), _dec(_cl
       })), (0, _emotion.jsx)(_uiGrid.Grid.Col, null, (0, _emotion.jsx)("span", {
         css: isEntityRemoved ? _this.props.styles.labelRemoved : null,
         "data-automation": isEntityRemoved ? 'sdk-shared-entity-removed' : null
-      }, entity.get('title'))), _this.props.improvedBankSharingEnabled && (0, _emotion.jsx)(_uiGrid.Grid.Col, {
+      }, entity.get('title'))), (0, _emotion.jsx)(_uiGrid.Grid.Col, {
         width: "auto"
       }, (0, _emotion.jsx)(_uiSimpleSelect.SimpleSelect, {
         onChange: _this.handlePermissionChange(entity),
@@ -102,7 +102,7 @@ var ShareList = (_dec = (0, _emotion.withStyle)(_styles.default, null), _dec(_cl
         value: _quizCommon.CAN_EDIT
       }, (0, _formatMessage.default)('Can edit')))), (0, _emotion.jsx)(_uiGrid.Grid.Col, {
         width: "auto"
-      }, _this.props.canEdit && (_this.props.improvedBankSharingEnabled || !isCourse) && _this.renderRemoveButton(entity, isEntityRemoved)));
+      }, _this.props.canEdit && _this.renderRemoveButton(entity, isEntityRemoved)));
     };
     return _this;
   }
@@ -124,7 +124,6 @@ var ShareList = (_dec = (0, _emotion.withStyle)(_styles.default, null), _dec(_cl
     permission: _propTypes.default.string.isRequired,
     id: _propTypes.default.string.isRequired
   })).isRequired,
-  improvedBankSharingEnabled: _propTypes.default.bool.isRequired,
   onPermissionChange: _propTypes.default.func.isRequired,
   onEntityRemove: _propTypes.default.func.isRequired,
   sharedBanks: _reactImmutableProptypes.default.list,

package/lib/banks/components/SharingModal/index.js

@@ -18,7 +18,6 @@ function mapStateToProps(state, ownProps) {
     return sharedBank.get('bankId') === ownProps.bank.id;
   }).toList();
   return {
-    improvedBankSharingEnabled: (0, _featureCheck.featureOn)('improved_bank_sharing'),
     subAccountBankSharingEnabled: (0, _featureCheck.featureOn)('sub_account_bank_sharing'),
     sharedBanks: sharedBanks
   };

package/lib/banks/components/SharingModal/presenter.js

@@ -333,13 +333,12 @@ var SharingModalPresenter = (_dec = (0, _emotion.withStyle)(_styles.default, _th
           getShareableCourses: _this.props.getShareableCourses,
           getShareableUsers: _this.props.getShareableUsers,
           getShareableSubaccounts: _this.props.getShareableSubaccounts,
-          improvedBankSharingEnabled: _this.props.improvedBankSharingEnabled,
           minChars: _this.props.minChars,
           onSelectionChange: _this.handleSearchSelectionChange,
           searchType: searchType,
           selectedOption: _this.state.searchSelection,
           selectionFilter: _this.handleSearchFilter
-        })), _this.props.improvedBankSharingEnabled && (0, _emotion.jsx)(_uiGrid.Grid.Col, {
+        })), (0, _emotion.jsx)(_uiGrid.Grid.Col, {
           width: "auto"
         }, (0, _emotion.jsx)(_uiSimpleSelect.SimpleSelect, {
           onChange: _this.handleSearchPermissionChange,
@@ -367,7 +366,6 @@ var SharingModalPresenter = (_dec = (0, _emotion.withStyle)(_styles.default, _th
       return (0, _emotion.jsx)(_index.default, {
         canEdit: _this.props.bank.canEdit(),
         entities: entities,
-        improvedBankSharingEnabled: _this.props.improvedBankSharingEnabled,
         onPermissionChange: _this.handleEntityPermissionChange,
         onEntityRemove: _this.handleEntityRemove,
         sharedBanks: _this.props.sharedBanks
@@ -582,7 +580,6 @@ var SharingModalPresenter = (_dec = (0, _emotion.withStyle)(_styles.default, _th
   getUserInfo: _propTypes.default.func.isRequired,
   getShareableSubaccounts: _propTypes.default.func.isRequired,
   getSubaccountInfo: _propTypes.default.func.isRequired,
-  improvedBankSharingEnabled: _propTypes.default.bool,
   isOpen: _propTypes.default.bool,
   minChars: _propTypes.default.number,
   modalTrigger: _propTypes.default.node,
@@ -601,7 +598,6 @@ var SharingModalPresenter = (_dec = (0, _emotion.withStyle)(_styles.default, _th
 }, _SharingModalPresenter.defaultProps = {
   bank: null,
   currentUser: null,
-  improvedBankSharingEnabled: false,
   isOpen: false,
   minChars: void 0,
   modalTrigger: null,

package/lib/common/components/layout/Page/index.js

@@ -14,6 +14,7 @@ var _propTypes = _interopRequireDefault(require("prop-types"));
 var _emotion = require("@instructure/emotion");
 var _styles = _interopRequireDefault(require("./styles.js"));
 var _theme = _interopRequireDefault(require("./theme.js"));
+var _formatMessage = _interopRequireDefault(require("@instructure/quiz-i18n/es/format-message"));
 var _dec, _class, _Page;
 /** @jsx jsx */
 var Page = (_dec = (0, _emotion.withStyle)(_styles.default, _theme.default), _dec(_class = (_Page = /*#__PURE__*/function (_Component) {
@@ -57,7 +58,8 @@ var Page = (_dec = (0, _emotion.withStyle)(_styles.default, _theme.default), _de
         tabIndex: "-1",
         ref: this.props.childrenWrapperRef,
         role: this.props.contentRole,
-        css: this.props.styles.childrenWrapper
+        css: this.props.styles.childrenWrapper,
+        "aria-label": (0, _formatMessage.default)('Quiz content')
       }, this.props.children));
     }
   }]);
@@ -75,7 +77,7 @@ var Page = (_dec = (0, _emotion.withStyle)(_styles.default, _theme.default), _de
   makeStyles: _propTypes.default.func
 }, _Page.defaultProps = {
   childrenWrapperRef: function childrenWrapperRef() {},
-  contentRole: 'main',
+  contentRole: 'region',
   sidebarComponent: null,
   sidebarOpen: null,
   sidebarRole: 'navigation',

package/lib/common/components/layout/sidebar/Sidebar/index.js

@@ -297,7 +297,8 @@ var Sidebar = (_dec = (0, _emotion.withStyle)(_styles.default, _theme.default),
         // take the click and scroll to itself when the nav is clicked by SR
         ,
         onClick: _noop.default,
-        id: _quizCommon.SIDEBAR_NAV_ID
+        id: _quizCommon.SIDEBAR_NAV_ID,
+        "aria-label": (0, _formatMessage.default)('Question Navigator')
       }, this.renderSidebarHeader(), this.renderTotalPointsPossible(), this.renderPinned(), this.renderTitle(), this.renderInstructions(), this.renderResults(), (0, _emotion.jsx)("div", {
         css: this.props.styles.sidebarContent
       }, this.props.children)))));

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@instructure/quiz-core",
-  "version": "20.31.2-snapshot.22+0ea974b4c",
+  "version": "20.31.2-snapshot.24+407ff7c27",
   "license": "MIT",
   "description": "The Quiz React SDK by Instructure Inc.",
   "author": "Instructure, Inc. Engineering and Product Design",
@@ -46,11 +46,11 @@
     "@instructure/emotion": "^8.56.3",
     "@instructure/grading-utils": "^1.0.0",
     "@instructure/outcomes-ui": "^3.2.2",
-    "@instructure/quiz-common": "^20.31.2-snapshot.22+0ea974b4c",
-    "@instructure/quiz-i18n": "^20.31.2-snapshot.22+0ea974b4c",
-    "@instructure/quiz-interactions": "^20.31.2-snapshot.22+0ea974b4c",
-    "@instructure/quiz-number-input": "^20.31.2-snapshot.22+0ea974b4c",
-    "@instructure/quiz-rce": "^20.31.2-snapshot.22+0ea974b4c",
+    "@instructure/quiz-common": "^20.31.2-snapshot.24+407ff7c27",
+    "@instructure/quiz-i18n": "^20.31.2-snapshot.24+407ff7c27",
+    "@instructure/quiz-interactions": "^20.31.2-snapshot.24+407ff7c27",
+    "@instructure/quiz-number-input": "^20.31.2-snapshot.24+407ff7c27",
+    "@instructure/quiz-rce": "^20.31.2-snapshot.24+407ff7c27",
     "@instructure/ui-a11y-content": "^8.56.3",
     "@instructure/ui-alerts": "^8.56.3",
     "@instructure/ui-avatar": "^8.56.3",
@@ -114,7 +114,7 @@
     "file-saver": "~2.0.5",
     "humps": "^2.0.0",
     "immutable": "^3.8.1",
-    "instructure-validations": "^20.31.2-snapshot.22+0ea974b4c",
+    "instructure-validations": "^20.31.2-snapshot.24+407ff7c27",
     "ipaddr.js": "^1.5.4",
     "isomorphic-fetch": "^2.2.0",
     "isuuid": "^0.1.0",
@@ -163,7 +163,7 @@
     "jquery": "^2.2.3",
     "karma-junit-reporter": "^2.0.1",
     "most-subject": "^5.3.0",
-    "quiz-presets": "^20.31.2-snapshot.22+0ea974b4c",
+    "quiz-presets": "^20.31.2-snapshot.24+407ff7c27",
     "react": "^16.8.6",
     "react-addons-test-utils": "^15.6.2",
     "react-dom": "^16.8.6",
@@ -179,5 +179,5 @@
   "publishConfig": {
     "access": "public"
   },
-  "gitHead": "0ea974b4cc0200b59fa970a34f8147ed6262ba97"
+  "gitHead": "407ff7c277ee01cdfc1d9615b051418a1eb63be6"
 }