Removed console warning

Author: thatblindgeye

packages/react-core/src/components/Page/PageGroup.tsx

@@ -46,13 +46,6 @@ export const PageGroup = ({
   isNoPlainOnGlass = false,
   ...props
 }: PageGroupProps) => {
-  if (isPlain && isNoPlainOnGlass) {
-    // eslint-disable-next-line no-console
-    console.warn(
-      `PageGroup: When both isPlain and isNoPlainOnGlass are true, isPlain will take precedence and isNoPlainOnGlass will have no effect. It's recommended to pass only one prop according to the current theme.`
-    );
-  }
-
   const { height, getVerticalBreakpoint } = useContext(PageContext);
 
   useEffect(() => {

packages/react-core/src/components/Page/PageSection.tsx

@@ -106,13 +106,6 @@ export const PageSection: React.FunctionComponent<PageSectionProps> = ({
   isNoPlainOnGlass = false,
   ...props
 }: PageSectionProps) => {
-  if (isPlain && isNoPlainOnGlass) {
-    // eslint-disable-next-line no-console
-    console.warn(
-      `PageSection: When both isPlain and isNoPlainOnGlass are true, isPlain will take precedence and isNoPlainOnGlass will have no effect. It's recommended to pass only one prop according to the current theme.`
-    );
-  }
-
   const { height, getVerticalBreakpoint } = useContext(PageContext);
 
   useEffect(() => {

packages/react-core/src/components/Page/__tests__/PageGroup.test.tsx

@@ -101,33 +101,3 @@ test(`Renders with ${styles.modifiers.noPlainOnGlass} class when isPlain is true
 
   expect(screen.getByText('test')).toHaveClass(styles.modifiers.noPlainOnGlass);
 });
-
-test('Does not log a warning when only isPlain is passed', () => {
-  const consoleWarning = jest.spyOn(console, 'warn').mockImplementation();
-
-  render(<PageGroup isPlain>test</PageGroup>);
-
-  expect(consoleWarning).not.toHaveBeenCalled();
-});
-
-test('Does not log a warning when only isNoPlainOnGlass is passed', () => {
-  const consoleWarning = jest.spyOn(console, 'warn').mockImplementation();
-
-  render(<PageGroup isNoPlainOnGlass>test</PageGroup>);
-
-  expect(consoleWarning).not.toHaveBeenCalled();
-});
-
-test('Logs a warning when both isPlain and isNoPlainOnGlass are passed', () => {
-  const consoleWarning = jest.spyOn(console, 'warn').mockImplementation();
-
-  render(
-    <PageGroup isPlain isNoPlainOnGlass>
-      test
-    </PageGroup>
-  );
-
-  expect(consoleWarning).toHaveBeenCalledWith(
-    `PageGroup: When both isPlain and isNoPlainOnGlass are true, isPlain will take precedence and isNoPlainOnGlass will have no effect. It's recommended to pass only one prop according to the current theme.`
-  );
-});

packages/react-core/src/components/Page/__tests__/PageSection.test.tsx

@@ -199,33 +199,3 @@ test(`Renders with ${styles.modifiers.noPlainOnGlass} class when isPlain is true
 
   expect(screen.getByText('test')).toHaveClass(styles.modifiers.noPlainOnGlass);
 });
-
-test('Does not log a warning when only isPlain is passed', () => {
-  const consoleWarning = jest.spyOn(console, 'warn').mockImplementation();
-
-  render(<PageSection isPlain>test</PageSection>);
-
-  expect(consoleWarning).not.toHaveBeenCalled();
-});
-
-test('Does not log a warning when only isNoPlainOnGlass is passed', () => {
-  const consoleWarning = jest.spyOn(console, 'warn').mockImplementation();
-
-  render(<PageSection isNoPlainOnGlass>test</PageSection>);
-
-  expect(consoleWarning).not.toHaveBeenCalled();
-});
-
-test('Logs a warning when both isPlain and isNoPlainOnGlass are passed', () => {
-  const consoleWarning = jest.spyOn(console, 'warn').mockImplementation();
-
-  render(
-    <PageSection isPlain isNoPlainOnGlass>
-      test
-    </PageSection>
-  );
-
-  expect(consoleWarning).toHaveBeenCalledWith(
-    `PageSection: When both isPlain and isNoPlainOnGlass are true, isPlain will take precedence and isNoPlainOnGlass will have no effect. It's recommended to pass only one prop according to the current theme.`
-  );
-});