fix: Nits on the code

Author: ChrisChV

src/course-unit/unit-sidebar/unit-info/UnitInfoSidebar.test.tsx

@@ -200,6 +200,30 @@ describe('<UnitInfoSidebar /> - rendering', () => {
     expect(screen.getByText('Content Summary')).toBeInTheDocument();
     expect(screen.queryByText('Unit Content Summary')).not.toBeInTheDocument();
   });
+
+  it('shows Settings tab when isVertical is true', () => {
+    unitSidebarContext.useUnitSidebarContext.mockReturnValue({
+      currentTabKey: 'details',
+      setCurrentTabKey: jest.fn(),
+      isVertical: true,
+    });
+
+    renderComponent();
+
+    expect(screen.getByRole('tab', { name: 'Settings' })).toBeInTheDocument();
+  });
+
+  it('hides Settings tab when isVertical is false', () => {
+    unitSidebarContext.useUnitSidebarContext.mockReturnValue({
+      currentTabKey: 'details',
+      setCurrentTabKey: jest.fn(),
+      isVertical: false,
+    });
+
+    renderComponent();
+
+    expect(screen.queryByRole('tab', { name: 'Settings' })).not.toBeInTheDocument();
+  });
 });
 
 describe('<UnitInfoSidebar /> - menu behavior', () => {

src/course-unit/unit-sidebar/unit-info/UnitInfoSidebar.tsx

@@ -121,6 +121,7 @@ export const UnitInfoSidebar = () => {
   const currentItemData = useSelector(getCourseUnitData);
   const {
     currentTabKey,
+    isVertical,
     setCurrentTabKey,
   } = useUnitSidebarContext();
   const { showToast } = useContext(ToastContext);
@@ -228,14 +229,16 @@ export const UnitInfoSidebar = () => {
             <UnitInfoDetails />
           </div>
         </Tab>
-        <Tab
-          eventKey="settings"
-          title={intl.formatMessage(messages.sidebarInfoSettingsTab)}
-        >
-          <div className="mt-4">
-            <UnitInfoSettings />
-          </div>
-        </Tab>
+        {isVertical && (
+          <Tab
+            eventKey="settings"
+            title={intl.formatMessage(messages.sidebarInfoSettingsTab)}
+          >
+            <div className="mt-4">
+              <UnitInfoSettings />
+            </div>
+          </Tab>
+        )}
       </Tabs>
       <DeleteModal
         isOpen={isDeleteModalOpen}