From d3e9ae9701161deb20d79829a825e52958617292 Mon Sep 17 00:00:00 2001 From: Alex P Date: Wed, 13 Jun 2018 11:28:25 -0700 Subject: [PATCH 001/100] Introduce presentational components for page header & title --- ui/src/shared/components/PageHeader.tsx | 25 ++++++++++++++++++++ ui/src/shared/components/PageHeaderTitle.tsx | 11 +++++++++ 2 files changed, 36 insertions(+) create mode 100644 ui/src/shared/components/PageHeader.tsx create mode 100644 ui/src/shared/components/PageHeaderTitle.tsx diff --git a/ui/src/shared/components/PageHeader.tsx b/ui/src/shared/components/PageHeader.tsx new file mode 100644 index 000000000..a73d1bde0 --- /dev/null +++ b/ui/src/shared/components/PageHeader.tsx @@ -0,0 +1,25 @@ +import React, {SFC, ReactElement} from 'react' +import classnames from 'classnames' + +interface Props { + renderLeft: () => ReactElement + renderRight?: () => ReactElement + fullWidth: boolean +} + +const PageHeader: SFC = ({renderLeft, renderRight, fullWidth}) => { + const className = classnames('page-header', {'full-width': fullWidth}) + + return ( +
+
+
{renderLeft()}
+ {renderRight && ( +
{renderRight()}
+ )} +
+
+ ) +} + +export default PageHeader diff --git a/ui/src/shared/components/PageHeaderTitle.tsx b/ui/src/shared/components/PageHeaderTitle.tsx new file mode 100644 index 000000000..46cdc6a23 --- /dev/null +++ b/ui/src/shared/components/PageHeaderTitle.tsx @@ -0,0 +1,11 @@ +import React, {SFC} from 'react' + +interface Props { + title: string +} + +const PageTitle: SFC = ({title}) => ( +

{title}

+) + +export default PageTitle From d1cedfd45eacf9cf03be926b9913f6d6962f36f5 Mon Sep 17 00:00:00 2001 From: Alex P Date: Wed, 13 Jun 2018 11:28:43 -0700 Subject: [PATCH 002/100] Rename page header class names to follow BEM conventions --- ui/src/admin/containers/AdminInfluxDBPage.js | 8 ++++---- .../chronograf/AdminChronografPage.js | 6 +++--- ui/src/alerts/containers/AlertsApp.tsx | 8 ++++---- .../dashboards/components/DashboardHeader.js | 12 +++++------ .../components/DashboardsHeader.tsx | 8 ++++---- .../TemplateVariableManager.js | 6 +++--- .../components/WriteDataHeader.tsx | 6 +++--- ui/src/data_explorer/containers/Header.tsx | 8 ++++---- ui/src/flux/components/FluxHeader.tsx | 8 ++++---- ui/src/flux/components/FluxOverlay.tsx | 6 +++--- ui/src/hosts/containers/HostsPage.js | 8 ++++---- ui/src/kapacitor/components/KapacitorForm.tsx | 6 +++--- ui/src/kapacitor/components/RuleHeader.js | 6 +++--- ui/src/kapacitor/components/RuleHeaderSave.js | 2 +- .../kapacitor/components/TICKscriptOverlay.js | 6 +++--- .../kapacitor/components/TickscriptHeader.tsx | 8 ++++---- .../containers/KapacitorRulesPage.tsx | 8 ++++---- ui/src/logs/components/LogViewerHeader.tsx | 8 ++++---- ui/src/sources/containers/ManageSources.tsx | 8 ++++---- ui/src/sources/containers/SourcePage.js | 8 ++++---- ui/src/status/containers/StatusPage.js | 8 ++++---- ui/src/style/layout/page-header.scss | 20 +++++++++---------- ui/src/style/unsorted.scss | 4 ++-- 23 files changed, 88 insertions(+), 88 deletions(-) diff --git a/ui/src/admin/containers/AdminInfluxDBPage.js b/ui/src/admin/containers/AdminInfluxDBPage.js index d39060fbc..02d59569f 100644 --- a/ui/src/admin/containers/AdminInfluxDBPage.js +++ b/ui/src/admin/containers/AdminInfluxDBPage.js @@ -224,11 +224,11 @@ class AdminInfluxDBPage extends Component { return (
-
-
-

InfluxDB Admin

+
+
+

InfluxDB Admin

-
+
diff --git a/ui/src/admin/containers/chronograf/AdminChronografPage.js b/ui/src/admin/containers/chronograf/AdminChronografPage.js index 5e1eb9f77..964c34ee4 100644 --- a/ui/src/admin/containers/chronograf/AdminChronografPage.js +++ b/ui/src/admin/containers/chronograf/AdminChronografPage.js @@ -50,9 +50,9 @@ const sections = me => [ const AdminChronografPage = ({me, source, params: {tab}}) => (
-
-
-

Chronograf Admin

+
+
+

Chronograf Admin

diff --git a/ui/src/alerts/containers/AlertsApp.tsx b/ui/src/alerts/containers/AlertsApp.tsx index 0eae2a813..5f0dcf086 100644 --- a/ui/src/alerts/containers/AlertsApp.tsx +++ b/ui/src/alerts/containers/AlertsApp.tsx @@ -96,11 +96,11 @@ class AlertsApp extends PureComponent { ) : (
-
-
-

Alert History

+
+
+

Alert History

-
+
isHidden ? null : (
-
+
{names && names.length > 1 ? ( @@ -51,7 +51,7 @@ const DashboardHeader = ({ {activeDashboard} +

{activeDashboard}

} >
) : ( -

{activeDashboard}

+

{activeDashboard}

)}
-
+
{dashboard ? ( diff --git a/ui/src/dashboards/components/DashboardsHeader.tsx b/ui/src/dashboards/components/DashboardsHeader.tsx index 7dde2440d..2eec2b0be 100644 --- a/ui/src/dashboards/components/DashboardsHeader.tsx +++ b/ui/src/dashboards/components/DashboardsHeader.tsx @@ -4,11 +4,11 @@ import SourceIndicator from 'src/shared/components/SourceIndicator' const DashboardsHeader = (): JSX.Element => (
-
-
-

Dashboards

+
+
+

Dashboards

-
+
diff --git a/ui/src/dashboards/components/template_variables/TemplateVariableManager.js b/ui/src/dashboards/components/template_variables/TemplateVariableManager.js index 5cdd9dbf6..251a6d618 100644 --- a/ui/src/dashboards/components/template_variables/TemplateVariableManager.js +++ b/ui/src/dashboards/components/template_variables/TemplateVariableManager.js @@ -22,10 +22,10 @@ const TemplateVariableManager = ({ }) => (
-
-

Template Variables

+
+

Template Variables

-
+
diff --git a/ui/src/flux/components/FluxOverlay.tsx b/ui/src/flux/components/FluxOverlay.tsx index 6f59936a0..51b0ef0b5 100644 --- a/ui/src/flux/components/FluxOverlay.tsx +++ b/ui/src/flux/components/FluxOverlay.tsx @@ -29,10 +29,10 @@ class FluxOverlay extends PureComponent { return (
-
-

Connect to Flux

+
+

Connect to Flux

-
+
-
-
-

Host List

+
+
+

Host List

-
+
{ return (
-
-
-

{this.headerText}

+
+
+

{this.headerText}

diff --git a/ui/src/kapacitor/components/RuleHeader.js b/ui/src/kapacitor/components/RuleHeader.js index 9ecf4c2eb..e4d84c62c 100644 --- a/ui/src/kapacitor/components/RuleHeader.js +++ b/ui/src/kapacitor/components/RuleHeader.js @@ -14,9 +14,9 @@ class RuleHeader extends Component { return (
-
-
-

Alert Rule Builder

+
+
+

Alert Rule Builder

( -
+
{validationError ? (