Skip to content

Commit

Permalink
refactor: Remove unsupported line height Aztec warning
Browse files Browse the repository at this point in the history
While arguably educational, this warning increases the amount of noise
within the server log, which can make it difficult to debug errors,
warnings, or intentionally logged values.
  • Loading branch information
dcalhoun committed Aug 1, 2023
1 parent afc73ee commit 286fbbe
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 4 deletions.
3 changes: 0 additions & 3 deletions packages/react-native-aztec/src/AztecView.js
Original file line number Diff line number Diff line change
Expand Up @@ -256,9 +256,6 @@ class AztecView extends Component {

if ( style.hasOwnProperty( 'lineHeight' ) ) {
delete style.lineHeight;
window.console.warn(
"Removing lineHeight style as it's not supported by native AztecView"
);
// Prevents passing line-height within styles to avoid a crash due to values without units
// We now support this but passing line-height as a prop instead.
}
Expand Down
1 change: 0 additions & 1 deletion packages/react-native-editor/src/setup.js
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,6 @@ import setupApiFetch from './api-fetch-setup';
const reactNativeSetup = () => {
LogBox.ignoreLogs( [
'Require cycle:', // TODO: Refactor to remove require cycles
'lineHeight', // TODO: Remove lineHeight warning from Aztec
/**
* TODO: Migrate to @gorhom/bottom-sheet or replace usage of
* LayoutAnimation to Animated. KeyboardAvoidingView's usage of
Expand Down

0 comments on commit 286fbbe

Please sign in to comment.