RubyGems - playbook_ui - Versions diffs - 6.1.0 → 7.0.0.pre.alpha2 - Mend

playbook_ui 6.1.0 → 7.0.0.pre.alpha2

Files changed (385) hide show

data/app/pb_kits/playbook/pb_legend/_legend.html.erb CHANGED

@@ -1,11 +1 @@
-<%= content_tag(:div, object.text,
-    aria: object.aria,
-    id: object.id,
-    data: object.data,
-    class: object.classname) do %>
-  <%= pb_rails("body", props: {color: object.body_color}) do %>
-    <span class="pb_legend_indicator_circle"></span>
-    <%= pb_rails("title", props: { dark: object.dark, text: object.prefix_text, tag: "span", size: 4 }) %>
-    <%= object.text %>
-  <% end %>
-<% end %>
+<%= react_component('Legend', object.chart_options) %>

data/app/pb_kits/playbook/pb_legend/_legend.jsx CHANGED

@@ -9,7 +9,7 @@ import { Body, Title } from '../'
 type LegendProps = {
   aria?: object,
-  className?: String,
+  className?: string,
   color?: | "data_1"
     | "data_2"
     | "data_3"
@@ -17,11 +17,11 @@ type LegendProps = {
     | "data_5"
     | "data_6"
     | "data_7",
-  dark?: Boolean,
+  dark?: boolean,
   data?: object,
-  id?: String,
-  prefixText?: String,
-  text: String,
+  id?: string,
+  prefixText?: string,
+  text: string,
 }
 const Legend = (props: LegendProps) => {
@@ -38,17 +38,17 @@ const Legend = (props: LegendProps) => {
   const ariaProps = buildAriaProps(aria)
   const dataProps = buildDataProps(data)
-  const darkClass = dark ? 'dark' : 'light'
-  const bodyCSS = classnames(
-    buildCss('pb_legend_kit', color, darkClass), className,
-    globalProps(props)
+  const bodyCss = classnames(
+    buildCss('pb_legend_kit', color),
+    globalProps(props),
+    className
   )
   return (
     <div
         {...ariaProps}
         {...dataProps}
-        className={bodyCSS}
+        className={bodyCss}
         id={id}
     >
       <Body color={dark ? 'lighter' : 'light'}>

data/app/pb_kits/playbook/pb_legend/legend.rb CHANGED

@@ -10,12 +10,7 @@ module Playbook
       prop :color, type: Playbook::Props::Enum,
                    values: (1..7).map { |n| "data_#{n}" },
                    default: "data_1"
-      prop :dark, type: Playbook::Props::Boolean,
-                  default: false
       prop :prefix_text
       prop :text, required: true
       def body_color
@@ -23,7 +18,7 @@ module Playbook
       end
       def classname
-        generate_classname("pb_legend_kit", color, dark ? "dark" : "light")
+        generate_classname("pb_legend_kit", color)
       end
     end
   end

data/app/pb_kits/playbook/pb_line_graph/_line_graph.html.erb CHANGED

@@ -1,13 +1 @@
-<%= content_tag(:div, "",
-    aria: object.aria,
-    id: object.id,
-    data: object.data,
-    class: object.classname
-    ) %>
-<% content_for :pb_js do %>
-  <%= javascript_tag do %>
-    window.addEventListener('DOMContentLoaded', function() {
-      new pbChart('.selector', <%= object.chart_options %>)
-    })
-  <% end %>
-<% end %>
+<%= react_component('LineGraph', object.chart_options) %>

data/app/pb_kits/playbook/pb_line_graph/_line_graph.jsx CHANGED

@@ -6,24 +6,24 @@ import { globalProps } from '../utilities/globalProps.js'
 import { pbChart } from '../'
 type LineGraphProps = {
-  axisTitle?: String,
-  xAxisCategories: Array,
-  yAxisMin: Number,
-  yAxisMax: Number,
-  className?: String,
-  chartData: Array<{
-      name: String,
-      data: Array<Number>,
+  axisTitle?: string,
+  xAxisCategories: array,
+  yAxisMin: number,
+  yAxisMax: number,
+  className?: string,
+  chartData: array<{
+      name: string,
+      data: array<number>,
   }>,
-  gradient?: Boolean,
-  id: String,
-  pointStart: Number,
-  subTitle?: String,
-  title: String,
-  type?: String,
-  legend?: Boolean,
-  toggleLegendClick?: Boolean,
-  height?: String,
+  gradient?: boolean,
+  id: string,
+  pointStart: number,
+  subTitle?: string,
+  title: string,
+  type?: string,
+  legend?: boolean,
+  toggleLegendClick?: boolean,
+  height?: string,
 }
 export default class LineGraph extends React.Component<LineGraphProps> {

data/app/pb_kits/playbook/pb_line_graph/line_graph.rb CHANGED

@@ -44,7 +44,7 @@ module Playbook
           legend: legend,
           toggleLegendClick: toggle_legend_click,
           height: height,
-        }.to_json.html_safe
+        }
       end
       def classname

data/app/pb_kits/playbook/pb_list/_list.jsx CHANGED

@@ -7,18 +7,18 @@ import { globalProps } from '../utilities/globalProps.js'
 type ListProps = {
   aria?: object,
-  borderless: Boolean,
-  className?: String,
-  children: Array<Node> | Node,
-  dark: Boolean,
+  borderless: boolean,
+  className?: string,
+  children: array<Node> | Node,
+  dark: boolean,
   data?: object,
-  id?: String,
+  id?: string,
   layout: "" | "left" | "right",
-  ordered: Boolean,
-  role?: String,
-  tabIndex?: String,
-  size?: String,
-  xpadding: Boolean,
+  ordered: boolean,
+  role?: string,
+  tabIndex?: string,
+  size?: string,
+  xpadding: boolean,
 }
 const List = (props: ListProps) => {

data/app/pb_kits/playbook/pb_list/_list_item.jsx CHANGED

@@ -7,11 +7,11 @@ import { globalProps } from '../utilities/globalProps.js'
 type ListItemProps = {
   aria?: object,
-  children: Array<Node> | Node,
-  className?: String,
+  children: array<Node> | Node,
+  className?: string,
   data?: object,
-  id?: String,
-  tabIndex?: String,
+  id?: string,
+  tabIndex?: string,
 }
 const ListItem = (props: ListItemProps) => {

data/app/pb_kits/playbook/pb_loading_inline/_loading_inline.jsx CHANGED

@@ -8,10 +8,10 @@ import { globalProps } from '../utilities/globalProps.js'
 type LoadingInlineProps = {
   align?: "left" | "center" | "right",
-  className?: String,
-  dark?: Boolean,
-  data?: String,
-  id?: String,
+  className?: string,
+  dark?: boolean,
+  data?: string,
+  id?: string,
 }
 const LoadingInline = (props: LoadingInlineProps) => {

data/app/pb_kits/playbook/pb_logistic/_logistic.jsx CHANGED

@@ -16,14 +16,14 @@ const dateString = (value: DateTime) => {
 type LogisticProps = {
   aria?: object,
-  className?: String,
-  dark?: Boolean,
+  className?: string,
+  dark?: boolean,
   data?: object,
-  date: String,
-  id?: String,
-  link?: String,
-  projectName?: String,
-  projectNumber?: Number,
+  date: string,
+  id?: string,
+  link?: string,
+  projectName?: string,
+  projectNumber?: number,
 }
 const Logistic = (props: LogisticProps) => {

data/app/pb_kits/playbook/pb_message/_message.jsx CHANGED

@@ -8,15 +8,15 @@ import { globalProps } from '../utilities/globalProps.js'
 type MessageProps = {
   aria: object,
-  avatarName?: String,
-  avatarStatus?: String,
-  avatarUrl?: String,
-  className?: String,
+  avatarName?: string,
+  avatarStatus?: string,
+  avatarUrl?: string,
+  className?: string,
   data?: object,
-  id?: String,
-  label?: String,
-  message: String,
-  timestamp?: String,
+  id?: string,
+  label?: string,
+  message: string,
+  timestamp?: string,
 }
 const Message = (props: MessageProps) => {

data/app/pb_kits/playbook/pb_multiple_users/_multiple_users.html.erb CHANGED

@@ -1,9 +1,10 @@
 <%= content_tag(:div,
-    id: object.id,
+    aria: object.aria,
+    class: object.classname,
     data: object.data,
-    class: object.classname) do %>
+    id: object.id) do %>
     <% object.users.take(object.display_count).each do |user| %>
-      <%= pb_rails("avatar", props: user.merge({size: "xs", classname: "pb_multiple_users_item"}) ) %>
+      <%= pb_rails("avatar", props: user.merge({size: "xs", classname: "pb_multiple_users_item", dark: object.dark}) ) %>
     <% end %>
     <% if object.more_than_four %>

data/app/pb_kits/playbook/pb_multiple_users/_multiple_users.jsx CHANGED

@@ -2,37 +2,55 @@
 import React from 'react'
 import classnames from 'classnames'
-import { buildCss } from '../utilities/props'
+import { buildAriaProps, buildCss, buildDataProps } from '../utilities/props'
 import { Avatar } from '../'
 import { globalProps } from '../utilities/globalProps.js'
 type MultipleUsersProps = {
-  className?: String,
-  id?: String,
-  maxDisplayedUsers?: Number,
-  reverse?: Boolean,
-  users: Array<Object>,
+  aria?: object,
+  className?: string,
+  dark?: boolean,
+  data?: object,
+  id?: string,
+  maxDisplayedUsers?: number,
+  reverse?: boolean,
+  users: array<object>,
 }
 const MultipleUsers = (props: MultipleUsersProps) => {
-  const { className, id, maxDisplayedUsers = 4, reverse = false, users } = props
+  const {
+    aria = {},
+    className,
+    dark = false,
+    data = {},
+    id,
+    maxDisplayedUsers = 4,
+    reverse = false,
+    users,
+  } = props
   const displayCount =
     users.length > maxDisplayedUsers ? maxDisplayedUsers - 1 : users.length
   const usersToDisplay = users.slice(0, displayCount)
+  const reverseClass = reverse === true ? 'reverse' : ''
+  const ariaProps = buildAriaProps(aria)
+  const dataProps = buildDataProps(data)
+  const classes = classnames(buildCss('pb_multiple_users_kit', reverseClass),
+    className, globalProps(props))
   return (
     <div
-        className={classnames(
-        className,
-        buildCss('pb_multiple_users_kit', reverse && 'reverse'),
-        globalProps(props)
-      )}
+        {...ariaProps}
+        {...dataProps}
+        className={classes}
         id={id}
     >
       {usersToDisplay.map((avatarData, index) => (
         <Avatar
             {...avatarData}
             className="pb_multiple_users_item"
+            dark={dark}
             key={index}
             size="xs"
         />

data/app/pb_kits/playbook/pb_multiple_users/_multiple_users.scss CHANGED

@@ -28,6 +28,12 @@ $pb_multiple_users_size: map-get($avatar-sizes, "xs");
     margin-left: $pb_multiple_users_overlap;
     margin-right: 0;
+    &.dark {
+      .avatar_wrapper {
+        border: $pb_multiple_users_border_size solid $bg_dark;
+      }
+    }
     .avatar_wrapper {
       border: $pb_multiple_users_border_size solid $white;
     }

data/app/pb_kits/playbook/pb_multiple_users/docs/_multiple_users_dark.html.erb ADDED

@@ -0,0 +1,41 @@
+<%= pb_rails("multiple_users", props: {
+  dark: true,
+  users: [
+    {
+      name: "Patrick Welch",
+      image_url: "https://randomuser.me/api/portraits/men/9.jpg",
+    },
+    {
+      name: "Lucille Sanchez",
+      image_url: "https://randomuser.me/api/portraits/women/6.jpg",
+    },
+    {
+      name: "Beverly Reyes",
+      image_url: "https://randomuser.me/api/portraits/women/74.jpg",
+    },
+    {
+      name: "Keith Craig",
+      image_url: "https://randomuser.me/api/portraits/men/40.jpg",
+    },
+    {
+      name: "Alicia Cooper",
+      image_url: "https://randomuser.me/api/portraits/women/46.jpg",
+    }
+  ]
+}) %>
+<br/><br/>
+<%= pb_rails("multiple_users", props: {
+  dark: true,
+  users: [
+    {
+      name: "Shawn Palmer",
+      image_url: "https://randomuser.me/api/portraits/men/93.jpg",
+    },
+    {
+      name: "Andrew Murray",
+      image_url: "https://randomuser.me/api/portraits/men/75.jpg",
+    }
+  ]
+}) %>

data/app/pb_kits/playbook/pb_multiple_users/docs/_multiple_users_dark.jsx ADDED

@@ -0,0 +1,52 @@
+import React from 'react'
+import { MultipleUsers } from '../../'
+const MultipleUsersDark = () => {
+  return (
+    <div>
+      <MultipleUsers
+          dark
+          users={[
+        {
+          name: 'Patrick Welch',
+          imageUrl: 'https://randomuser.me/api/portraits/men/9.jpg',
+        },
+        {
+          name: 'Lucille Sanchez',
+          imageUrl: 'https://randomuser.me/api/portraits/women/6.jpg',
+        },
+        {
+          name: 'Beverly Reyes',
+          imageUrl: 'https://randomuser.me/api/portraits/women/74.jpg',
+        },
+        {
+          name: 'Keith Craig',
+          imageUrl: 'https://randomuser.me/api/portraits/men/40.jpg',
+        },
+        {
+          name: 'Alicia Cooper',
+          imageUrl: 'https://randomuser.me/api/portraits/women/46.jpg',
+        },
+      ]}
+      />
+      <br />
+      <br />
+      <MultipleUsers
+          dark
+          users={[
+        {
+          name: 'Shawn Palmer',
+          imageUrl: 'https://randomuser.me/api/portraits/men/93.jpg',
+        },
+        {
+          name: 'Andrew Murray Cooper Craig',
+        },
+      ]}
+      />
+    </div>
+  )
+}
+export default MultipleUsersDark