@instructure/outcomes-ui 4.1.1 → 4.1.3

package/lib/translated/en-CA/components/Gradebook/popovers/StudentPopover/__tests__/index.test.js

@@ -8,7 +8,8 @@ var _react = _interopRequireDefault(require("react"));
 var _react2 = require("@testing-library/react");
 require("@testing-library/jest-dom");
 var _index = require("../../../context/GradebookConfigContext/index.js");
-var _mockData = require("../../../__mocks__/mockData.js");
+var _mockData = require("../../../gradebook-table/StudentCell/__mocks__/mockData.js");
+var _mockData2 = require("../__mocks__/mockData.js");
 var _index2 = require("../../../../../../lib/index.js");
 jest.mock('format-message', function () {
   return function (msg) {
@@ -16,26 +17,16 @@ jest.mock('format-message', function () {
   };
 });
 describe('StudentPopover', function () {
-  var defaultProps = {
-    student: _mockData.mockStudent,
-    studentName: _mockData.mockStudent.display_name,
-    userDetails: _mockData.mockUserDetailsDefault,
-    masteryScores: _mockData.mockMasteryScores,
+  var defaultProps = (0, _objectSpread2.default)((0, _objectSpread2.default)({
+    studentName: _mockData.mockStudent.displayName
+  }, _mockData2.mockCaptionDefault), {}, {
+    masteryScores: _mockData2.mockMasteryScores,
     studentGradesUrl: '/courses/123/grades/1'
-  };
+  });
   var renderComponent = function renderComponent() {
     var props = arguments.length > 0 && arguments[0] !== void 0 ? arguments[0] : {};
     return (0, _react2.render)(/*#__PURE__*/_react.default.createElement(_index.GradebookConfigProvider, {
-      config: {
-        components: {
-          StudentPopover: function StudentPopover() {
-            return null;
-          },
-          SettingsTrayContent: function SettingsTrayContent() {
-            return null;
-          }
-        }
-      }
+      config: {}
     }, /*#__PURE__*/_react.default.createElement(_index2.StudentPopover, (0, _objectSpread2.default)((0, _objectSpread2.default)({}, defaultProps), props))));
   };
   describe('Rendering', function () {
@@ -43,12 +34,11 @@ describe('StudentPopover', function () {
       renderComponent();
       var link = _react2.screen.getByTestId('student-cell-link');
       expect(link).toBeInTheDocument();
-      expect((0, _react2.within)(link).getByText(_mockData.mockStudent.display_name)).toBeInTheDocument();
+      expect((0, _react2.within)(link).getByText(_mockData.mockStudent.displayName)).toBeInTheDocument();
     });
     it('renders long student names with truncation', function () {
       renderComponent({
-        student: _mockData.mockStudentLongName,
-        studentName: _mockData.mockStudentLongName.display_name
+        studentName: _mockData.mockStudentLongName.displayName
       });
       var link = _react2.screen.getByTestId('student-cell-link');
       expect(link).toBeInTheDocument();
@@ -148,8 +138,8 @@ describe('StudentPopover', function () {
       }, _callee4);
     })));
   });
-  describe('User Details Display', function () {
-    it('displays user details including avatar, name, course, and sections', /*#__PURE__*/(0, _asyncToGenerator2.default)(/*#__PURE__*/(0, _regenerator2.default)().m(function _callee5() {
+  describe('Header Display', function () {
+    it('displays avatar, name, description, and metadata', /*#__PURE__*/(0, _asyncToGenerator2.default)(/*#__PURE__*/(0, _regenerator2.default)().m(function _callee5() {
       return (0, _regenerator2.default)().w(function (_context5) {
         while (1) switch (_context5.n) {
           case 0:
@@ -158,26 +148,26 @@ describe('StudentPopover', function () {
             _context5.n = 1;
             return _react2.screen.findByTestId('lmgb-student-popover-avatar');
           case 1:
-            expect(_react2.screen.getAllByText(_mockData.mockStudent.display_name).length).toBeGreaterThan(0);
-            expect(_react2.screen.getByText('Introduction to Computer Science')).toBeInTheDocument();
-            expect(_react2.screen.getByText('Section A, Section B')).toBeInTheDocument();
+            expect(_react2.screen.getAllByText(_mockData.mockStudent.displayName).length).toBeGreaterThan(0);
+            expect(_react2.screen.getByText(_mockData2.mockCaptionDefault.description)).toBeInTheDocument();
+            expect(_react2.screen.getByText(_mockData2.mockCaptionDefault.metadata)).toBeInTheDocument();
           case 2:
             return _context5.a(2);
         }
       }, _callee5);
     })));
-    it('does not display sections when none exist', /*#__PURE__*/(0, _asyncToGenerator2.default)(/*#__PURE__*/(0, _regenerator2.default)().m(function _callee6() {
+    it('does not display metadata when not provided', /*#__PURE__*/(0, _asyncToGenerator2.default)(/*#__PURE__*/(0, _regenerator2.default)().m(function _callee6() {
       return (0, _regenerator2.default)().w(function (_context6) {
         while (1) switch (_context6.n) {
           case 0:
-            renderComponent({
-              userDetails: _mockData.mockUserDetailsNoSections
-            });
+            renderComponent((0, _objectSpread2.default)((0, _objectSpread2.default)({}, _mockData2.mockCaptionNoSections), {}, {
+              metadata: void 0
+            }));
             _react2.fireEvent.click(_react2.screen.getByTestId('student-cell-link'));
             _context6.n = 1;
             return _react2.screen.findByTestId('lmgb-student-popover-avatar');
           case 1:
-            expect(_react2.screen.queryByText(/Section/)).not.toBeInTheDocument();
+            expect(_react2.screen.queryByText(_mockData2.mockCaptionDefault.metadata)).not.toBeInTheDocument();
           case 2:
             return _context6.a(2);
         }
@@ -260,7 +250,7 @@ describe('StudentPopover', function () {
             _context0.n = 1;
             return _react2.screen.findByTestId('lmgb-student-popover-avatar');
           case 1:
-            expect(_react2.screen.getAllByText(_mockData.mockStudent.display_name).length).toBeGreaterThan(0);
+            expect(_react2.screen.getAllByText(_mockData.mockStudent.displayName).length).toBeGreaterThan(0);
           case 2:
             return _context0.a(2);
         }

package/lib/translated/en-CA/components/Gradebook/popovers/StudentPopover/index.js

@@ -21,9 +21,10 @@ var _uiSpinner = require("@instructure/ui-spinner");
 var _index = require("../../context/GradebookConfigContext/index.js");
 var _index2 = _interopRequireDefault(require("../../icons/MasteryLevelIcon/index.js"));
 var Header = function Header(_ref) {
-  var student = _ref.student,
+  var avatarUrl = _ref.avatarUrl,
     studentName = _ref.studentName,
-    userDetails = _ref.userDetails;
+    description = _ref.description,
+    metadata = _ref.metadata;
   return /*#__PURE__*/_react.default.createElement(_uiFlex.Flex, {
     gap: "small",
     alignItems: "start"
@@ -33,7 +34,7 @@ var Header = function Header(_ref) {
     as: "div",
     size: "large",
     name: studentName,
-    src: student.avatar_url,
+    src: avatarUrl,
     "data-testid": "lmgb-student-popover-avatar"
   })), /*#__PURE__*/_react.default.createElement(_uiFlex.Flex.Item, null, /*#__PURE__*/_react.default.createElement(_uiView.View, {
     display: "block",
@@ -42,13 +43,11 @@ var Header = function Header(_ref) {
   }, /*#__PURE__*/_react.default.createElement(_uiView.View, null, /*#__PURE__*/_react.default.createElement(_uiText.Text, {
     size: "content",
     weight: "bold"
-  }, /*#__PURE__*/_react.default.createElement(_uiTruncateText.TruncateText, null, studentName))), userDetails.course.name && /*#__PURE__*/_react.default.createElement(_uiView.View, null, /*#__PURE__*/_react.default.createElement(_uiText.Text, {
+  }, /*#__PURE__*/_react.default.createElement(_uiTruncateText.TruncateText, null, studentName))), description && /*#__PURE__*/_react.default.createElement(_uiView.View, null, /*#__PURE__*/_react.default.createElement(_uiText.Text, {
     size: "contentSmall"
-  }, /*#__PURE__*/_react.default.createElement(_uiTruncateText.TruncateText, null, userDetails.course.name))), userDetails.user.sections.length && /*#__PURE__*/_react.default.createElement(_uiView.View, null, /*#__PURE__*/_react.default.createElement(_uiText.Text, {
+  }, /*#__PURE__*/_react.default.createElement(_uiTruncateText.TruncateText, null, description))), metadata && /*#__PURE__*/_react.default.createElement(_uiView.View, null, /*#__PURE__*/_react.default.createElement(_uiText.Text, {
     size: "legend"
-  }, /*#__PURE__*/_react.default.createElement(_uiTruncateText.TruncateText, null, userDetails.user.sections.map(function (section) {
-    return section.name;
-  }).join(', ')))))));
+  }, /*#__PURE__*/_react.default.createElement(_uiTruncateText.TruncateText, null, metadata))))));
 };
 var MasteryScores = function MasteryScores(_ref2) {
   var _masteryLevelConfig$m, _masteryLevelConfig$m2;
@@ -167,10 +166,11 @@ var ErrorMessage = function ErrorMessage(_ref5) {
 };
 var StudentPopover = exports.StudentPopover = function StudentPopover(_ref6) {
   var studentName = _ref6.studentName,
-    headerOverride = _ref6.headerOverride,
-    student = _ref6.student,
-    userDetails = _ref6.userDetails,
+    avatarUrl = _ref6.avatarUrl,
+    description = _ref6.description,
+    metadata = _ref6.metadata,
     masteryScores = _ref6.masteryScores,
+    headerOverride = _ref6.headerOverride,
     masteryScoresOverride = _ref6.masteryScoresOverride,
     actionsOverride = _ref6.actionsOverride,
     studentGradesUrl = _ref6.studentGradesUrl,
@@ -190,7 +190,8 @@ var StudentPopover = exports.StudentPopover = function StudentPopover(_ref6) {
   return /*#__PURE__*/_react.default.createElement(_react.default.Fragment, null, /*#__PURE__*/_react.default.createElement(_uiPopover.Popover, {
     renderTrigger: /*#__PURE__*/_react.default.createElement(_uiLink.Link, {
       isWithinText: false,
-      "data-testid": "student-cell-link"
+      "data-testid": "student-cell-link",
+      "aria-haspopup": "dialog"
     }, /*#__PURE__*/_react.default.createElement(_uiTruncateText.TruncateText, null, studentName)),
     isShowingContent: isShowingContent,
     onShowContent: function onShowContent() {
@@ -229,9 +230,10 @@ var StudentPopover = exports.StudentPopover = function StudentPopover(_ref6) {
     direction: "column",
     alignItems: "start"
   }, /*#__PURE__*/_react.default.createElement(_uiFlex.Flex.Item, null, headerOverride !== null && headerOverride !== void 0 ? headerOverride : /*#__PURE__*/_react.default.createElement(Header, {
-    student: student,
+    avatarUrl: avatarUrl,
     studentName: studentName,
-    userDetails: userDetails
+    description: description,
+    metadata: metadata
   })), /*#__PURE__*/_react.default.createElement(_uiFlex.Flex.Item, null, masteryScoresOverride !== null && masteryScoresOverride !== void 0 ? masteryScoresOverride : /*#__PURE__*/_react.default.createElement(MasteryScores, {
     masteryScores: masteryScores
   }))), /*#__PURE__*/_react.default.createElement(Divider, null), actionsOverride !== null && actionsOverride !== void 0 ? actionsOverride : /*#__PURE__*/_react.default.createElement(Actions, {

package/lib/translated/en-CA/components/Gradebook/popovers/StudentPopover/index.stories.js

@@ -4,10 +4,10 @@ var _interopRequireDefault = require("@babel/runtime/helpers/interopRequireDefau
 Object.defineProperty(exports, "__esModule", {
   value: true
 });
-exports.default = exports.SingleSection = exports.NoSections = exports.NoScores = exports.ManySections = exports.LowPerformingStudent = exports.LongStudentName = exports.LongCourseName = exports.Loading = exports.HighPerformingStudent = exports.ErrorMessage = exports.Default = exports.CustomMasteryScores = exports.CustomMasteryLevelConfig = exports.CustomHeader = exports.CustomActions = void 0;
-var _objectSpread2 = _interopRequireDefault(require("@babel/runtime/helpers/objectSpread2"));
+exports.default = exports.WithLongMetaData = exports.WithLongDescription = exports.WithDescriptionOnly = exports.NoScores = exports.LowPerformingStudent = exports.LongStudentName = exports.Loading = exports.HighPerformingStudent = exports.ErrorMessage = exports.Default = exports.CustomMasteryScores = exports.CustomMasteryLevelConfig = exports.CustomHeader = exports.CustomActions = void 0;
 var _regenerator2 = _interopRequireDefault(require("@babel/runtime/helpers/regenerator"));
 var _asyncToGenerator2 = _interopRequireDefault(require("@babel/runtime/helpers/asyncToGenerator"));
+var _objectSpread2 = _interopRequireDefault(require("@babel/runtime/helpers/objectSpread2"));
 var _react = _interopRequireDefault(require("react"));
 var _addonActions = require("@storybook/addon-actions");
 var _testingLibrary = require("@storybook/testing-library");
@@ -16,7 +16,8 @@ var _uiFlex = require("@instructure/ui-flex");
 var _uiText = require("@instructure/ui-text");
 var _uiButtons = require("@instructure/ui-buttons");
 var _decorators = require("../../storybook/decorators.js");
-var _mockData = require("../../__mocks__/mockData.js");
+var _mockData = require("../../gradebook-table/StudentCell/__mocks__/mockData.js");
+var _mockData2 = require("./__mocks__/mockData.js");
 var _index = require("../../../../index.js");
 // Storybook's args type (Partial<Props>) doesn't handle discriminated
 // unions in StudentPopoverProps so define a flatten version of it
@@ -33,14 +34,14 @@ var meta = {
 };
 var _default = exports.default = meta;
 var createStoryArgs = function createStoryArgs(student) {
-  var masteryScores = arguments.length > 1 && arguments[1] !== void 0 ? arguments[1] : _mockData.mockMasteryScores;
-  return {
+  var masteryScores = arguments.length > 1 && arguments[1] !== void 0 ? arguments[1] : _mockData2.mockMasteryScores;
+  return (0, _objectSpread2.default)((0, _objectSpread2.default)({
     student: student,
-    studentName: student.display_name,
-    userDetails: _mockData.mockUserDetailsDefault,
+    studentName: student.displayName
+  }, _mockData2.mockCaptionDefault), {}, {
     masteryScores: masteryScores,
     studentGradesUrl: "/courses/123/grades/".concat(student.id)
-  };
+  });
 };
 var openPopover = /*#__PURE__*/function () {
   var _ref2 = (0, _asyncToGenerator2.default)(/*#__PURE__*/(0, _regenerator2.default)().m(function _callee(_ref) {
@@ -70,42 +71,39 @@ var Default = exports.Default = {
   play: openPopover
 };
 var HighPerformingStudent = exports.HighPerformingStudent = {
-  args: createStoryArgs(_mockData.mockStudent, _mockData.highMasteryScores),
+  args: createStoryArgs(_mockData.mockStudent, _mockData2.highMasteryScores),
   play: openPopover
 };
 var LowPerformingStudent = exports.LowPerformingStudent = {
-  args: createStoryArgs(_mockData.mockStudent, _mockData.lowMasteryScores),
+  args: createStoryArgs(_mockData.mockStudent, _mockData2.lowMasteryScores),
   play: openPopover
 };
 var NoScores = exports.NoScores = {
-  args: createStoryArgs(_mockData.mockStudent, _mockData.unassessedMasteryScores),
+  args: createStoryArgs(_mockData.mockStudent, _mockData2.unassessedMasteryScores),
   play: openPopover
 };
 var LongStudentName = exports.LongStudentName = {
   args: createStoryArgs(_mockData.mockStudentLongName),
   play: openPopover
 };
-var LongCourseName = exports.LongCourseName = {
-  args: (0, _objectSpread2.default)((0, _objectSpread2.default)({}, createStoryArgs((0, _mockData.createStudent)('4', 'Oliver', 'Brown'))), {}, {
-    userDetails: _mockData.mockUserDetailsLongCourseName
-  }),
-  play: openPopover
-};
-var ManySections = exports.ManySections = {
-  args: (0, _objectSpread2.default)((0, _objectSpread2.default)({}, createStoryArgs((0, _mockData.createStudent)('3', 'Emma', 'Wilson'))), {}, {
-    userDetails: _mockData.mockUserDetailsManySections
+var WithDescriptionOnly = exports.WithDescriptionOnly = {
+  args: (0, _objectSpread2.default)((0, _objectSpread2.default)({}, createStoryArgs(_mockData.mockStudent)), {}, {
+    description: 'Introduction to Computer Science',
+    metadata: void 0
   }),
   play: openPopover
 };
-var NoSections = exports.NoSections = {
-  args: (0, _objectSpread2.default)((0, _objectSpread2.default)({}, createStoryArgs((0, _mockData.createStudent)('6', 'Liam', 'Miller'))), {}, {
-    userDetails: _mockData.mockUserDetailsNoSections
+var WithLongDescription = exports.WithLongDescription = {
+  args: (0, _objectSpread2.default)((0, _objectSpread2.default)({}, createStoryArgs(_mockData.mockStudent)), {}, {
+    description: 'Advanced Placement European History: Renaissance to Modern Era',
+    metadata: 'AP Section'
   }),
   play: openPopover
 };
-var SingleSection = exports.SingleSection = {
-  args: (0, _objectSpread2.default)((0, _objectSpread2.default)({}, createStoryArgs((0, _mockData.createStudent)('2', 'Jane', 'Doe'))), {}, {
-    userDetails: _mockData.mockUserDetailsSingleSection
+var WithLongMetaData = exports.WithLongMetaData = {
+  args: (0, _objectSpread2.default)((0, _objectSpread2.default)({}, createStoryArgs(_mockData.mockStudent)), {}, {
+    description: 'World History',
+    metadata: 'Section A - Morning, Section B - Afternoon, Section C - Evening, Section D - Online'
   }),
   play: openPopover
 };
@@ -113,9 +111,10 @@ var CustomHeader = exports.CustomHeader = {
   args: createStoryArgs(_mockData.mockStudent),
   render: function render(args) {
     return /*#__PURE__*/_react.default.createElement(_index.StudentPopover, Object.assign({}, args, {
-      student: void 0,
+      avatarUrl: void 0,
       studentName: "John Doe",
-      userDetails: void 0,
+      description: void 0,
+      metadata: void 0,
       headerOverride: /*#__PURE__*/_react.default.createElement(_uiFlex.Flex, {
         padding: "0 0 medium 0"
       }, /*#__PURE__*/_react.default.createElement(_uiFlex.Flex.Item, null, /*#__PURE__*/_react.default.createElement(_uiText.Text, null, "Custom Header for ", args.studentName)))
@@ -161,7 +160,7 @@ var CustomActions = exports.CustomActions = {
 };
 var Loading = exports.Loading = {
   args: {
-    studentName: _mockData.mockStudent.display_name,
+    studentName: _mockData.mockStudent.displayName,
     studentGradesUrl: "/courses/123/grades/".concat(_mockData.mockStudent.id),
     isLoading: true
   },
@@ -169,7 +168,7 @@ var Loading = exports.Loading = {
 };
 var ErrorMessage = exports.ErrorMessage = {
   args: {
-    studentName: _mockData.mockStudent.display_name,
+    studentName: _mockData.mockStudent.displayName,
     studentGradesUrl: "/courses/123/grades/".concat(_mockData.mockStudent.id),
     error: 'Failed to load student details. Please try again.'
   },

package/lib/translated/en-CA/components/Gradebook/storybook/decorators.js

@@ -4,7 +4,7 @@ var _interopRequireDefault = require("@babel/runtime/helpers/interopRequireDefau
 Object.defineProperty(exports, "__esModule", {
   value: true
 });
-exports.StoryWrapper = void 0;
+exports.StoryWrapper = exports.ExampleSettingsTrayContent = void 0;
 var _regenerator2 = _interopRequireDefault(require("@babel/runtime/helpers/regenerator"));
 var _asyncToGenerator2 = _interopRequireDefault(require("@babel/runtime/helpers/asyncToGenerator"));
 var _objectSpread2 = _interopRequireDefault(require("@babel/runtime/helpers/objectSpread2"));
@@ -13,19 +13,10 @@ var _addonActions = require("@storybook/addon-actions");
 var _uiCheckbox = require("@instructure/ui-checkbox");
 var _uiFlex = require("@instructure/ui-flex");
 var _NameDisplaySelector = require("../toolbar/SettingsTray/NameDisplaySelector.js");
-var _index = require("../popovers/StudentPopover/index.js");
 var _constants = require("../../../util/gradebook/constants.js");
 var _GradebookApp = require("../GradebookApp.js");
-var _mockData = require("../__mocks__/mockData.js");
-var StudentPopoverWrapper = function StudentPopoverWrapper(props) {
-  return /*#__PURE__*/_react.default.createElement(_index.StudentPopover, Object.assign({}, props, {
-    student: props.student || (0, _mockData.createStudent)('1', 'John', 'Doe'),
-    studentName: props.studentName || 'John Doe',
-    studentGradesUrl: "/test-url",
-    userDetails: _mockData.mockUserDetailsDefault
-  }));
-};
-var ExampleSettingsTrayContent = function ExampleSettingsTrayContent(_ref) {
+var _index = require("../context/GradebookAppContext/index.js");
+var ExampleSettingsTrayContent = exports.ExampleSettingsTrayContent = function ExampleSettingsTrayContent(_ref) {
   var settings = _ref.settings,
     _onChange = _ref.onChange;
   return /*#__PURE__*/_react.default.createElement(_uiFlex.Flex, {
@@ -52,17 +43,12 @@ var ExampleSettingsTrayContent = function ExampleSettingsTrayContent(_ref) {
 };
 var StoryWrapper = exports.StoryWrapper = function StoryWrapper(_ref2) {
   var children = _ref2.children,
-    masteryLevelConfig = _ref2.masteryLevelConfig,
-    components = _ref2.components;
-  var defaultComponents = {
-    StudentPopover: StudentPopoverWrapper,
-    SettingsTrayContent: ExampleSettingsTrayContent
-  };
+    masteryLevelConfig = _ref2.masteryLevelConfig;
   return /*#__PURE__*/_react.default.createElement(_GradebookApp.GradebookApp, {
     config: {
-      components: (0, _objectSpread2.default)((0, _objectSpread2.default)({}, defaultComponents), components),
       masteryLevelConfig: masteryLevelConfig
-    },
+    }
+  }, /*#__PURE__*/_react.default.createElement(_index.GradebookAppProvider, {
     settings: {
       settings: {
         showStudentNames: true,
@@ -92,5 +78,5 @@ var StoryWrapper = exports.StoryWrapper = function StoryWrapper(_ref2) {
         return onSave;
       }()
     }
-  }, children);
+  }, children));
 };

package/lib/translated/en-CA/components/Gradebook/table/Cell/index.js

@@ -1,16 +1,17 @@
 "use strict";
 
 var _interopRequireDefault = require("@babel/runtime/helpers/interopRequireDefault").default;
+var _interopRequireWildcard = require("@babel/runtime/helpers/interopRequireWildcard").default;
 Object.defineProperty(exports, "__esModule", {
   value: true
 });
 exports.default = exports.Cell = void 0;
 var _slicedToArray2 = _interopRequireDefault(require("@babel/runtime/helpers/slicedToArray"));
 var _objectWithoutProperties2 = _interopRequireDefault(require("@babel/runtime/helpers/objectWithoutProperties"));
-var _react = _interopRequireDefault(require("react"));
+var _react = _interopRequireWildcard(require("react"));
 var _uiView = require("@instructure/ui-view");
 var _excluded = ["id", "children", "width", "boxShadow", "isSticky", "isStacked", "header"];
-var Cell = exports.Cell = function Cell(_ref) {
+var CellComponent = function CellComponent(_ref) {
   var id = _ref.id,
     children = _ref.children,
     width = _ref.width,
@@ -19,19 +20,34 @@ var Cell = exports.Cell = function Cell(_ref) {
     isStacked = _ref.isStacked,
     header = _ref.header,
     props = (0, _objectWithoutProperties2.default)(_ref, _excluded);
-  var _React$useState = _react.default.useState(false),
-    _React$useState2 = (0, _slicedToArray2.default)(_React$useState, 2),
-    focus = _React$useState2[0],
-    setFocus = _React$useState2[1];
-  var cellRef = _react.default.useRef(null);
-  var handleBlur = function handleBlur(event) {
+  var _useState = (0, _react.useState)(false),
+    _useState2 = (0, _slicedToArray2.default)(_useState, 2),
+    focus = _useState2[0],
+    setFocus = _useState2[1];
+  var cellRef = (0, _react.useRef)(null);
+  var handleBlur = (0, _react.useCallback)(function (event) {
     var _cellRef$current;
     var relatedTarget = event.relatedTarget;
     if (relatedTarget && (_cellRef$current = cellRef.current) !== null && _cellRef$current !== void 0 && _cellRef$current.contains(relatedTarget)) {
       return;
     }
     setFocus(false);
-  };
+  }, []);
+  var handleFocus = (0, _react.useCallback)(function () {
+    return setFocus(true);
+  }, []);
+  var elementRef = (0, _react.useCallback)(function (el) {
+    if (el instanceof HTMLElement) {
+      cellRef.current = el;
+    }
+  }, []);
+  var themeOverride = (0, _react.useMemo)(function () {
+    return function (_componentTheme, currentTheme) {
+      return {
+        shadowAbove: boxShadow !== null && boxShadow !== void 0 ? boxShadow : "2px 0 0 0 ".concat(currentTheme.colors.contrasts.grey1214)
+      };
+    };
+  }, [boxShadow]);
   return /*#__PURE__*/_react.default.createElement(_uiView.View, Object.assign({
     id: id,
     width: width,
@@ -45,20 +61,55 @@ var Cell = exports.Cell = function Cell(_ref) {
     insetInlineStart: isSticky ? '0' : void 0,
     background: isSticky && !props.background ? 'primary' : props.background,
     focusPosition: "inset",
-    elementRef: function elementRef(el) {
-      if (el instanceof HTMLElement) {
-        cellRef.current = el;
-      }
-    },
-    onFocus: function onFocus() {
-      return setFocus(true);
-    },
+    elementRef: elementRef,
+    onFocus: handleFocus,
     onBlur: handleBlur,
-    themeOverride: function themeOverride(_componentTheme, currentTheme) {
-      return {
-        shadowAbove: boxShadow !== null && boxShadow !== void 0 ? boxShadow : "2px 0 0 0 ".concat(currentTheme.colors.contrasts.grey1214)
-      };
-    }
+    themeOverride: themeOverride
   }, props), header ? typeof header === 'function' ? header() : header : null, typeof children === 'function' ? children(focus) : children);
 };
+
+// Custom comparison function for React.memo
+// We need to compare most props, but be smart about function props
+var arePropsEqual = function arePropsEqual(prevProps, nextProps) {
+  // If the id changed, definitely re-render
+  if (prevProps.id !== nextProps.id) return false;
+
+  // Check data attributes that affect keyboard navigation
+  if (prevProps['data-row-index'] !== nextProps['data-row-index']) return false;
+  if (prevProps['data-col-key'] !== nextProps['data-col-key']) return false;
+  if (prevProps['data-cell-id'] !== nextProps['data-cell-id']) return false;
+
+  // Check visual props
+  if (prevProps.width !== nextProps.width) return false;
+  if (prevProps.boxShadow !== nextProps.boxShadow) return false;
+  if (prevProps.isSticky !== nextProps.isSticky) return false;
+  if (prevProps.isStacked !== nextProps.isStacked) return false;
+
+  // Check other props that could change
+  if (prevProps.tabIndex !== nextProps.tabIndex) return false;
+
+  // onKeyDown should be stable (useCallback), but check just in case
+  if (prevProps.onKeyDown !== nextProps.onKeyDown) return false;
+
+  // For function children (render functions), we can't compare the function itself
+  // Instead, we rely on the fact that if all other props (especially data-cell-id
+  // which includes col.key and row data) are the same, the function would produce
+  // the same output. This is safe because:
+  // - row data doesn't change during column reorder
+  // - col.key is stable per column
+  // - The function just calls col.render with that data
+  var prevChildrenIsFunction = typeof prevProps.children === 'function';
+  var nextChildrenIsFunction = typeof nextProps.children === 'function';
+  if (prevChildrenIsFunction !== nextChildrenIsFunction) return false;
+
+  // For non-function children, compare directly
+  if (!nextChildrenIsFunction && prevProps.children !== nextProps.children) return false;
+
+  // For function children, skip comparison - they're always new references but produce same output
+  // when all other props are the same
+
+  return true;
+};
+var Cell = exports.Cell = /*#__PURE__*/_react.default.memo(CellComponent, arePropsEqual);
+Cell.displayName = 'Cell';
 var _default = exports.default = Cell;

package/lib/translated/en-CA/components/Gradebook/table/ColumnHeader/index.js

@@ -11,7 +11,40 @@ var _react = _interopRequireWildcard(require("react"));
 var _uiView = require("@instructure/ui-view");
 var _excluded = ["children", "isSticky", "connectDragSource", "connectDragPreview", "connectDropTarget", "isDragging", "isStacked", "data-cell-id"],
   _excluded2 = ["onMove", "onDragEnd", "itemId", "index", "type", "component"];
-var ColumnHeader = exports.ColumnHeader = function ColumnHeader(_ref) {
+// Custom comparison function to prevent re-renders when column order changes
+// but individual column data hasn't changed
+var arePropsEqual = function arePropsEqual(prevProps, nextProps) {
+  // Check stable identifier
+  if (prevProps.id !== nextProps.id) return false;
+  if (prevProps['data-cell-id'] !== nextProps['data-cell-id']) return false;
+  if (prevProps['data-col-key'] !== nextProps['data-col-key']) return false;
+
+  // Check visual props
+  if (prevProps.width !== nextProps.width) return false;
+  if (prevProps.isSticky !== nextProps.isSticky) return false;
+  if (prevProps.isStacked !== nextProps.isStacked) return false;
+
+  // Check keyboard navigation
+  if (prevProps.onKeyDown !== nextProps.onKeyDown) return false;
+  if (prevProps.tabIndex !== nextProps.tabIndex) return false;
+
+  // Check drag-drop state (these should be stable via useCallback)
+  if (prevProps.connectDragSource !== nextProps.connectDragSource) return false;
+  if (prevProps.connectDragPreview !== nextProps.connectDragPreview) return false;
+  if (prevProps.connectDropTarget !== nextProps.connectDropTarget) return false;
+  if (prevProps.isDragging !== nextProps.isDragging) return false;
+
+  // onMove and onDragEnd should be stable (useCallback in stories/parent)
+  // but we don't need to check them since they're filtered out before DOM
+
+  // Skip children comparison - children might be new React elements on each render,
+  // but if all stable identifiers (id, data-cell-id, data-col-key) are the same,
+  // the column hasn't actually changed and we can safely skip re-rendering.
+  // This matches Cell's approach of relying on stable props rather than comparing children.
+
+  return true;
+};
+var ColumnHeaderComponent = function ColumnHeaderComponent(_ref) {
   var children = _ref.children,
     isSticky = _ref.isSticky,
     connectDragSource = _ref.connectDragSource,
@@ -43,13 +76,21 @@ var ColumnHeader = exports.ColumnHeader = function ColumnHeader(_ref) {
       });
     }
   }, [connectDragPreview]);
-  var headerContent = /*#__PURE__*/_react.default.createElement("div", {
-    style: {
+  var headerContentStyle = _react.default.useMemo(function () {
+    return {
       opacity: isDragging ? 0.5 : 1,
       transition: 'opacity 0.15s ease-in-out',
       width: cleanViewProps.width,
       cursor: connectDragSource ? 'grab' : 'default'
-    },
+    };
+  }, [isDragging, cleanViewProps.width, connectDragSource]);
+  var elementRef = _react.default.useCallback(function (ref) {
+    if (connectDropTarget && ref instanceof HTMLElement) {
+      connectDropTarget(ref);
+    }
+  }, [connectDropTarget]);
+  var headerContent = /*#__PURE__*/_react.default.createElement("div", {
+    style: headerContentStyle,
     "data-testid": "column-header-content"
   }, children);
   var content = /*#__PURE__*/_react.default.createElement(_uiView.View, Object.assign({
@@ -57,11 +98,7 @@ var ColumnHeader = exports.ColumnHeader = function ColumnHeader(_ref) {
     as: isStacked ? 'div' : 'th',
     scope: "col",
     focusPosition: "inset",
-    elementRef: function elementRef(ref) {
-      if (connectDropTarget && ref instanceof HTMLElement) {
-        connectDropTarget(ref);
-      }
-    },
+    elementRef: elementRef,
     position: isSticky ? 'sticky' : 'static',
     insetInlineStart: isSticky ? '0' : void 0,
     background: !cleanViewProps.background ? 'primary' : cleanViewProps.background,
@@ -69,4 +106,6 @@ var ColumnHeader = exports.ColumnHeader = function ColumnHeader(_ref) {
     stacking: "above"
   }, cleanViewProps), connectDragSource ? connectDragSource(headerContent) : headerContent);
   return content;
-};
+};
+var ColumnHeader = exports.ColumnHeader = /*#__PURE__*/_react.default.memo(ColumnHeaderComponent, arePropsEqual);
+ColumnHeader.displayName = 'ColumnHeader';

package/lib/translated/en-CA/components/Gradebook/table/Row/index.js

@@ -9,7 +9,7 @@ var _react = _interopRequireWildcard(require("react"));
 var _uiView = require("@instructure/ui-view");
 var _uiTable = require("@instructure/ui-table");
 var _uiReactUtils = require("@instructure/ui-react-utils");
-var Row = exports.Row = function Row(_ref) {
+var Row = exports.Row = /*#__PURE__*/_react.default.memo(function (_ref) {
   var children = _ref.children,
     setRef = _ref.setRef;
   var context = (0, _react.useContext)(_uiTable.TableContext);
@@ -32,4 +32,5 @@ var Row = exports.Row = function Row(_ref) {
     }
     return child;
   }));
-};
+});
+Row.displayName = 'Row';

package/lib/translated/en-CA/components/Gradebook/table/RowHeader/index.js

@@ -9,7 +9,7 @@ var _objectWithoutProperties2 = _interopRequireDefault(require("@babel/runtime/h
 var _react = _interopRequireDefault(require("react"));
 var _uiView = require("@instructure/ui-view");
 var _excluded = ["children", "isSticky", "isStacked", "data-cell-id"];
-var RowHeader = exports.RowHeader = function RowHeader(_ref) {
+var RowHeader = exports.RowHeader = /*#__PURE__*/_react.default.memo(function (_ref) {
   var children = _ref.children,
     isSticky = _ref.isSticky,
     isStacked = _ref.isStacked,
@@ -28,4 +28,5 @@ var RowHeader = exports.RowHeader = function RowHeader(_ref) {
     focusPosition: "inset",
     "data-cell-id": dataCellId
   }, props), children);
-};
+});
+RowHeader.displayName = 'RowHeader';