playbook_ui 14.12.0.pre.alpha.PLAY1865reactdatepickerreinitializingbug5732 → 14.12.0.pre.alpha.advancedtablealignmentfixes5693

data/lib/playbook/pb_forms_helper.rb

@@ -1,10 +1,7 @@
 # frozen_string_literal: true
 
-require_relative "pb_forms_global_props_helper"
-
 module Playbook
   module PbFormsHelper
-    include Playbook::PbFormsGlobalPropsHelper
     # Renders a pb form with ::Playbook::Forms::Builder, that can render
     # Playbook kits in the most railsie way.
     #
@@ -26,17 +23,11 @@ module Playbook
     # @param validate [Boolean] whether validation should be triggered or not
     # @see [#form_with] for other options
     def pb_form_with(data: {}, validate: false, loading: false, **kwargs, &block)
-      global_props, form_options = extract_all_props(kwargs)
-
-      classnames = ["pb-form"]
-      classnames << form_options[:class] if form_options[:class].present?
-      classnames << "pb_form_loading" if loading
-      classnames.concat(generate_prop_classes(global_props))
-
       data = data.merge("pb-form-validation" => validate)
-
-      options = form_options.merge(
-        class: classnames.compact.join(" "),
+      classname = ["pb-form", kwargs[:class]].join(" ")
+      classname += " pb_form_loading" if loading
+      options = kwargs.merge(
+        class: classname,
         data: data,
         builder: ::Playbook::Forms::Builder
       )

data/lib/playbook/version.rb

@@ -2,5 +2,5 @@
 
 module Playbook
   PREVIOUS_VERSION = "14.12.0"
-  VERSION = "14.12.0.pre.alpha.PLAY1865reactdatepickerreinitializingbug5732"
+  VERSION = "14.12.0.pre.alpha.advancedtablealignmentfixes5693"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: playbook_ui
 version: !ruby/object:Gem::Version
-  version: 14.12.0.pre.alpha.PLAY1865reactdatepickerreinitializingbug5732
+  version: 14.12.0.pre.alpha.advancedtablealignmentfixes5693
 platform: ruby
 authors:
 - Power UX
@@ -9,7 +9,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2025-01-29 00:00:00.000000000 Z
+date: 2025-01-28 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: actionpack
@@ -300,10 +300,8 @@ files:
 - app/pb_kits/playbook/pb_advanced_table/docs/_advanced_table_expanded_control.md
 - app/pb_kits/playbook/pb_advanced_table/docs/_advanced_table_inline_row_loading.jsx
 - app/pb_kits/playbook/pb_advanced_table/docs/_advanced_table_inline_row_loading.md
-- app/pb_kits/playbook/pb_advanced_table/docs/_advanced_table_loading.html.erb
 - app/pb_kits/playbook/pb_advanced_table/docs/_advanced_table_loading.jsx
-- app/pb_kits/playbook/pb_advanced_table/docs/_advanced_table_loading_rails.md
-- app/pb_kits/playbook/pb_advanced_table/docs/_advanced_table_loading_react.md
+- app/pb_kits/playbook/pb_advanced_table/docs/_advanced_table_loading.md
 - app/pb_kits/playbook/pb_advanced_table/docs/_advanced_table_no_subrows.jsx
 - app/pb_kits/playbook/pb_advanced_table/docs/_advanced_table_pagination.jsx
 - app/pb_kits/playbook/pb_advanced_table/docs/_advanced_table_pagination.md
@@ -798,14 +796,6 @@ files:
 - app/pb_kits/playbook/pb_contact/docs/_description.md
 - app/pb_kits/playbook/pb_contact/docs/example.yml
 - app/pb_kits/playbook/pb_contact/docs/index.js
-- app/pb_kits/playbook/pb_copy_button/_copy_button.scss
-- app/pb_kits/playbook/pb_copy_button/_copy_button.tsx
-- app/pb_kits/playbook/pb_copy_button/copy_button.test.jsx
-- app/pb_kits/playbook/pb_copy_button/docs/_copy_button_default.jsx
-- app/pb_kits/playbook/pb_copy_button/docs/_copy_button_from.jsx
-- app/pb_kits/playbook/pb_copy_button/docs/_copy_button_from.md
-- app/pb_kits/playbook/pb_copy_button/docs/example.yml
-- app/pb_kits/playbook/pb_copy_button/docs/index.js
 - app/pb_kits/playbook/pb_currency/_currency.scss
 - app/pb_kits/playbook/pb_currency/_currency.tsx
 - app/pb_kits/playbook/pb_currency/currency.html.erb
@@ -3309,8 +3299,8 @@ files:
 - app/pb_kits/playbook/utilities/test/globalProps/truncate.test.js
 - app/pb_kits/playbook/utilities/text.ts
 - app/pb_kits/playbook/utilities/validEmojiChecker.ts
-- dist/chunks/_typeahead-BIhRQo8Q.js
-- dist/chunks/_weekday_stacked-bORvL0Zi.js
+- dist/chunks/_typeahead-BWwaAo_0.js
+- dist/chunks/_weekday_stacked-N1NVUtQO.js
 - dist/chunks/lazysizes-B7xYodB-.js
 - dist/chunks/lib-kMuhBuU7.js
 - dist/chunks/pb_form_validation-DBJ0wZuS.js
@@ -3365,7 +3355,6 @@ files:
 - lib/playbook/overflow.rb
 - lib/playbook/pagination_renderer.rb
 - lib/playbook/pb_doc_helper.rb
-- lib/playbook/pb_forms_global_props_helper.rb
 - lib/playbook/pb_forms_helper.rb
 - lib/playbook/pb_kit_helper.rb
 - lib/playbook/position.rb

data/app/pb_kits/playbook/pb_advanced_table/docs/_advanced_table_loading.html.erb

@@ -1,33 +0,0 @@
-<% column_definitions = [
-    {
-      accessor: "year",
-      label: "Year",
-      cellAccessors: ["quarter", "month", "day"],
-    },
-    {
-      accessor: "newEnrollments",
-      label: "New Enrollments",
-    },
-    {
-      accessor: "scheduledMeetings",
-      label: "Scheduled Meetings",
-    },
-    {
-      accessor: "attendanceRate",
-      label: "Attendance Rate",
-    },
-    {
-      accessor: "completedClasses",
-      label: "Completed Classes",
-    },
-    {
-      accessor: "classCompletionRate",
-      label: "Class Completion Rate",
-    },
-    {
-      accessor: "graduatedStudents",
-      label: "Graduated Students",
-    }
-] %>
-
-<%= pb_rails("advanced_table", props: { id: "beta_table", table_data: @table_data, column_definitions: column_definitions, loading: true }) %>

data/app/pb_kits/playbook/pb_advanced_table/docs/_advanced_table_loading_rails.md

@@ -1 +0,0 @@
-The optional `loading` prop takes a boolean value that can be managed using state. If loading is true, the table will display the loading skeleton and once loading is false, the table will render with the data provided. 

data/app/pb_kits/playbook/pb_copy_button/_copy_button.scss

@@ -1,3 +0,0 @@
-.pb_copy_button_kit {
-
-}

data/app/pb_kits/playbook/pb_copy_button/_copy_button.tsx

@@ -1,92 +0,0 @@
-
-import React, { useState } from 'react'
-import classnames from 'classnames'
-import { buildAriaProps, buildCss, buildDataProps } from '../utilities/props'
-import { globalProps } from '../utilities/globalProps'
-
-import Button from '../pb_button/_button'
-import Tooltip from '../pb_tooltip/_tooltip'
-
-type CopyButtonProps = {
-  aria?: { [key: string]: string },
-  className?: string,
-  data?: { [key: string]: string },
-  id?: string,
-  from?: string,
-  text?: string,
-  tooltipPlacement?:  "top" | "right" | "bottom" | "left",
-  tooltipText?: string,
-  value?: string,
-}
-
-const CopyButton = (props: CopyButtonProps) => {
-  const {
-    aria = {},
-    className,
-    data = {},
-    from = '',
-    id,
-    text= 'Copy',
-    tooltipPlacement= 'bottom',
-    tooltipText = 'Copied!',
-    value = '',
-  } = props
-
-  const [copied, setCopied] = useState(false)
-
-  const ariaProps = buildAriaProps(aria)
-  const dataProps = buildDataProps(data)
-  const classes = classnames(buildCss('pb_copy_button_kit'), globalProps(props), className)
-
-  const copy = () => {
-    if (!from && !value) {
-      return
-    }
-
-    if (value) {
-      navigator.clipboard.writeText(value)
-    } else if (from) {
-      const copyElement = document.getElementById(from);
-      let copyText = copyElement?.innerText
-
-      if (copyElement instanceof HTMLTextAreaElement || copyElement instanceof HTMLInputElement) {
-        copyText = copyElement.value;
-      }
-
-      if (copyText) {
-        navigator.clipboard.writeText(copyText)
-      }
-    }
-
-    setCopied(true)
-
-    setTimeout(() => {
-      setCopied(false)
-    }, 1000);
-  }
-
-  return (
-    <div
-        {...ariaProps}
-        {...dataProps}
-        className={classes}
-        id={id}
-    >
-      <Tooltip 
-          forceOpenTooltip={copied}
-          placement={tooltipPlacement}
-          showTooltip={false}
-          text={tooltipText}
-      >
-        <Button
-            icon='copy'
-            onClick={copy}
-        >
-          { text }
-        </Button>
-      </Tooltip>
-    </div>
-  )
-}
-
-export default CopyButton

data/app/pb_kits/playbook/pb_copy_button/copy_button.test.jsx

@@ -1,64 +0,0 @@
-import React from 'react';
-import { CopyButton } from 'playbook-ui'
-import { ensureAccessible, renderKit, render, fireEvent, screen } from '../utilities/test-utils'
-
-const props = {
-  data: { testid: 'default', value: 'copy' }
-}
-
-test('returns namespaced class name', () => {
-  const kit = renderKit(CopyButton, props)
-  expect(kit).toBeInTheDocument()
-  expect(kit).toHaveClass('pb_copy_button_kit')
-})
-
-it('should be accessible', async () => {
-  ensureAccessible(CopyButton, props)
-})
-
-// It's difficult to actually use navigator.clipboard.readText, so we mock
-it('copies the value to clipboard and pastes it into an input', async () => {
-  Object.defineProperty(global, 'navigator', {
-    value: {
-      clipboard: {
-        writeText: jest.fn().mockResolvedValueOnce(undefined),
-      },
-    },
-    writable: true,
-  })
-
-  render(<CopyButton {...props} />)
-
-  const copyButton = screen.getByTestId('default')
-  fireEvent.click(copyButton)
-
-  await navigator.clipboard.writeText('copy')
-
-  expect(navigator.clipboard.writeText).toHaveBeenCalledWith("copy");
-})
-
-test('passes text and tooltip props to button', () => {
-  render(
-      <CopyButton
-          data={{ testid: 'text-test' }}
-          text={"text"}
-          tooltipPlacement="right"
-          tooltipText="Text copied!"
-          value="copy"
-      />
-  )
-
-  const content = screen.getByText("text")
-  expect(content).toHaveTextContent("text")
-
-  const kit = screen.getByTestId('text-test')
-  const button = kit.querySelector('.pb_button_kit_primary_inline_enabled')
-  expect(button).toBeInTheDocument()
-
-  fireEvent.click(button)
-  const tooltipContent = screen.getByText("Text copied!")
-  expect(tooltipContent).toHaveTextContent("Text copied!")
-
-  const tooltip = kit.querySelector('.pb_tooltip_kit')
-  expect(tooltip).toBeInTheDocument()
-})

data/app/pb_kits/playbook/pb_copy_button/docs/_copy_button_default.jsx

@@ -1,21 +0,0 @@
-import React from 'react'
-import { CopyButton, Textarea } from 'playbook-ui'
-
-const CopyButtonDefault = (props) => (
-  <div>
-    <CopyButton
-        {...props}
-        text="Copy Text"
-        tooltipPlacement="right"
-        tooltipText="Text copied!"
-        value="Playbook makes it easy to support bleeding edge, or legacy systems. Use Playbook’s 200+ components and end-to-end design language to create simple, intuitive and beautiful experiences with ease."
-    />
-
-    <Textarea
-        {...props}
-        placeholder="Copy and paste here"
-    />
-  </div>
-)
-
-export default CopyButtonDefault

data/app/pb_kits/playbook/pb_copy_button/docs/_copy_button_from.jsx

@@ -1,45 +0,0 @@
-import React, { useState } from 'react'
-import { CopyButton, Body, TextInput, Textarea } from 'playbook-ui'
-
-const CopyButtonFrom = (props) => {
-  const [text, setText] = useState("Copy this text input text")
-
-  const handleChange = (event) => {
-    setText(event.target.value);
-  }
-
-  return (<div>
-    <Body id="body">Copy this body text!</Body>
-    <CopyButton
-        {...props}
-        from="body"
-        marginBottom="sm"
-        text="Copy Body text"
-        tooltipPlacement="right"
-        tooltipText="Body text copied!"
-    />
-
-    <TextInput
-        {...props}
-        id="textinput"
-        onChange={handleChange}
-        value={text}
-    />
-    <CopyButton
-        {...props}
-        from="textinput"
-        marginBottom="sm"
-        text="Copy Text Input"
-        tooltipPlacement="right"
-        tooltipText="Text input copied!"
-    />
-
-    <Textarea
-        {...props}
-        placeholder="Copy and paste here"
-    />
-  </div>
-  )
-}
-
-export default CopyButtonFrom

data/app/pb_kits/playbook/pb_copy_button/docs/_copy_button_from.md

@@ -1 +0,0 @@
-Provide an element's ID as the `from` parameter, and its text will be copied. If the element is an input, its `value` will be copied; otherwise, the `innerText` will be used. Additionally, if a `value` prop is provided, it will override the content from the `from` element and be copied instead.

data/app/pb_kits/playbook/pb_copy_button/docs/example.yml

@@ -1,8 +0,0 @@
-examples:
-  
-  
-  react:
-  - copy_button_default: Default
-  - copy_button_from: Copy From
-  
-  

data/app/pb_kits/playbook/pb_copy_button/docs/index.js

@@ -1,2 +0,0 @@
-export { default as CopyButtonDefault } from './_copy_button_default.jsx'
-export { default as CopyButtonFrom } from './_copy_button_from.jsx'