CONSOLE-4447: Remove orphaned CSS classes from _modals.scss

Removes unused CSS classes that were only used by the deprecated modal components:
- .co-overlay (used by old ModalWrapper component)
- .modal-footer (replaced by PatternFly ModalFooter)
- .modal-paragraph (no longer needed with PatternFly modals)

All remaining classes in _modals.scss are still in use by existing modals or integration tests.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>

Author: rhamilto

frontend/packages/console-app/src/actions/hooks/useCommonActions.ts

@@ -135,7 +135,6 @@ export const useCommonActions = <T extends readonly CommonActionCreator[]>(
           launchModal(LazyTolerationsModalOverlay, {
             resourceKind: kind,
             resource,
-            modalClassName: 'modal-lg',
           }),
         accessReview: asAccessReview(kind as K8sModel, resource as K8sResourceKind, 'patch'),
       }),

frontend/packages/console-dynamic-plugin-sdk/CHANGELOG-core.md

@@ -15,6 +15,7 @@ table in [Console dynamic plugins README](./README.md).
 - **Breaking**: Changed `AlertAction` extension to use `LaunchOverlay` instead of `LaunchModal`. The `action` callback parameter is now typed as `LaunchOverlay`, which does not support the optional `id` parameter that `LaunchModal` had. Plugins implementing `console.alert-action` extensions must update their action callbacks accordingly. ([CONSOLE-4447])
 - **Breaking**: Removed `pluginID` from the result in `useResolvedExtensions` hook ([CONSOLE-3769], [#15904])
 - **Breaking**: Removed `AppInitSDK` and `useReduxStore` in `app` directory ([CONSOLE-5063], [#16019])
+- **Breaking**: Removed CSS classes from Console core that were used for deprecated modal components. Plugins that reference these classes in their custom CSS should migrate to PatternFly v6 modal classes (e.g., `.pf-v6-c-modal-box__body`) or use data-test attributes. Removed classes: `.co-overlay`, `.modal-footer`, `.modal-paragraph`, `.modal-header`, `.modal-body`, `.modal-body-border`, `.modal-body-content`, `.modal-content`, `.modal-dialog`, `.co-modal-table`, `.modal__inline-resource-link` ([CONSOLE-4447])
 - **Deprecated**: `useUserSettings` hook has been renamed to `useUserPreference` for consistency ([OCPBUGS-44612], [#16057])
 - **Type breaking**: Changed `useDeleteModal` hook's `redirectTo` parameter type from `LocationDescriptor` (from `history`) to `To` (from `react-router-dom-v5-compat`) ([CONSOLE-4990], [#15959])
 - **Type breaking**: Changed `FileUploadHandler` return type from `void` to `To | void`. Handlers can now return a path (from `react-router-dom-v5-compat`) for programmatic navigation instead of calling history methods directly ([CONSOLE-4990], [#15959])
@@ -180,6 +181,7 @@ table in [Console dynamic plugins README](./README.md).
 [CONSOLE-4400]: https://issues.redhat.com/browse/CONSOLE-4400
 [CONSOLE-4402]: https://issues.redhat.com/browse/CONSOLE-4402
 [CONSOLE-4407]: https://issues.redhat.com/browse/CONSOLE-4407
+[CONSOLE-4447]: https://issues.redhat.com/browse/CONSOLE-4447
 [CONSOLE-4499]: https://issues.redhat.com/browse/CONSOLE-4499
 [CONSOLE-4576]: https://issues.redhat.com/browse/CONSOLE-4576
 [CONSOLE-4630]: https://issues.redhat.com/browse/CONSOLE-4630

frontend/packages/operator-lifecycle-manager/src/components/modals/uninstall-operator-modal.scss

@@ -16,8 +16,3 @@
 .co-operator-uninstall__operands-section loading-skeleton--table::after {
   min-height: 160px;
 }
-
-.co-catalog-install-modal .modal-header .co-clusterserviceversion-logo__icon {
-  border: var(--pf-t--global--border--width--regular) solid
-    var(--pf-t--global--border--color--default);
-}

frontend/public/components/factory/_modal-wrapper.scss

@@ -152,7 +152,7 @@ export const DeleteModal = (props: DeleteModalProps) => {
                 <Trans t={t} ns="public">
                   This resource is managed by{' '}
                   <ResourceLink
-                    className="modal__inline-resource-link"
+                    className="pf-v6-u-mr-0"
                     inline
                     kind={referenceForOwnerRef(owner)}
                     name={owner.name}

frontend/public/components/modals/_modals.scss

@@ -39,7 +39,7 @@ const ManagedResourceSaveModal: FC<ManagedResourceSaveModalProps> = (props) => {
             <Trans t={t} ns="public">
               This resource is managed by{' '}
               <ResourceLink
-                className="modal__inline-resource-link"
+                className="pf-v6-u-mr-0"
                 inline
                 kind={referenceForOwnerRef(owner)}
                 name={owner.name}

frontend/public/components/modals/delete-modal.tsx

@@ -88,12 +88,7 @@ const TaintsModal = (props: TaintsModalProps) => {
           {_.isEmpty(taints) ? (
             <EmptyBox label={t('Taints')} />
           ) : (
-            <Table
-              aria-label={t('Taints')}
-              variant="compact"
-              borders={false}
-              className="co-modal-table"
-            >
+            <Table aria-label={t('Taints')} variant="compact" borders={false}>
               <Thead>
                 <Tr>
                   <Th>{t('Key')}</Th>

frontend/public/components/modals/managed-resource-save-modal.tsx

@@ -118,12 +118,7 @@ const TolerationsModal = (props: TolerationsModalProps) => {
           {_.isEmpty(tolerations) ? (
             <EmptyBox label={t('public~Tolerations')} />
           ) : (
-            <Table
-              aria-label={t('public~Tolerations')}
-              variant="compact"
-              borders={false}
-              className="co-modal-table"
-            >
+            <Table aria-label={t('public~Tolerations')} variant="compact" borders={false}>
               <Thead>
                 <Tr>
                   <Th>{t('public~Key')}</Th>

frontend/public/components/modals/taints-modal.tsx

@@ -67,7 +67,6 @@
 @import 'components/edit-yaml';
 @import 'components/graphs/graphs';
 @import 'components/masthead';
-@import 'components/modals/modals';
 @import 'components/environment';
 @import 'components/utils/name-value-editor';
 @import 'components/utils/value-from-pair';