change function name to isVisibleGivenCustomConfig

Author: VeraZab

src/components/containers/PlotlySection.js

@@ -3,7 +3,7 @@ import PropTypes from 'prop-types';
 import {
   containerConnectedContextTypes,
   unpackPlotProps,
-  validateVisibilityWithCustomConfig,
+  isVisibleGivenCustomConfig,
 } from '../../lib';
 
 export class Section extends Component {
@@ -49,7 +49,7 @@ export default class PlotlySection extends Section {
 
   determineVisibility(nextProps, nextContext) {
     const {isVisible} = unpackPlotProps(nextProps, nextContext);
-    this.sectionVisible = validateVisibilityWithCustomConfig(isVisible, nextProps, nextContext);
+    this.sectionVisible = isVisibleGivenCustomConfig(isVisible, nextProps, nextContext);
 
     React.Children.forEach(nextProps.children, child => {
       if (!child || this.sectionVisible) {
@@ -62,7 +62,7 @@ export default class PlotlySection extends Section {
           child.type.modifyPlotProps(child.props, nextContext, plotProps);
         }
 
-        this.sectionVisible = validateVisibilityWithCustomConfig(
+        this.sectionVisible = isVisibleGivenCustomConfig(
           this.sectionVisible || plotProps.isVisible,
           child.props,
           nextContext,

src/lib/connectToContainer.js

@@ -1,6 +1,6 @@
 import React, {Component} from 'react';
 import PropTypes from 'prop-types';
-import unpackPlotProps, {validateVisibilityWithCustomConfig} from './unpackPlotProps';
+import unpackPlotProps, {isVisibleGivenCustomConfig} from './unpackPlotProps';
 import {getDisplayName} from '../lib';
 
 export const containerConnectedContextTypes = {
@@ -68,7 +68,7 @@ export default function connectToContainer(WrappedComponent, config = {}) {
         WrappedComponent && WrappedComponent.displayName ? WrappedComponent.displayName : null;
 
       if (
-        validateVisibilityWithCustomConfig(
+        isVisibleGivenCustomConfig(
           props.isVisible,
           props,
           this.context,

src/lib/index.js

@@ -26,7 +26,7 @@ import tinyColor from 'tinycolor2';
 import unpackPlotProps, {
   computeCustomConfigVisibility,
   hasValidCustomConfigVisibilityRules,
-  validateVisibilityWithCustomConfig,
+  isVisibleGivenCustomConfig,
 } from './unpackPlotProps';
 import walkObject, {isPlainObject} from './walkObject';
 import {traceTypeToPlotlyInitFigure, plotlyTraceToCustomTrace} from './customTraceType';
@@ -267,6 +267,6 @@ export {
   transpose,
   unpackPlotProps,
   upperCase,
-  validateVisibilityWithCustomConfig,
+  isVisibleGivenCustomConfig,
   walkObject,
 };

src/lib/unpackPlotProps.js

@@ -91,12 +91,7 @@ export function computeCustomConfigVisibility(props, customConfig, wrappedCompon
   return isVisible;
 }
 
-export function validateVisibilityWithCustomConfig(
-  initial,
-  nextProps,
-  nextContext,
-  componentDisplayName
-) {
+export function isVisibleGivenCustomConfig(initial, nextProps, nextContext, componentDisplayName) {
   let show = initial;
   if (show && nextContext.hasValidCustomConfigVisibilityRules) {
     show = computeCustomConfigVisibility(nextProps, nextContext.customConfig, componentDisplayName);