playbook_ui 15.2.0.pre.alpha.advancedtableicons11557 → 15.2.0.pre.alpha.buttonfix11269

data/app/pb_kits/playbook/pb_rich_text_editor/_tiptap_styles.scss

@@ -9,37 +9,6 @@
 @import "previewer_mixin";
 
 [class^="pb_rich_text_editor_kit"] {
-    &.inline {
-        .toolbar {
-          opacity: 0;
-          transition: all 0.3s ease-in-out 0s;
-        }
-    
-        &:focus-within .toolbar {
-          opacity: 100;
-        }
-    
-        .ProseMirror {
-          border: 1px solid transparent;
-          transition: all 0.3s ease-in-out 0s;
-        }
-    
-        &:focus-within .ProseMirror {
-          border: 1px solid $border_light;
-          border-top: none;
-        }
-    
-        &:hover {
-          .toolbar {
-            opacity: 100;
-          }
-    
-          .ProseMirror {
-            border: 1px solid $border_light;
-            border-top: none;
-          }
-        }
-      }
   .toolbar_button {
     display: flex;
     align-items: center;
@@ -73,11 +42,6 @@
     }
   }
 
-  .pb_rich_text_editor_tiptap_toolbar_sticky {
-    position: sticky;
-    top: 0;
-    z-index: 10;
-  }
   .toolbar {
     border-radius: $border_rad_heaviest $border_rad_heaviest 0 0;
     border: 1px solid $border_light;

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

@@ -7,7 +7,7 @@ examples:
   - rich_text_editor_focus: Focus
   - rich_text_editor_sticky: Sticky
   - rich_text_editor_templates: Templates
-  # - rich_text_editor_toolbar_bottom: Toolbar Bottom
+  - rich_text_editor_toolbar_bottom: Toolbar Bottom
   - rich_text_editor_inline: Inline
   - rich_text_editor_preview: Preview
 
@@ -17,17 +17,11 @@ examples:
   - rich_text_editor_more_extensions: Advanced (Extra Extensions)
   - rich_text_editor_toolbar_disabled: Advanced (Toolbar disabled)
   - rich_text_editor_simple: Simple
-  - rich_text_editor_advanced_simple: Advanced (Simple)
   - rich_text_editor_attributes: Attributes
-  - rich_text_editor_advanced_attributes: Advanced (Attributes)
   - rich_text_editor_focus: Focus
-  - rich_text_editor_advanced_focus: Advanced (Focus)
   - rich_text_editor_sticky: Sticky
-  - rich_text_editor_advanced_sticky: Advanced (Sticky)
   - rich_text_editor_templates: Templates
-  - rich_text_editor_advanced_templates: Advanced (Templates)
-  # - rich_text_editor_toolbar_bottom: Toolbar Bottom
+  - rich_text_editor_toolbar_bottom: Toolbar Bottom
   - rich_text_editor_inline: Inline
-  - rich_text_editor_advanced_inline: Advanced (Inline)
   - rich_text_editor_preview: Preview
   - rich_text_editor_advanced_preview: Advanced Preview

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

@@ -11,9 +11,3 @@ export { default as RichTextEditorAdvancedDefault } from './_rich_text_editor_ad
 export { default as RichTextEditorMoreExtensions } from './_rich_text_editor_more_extensions.jsx'
 export { default as RichTextEditorToolbarDisabled } from './_rich_text_editor_toolbar_disabled.jsx'
 export { default as RichTextEditorAdvancedPreview } from './_rich_text_editor_advanced_preview.jsx'
-export { default as RichTextEditorAdvancedSimple } from './_rich_text_editor_advanced_simple.jsx'
-export { default as RichTextEditorAdvancedFocus } from './_rich_text_editor_advanced_focus.jsx'
-export { default as RichTextEditorAdvancedTemplates } from './_rich_text_editor_advanced_templates.jsx'
-export { default as RichTextEditorAdvancedAttributes } from './_rich_text_editor_advanced_attributes.jsx'
-export { default as RichTextEditorAdvancedSticky } from './_rich_text_editor_advanced_sticky.jsx'
-export { default as RichTextEditorAdvancedInline } from './_rich_text_editor_advanced_inline.jsx'

data/app/pb_kits/playbook/pb_rich_text_editor/rich_text_editor.test.js

@@ -1,7 +1,5 @@
 import React from 'react'
-import { render, screen, fireEvent, waitFor } from '../utilities/test-utils'
-import { useEditor, EditorContent } from "@tiptap/react"
-import StarterKit from "@tiptap/starter-kit"
+import { render, screen } from '../utilities/test-utils'
 
 import RichTextEditor from './_rich_text_editor'
 
@@ -64,127 +62,3 @@ test('returns "Dark" class name', () => {
   const kit = screen.getByTestId(testId)
   expect(kit).toHaveClass(`${kitClass} dark`)
 })
-
-// TipTap testing
-const TestAdvancedEditor = ({ toolbarOnFocus = false, ...props }) => {
-  const editor = useEditor({
-    extensions: [StarterKit],
-    content: '<p>Test content</p>',
-  })
-
-  if (!editor) return null
-
-  return (
-    <RichTextEditor
-        advancedEditor={editor}
-        data={{ testid: testId }}
-        toolbarOnFocus={toolbarOnFocus}
-        {...props}
-    >
-      <EditorContent editor={editor} />
-    </RichTextEditor>
-  )
-}
-
-describe('Advanced TipTap Editor works as expected', () => {
-  test('renders advanced editor with toolbar', () => {
-    render(<TestAdvancedEditor />)
-    
-    const kit = screen.getByTestId(testId)
-    expect(kit).toHaveClass(kitClass)
-    
-    // Check for advanced container
-    const advancedContainer = kit.querySelector('.pb_rich_text_editor_advanced_container')
-    expect(advancedContainer).toBeInTheDocument()
-    
-    // Check for toolbar
-    const toolbar = kit.querySelector('.toolbar')
-    expect(toolbar).toBeInTheDocument()
-  })
-
-  test('renders advanced editor without toolbar when advancedEditorToolbar is false', () => {
-    render(<TestAdvancedEditor advancedEditorToolbar={false} />)
-    
-    const kit = screen.getByTestId(testId)
-    const toolbar = kit.querySelector('.toolbar')
-    expect(toolbar).not.toBeInTheDocument()
-  })
-
-  test('shows/hides toolbar on focus when focus is enabled', async () => {
-    render(<TestAdvancedEditor focus />)
-    
-    const kit = screen.getByTestId(testId)
-    
-    // Initially toolbar should be hidden
-    let toolbar = kit.querySelector('.toolbar')
-    expect(toolbar).not.toBeInTheDocument()
-    
-    const editorElement = kit.querySelector('.ProseMirror')
-    // Focus the editor
-    fireEvent.focus(editorElement)
-    
-    // Toolbar should now be visible
-    await waitFor(() => {
-      toolbar = kit.querySelector('.toolbar')
-      expect(toolbar).toBeInTheDocument()
-    })
-  })
-
-
-  test('supports simple prop with advanced editor', () => {
-    render(<TestAdvancedEditor simple />)
-    
-    const kit = screen.getByTestId(testId)
-    const toolbar = kit.querySelector('.toolbar')
-    expect(toolbar).toBeInTheDocument()
-    const toolbarDropdown = kit.querySelector('.editor_dropdown_button')
-    expect(toolbarDropdown).not.toBeInTheDocument()
-    expect(kit).toHaveClass(`${kitClass} simple`)
-  })
-
-    test('supports sticky prop with advanced editor', () => {
-    render(<TestAdvancedEditor sticky />)
-    
-    const kit = screen.getByTestId(testId)
-    const stickyToolbar = kit.querySelector('.pb_rich_text_editor_tiptap_toolbar_sticky')
-    expect(stickyToolbar).toBeInTheDocument()
-    expect(kit).toHaveClass(`${kitClass} sticky`)
-  })
-
-  test('applies aria-label when provided', () => {
-    const ariaLabel = 'Rich Text Editor'
-    render(<TestAdvancedEditor aria={{ label: ariaLabel }} />)
-    
-    const kit = screen.getByTestId(testId)
-    expect(kit).toHaveAttribute('aria-label', ariaLabel)
-  })
-
-  test('supports inline prop with advanced editor', () => {
-    render(<TestAdvancedEditor inline />)
-    
-    const kit = screen.getByTestId(testId)
-    const toolbar = kit.querySelector('.toolbar')
-    expect(toolbar).toBeInTheDocument()
-    expect(kit).toHaveClass(`${kitClass} inline`)
-  })
-
-describe('TipTap Editor Functionality', () => {
-  test('can type and update content', async () => {
-    render(<TestAdvancedEditor />)
-    
-    const kit = screen.getByTestId(testId)
-    const editorContent = kit.querySelector('.ProseMirror')
-    
-    // Focus and type in the editor
-    fireEvent.focus(editorContent)
-    fireEvent.input(editorContent, { 
-      target: { textContent: 'New content' }
-    })
-    
-    await waitFor(() => {
-      expect(editorContent).toHaveTextContent('New content')
-    })
-  })
-})
-})
-

data/app/pb_kits/playbook/pb_selectable_list/selectable_list_item.html.erb

@@ -7,7 +7,7 @@
   <% if object.drag_id && object.drag_handle %>
     <span style="vertical-align: middle;">
       <%= pb_rails("body") do %>
-        <svg width="1.5em" viewBox="0 0 31 25" fill="none" xmlns="http://www.w3.org/2000/svg" color="currentColor" class="pb_custom_icon svg-inline--fa vertical_align_middle  svg_fw"><path d="M12.904 6.355a1.48 1.48 0 01-1.5-1.5c0-.796.656-1.5 1.5-1.5.797 0 1.5.704 1.5 1.5 0 .844-.703 1.5-1.5 1.5zm0 7.5a1.48 1.48 0 01-1.5-1.5c0-.796.656-1.5 1.5-1.5.797 0 1.5.704 1.5 1.5 0 .844-.703 1.5-1.5 1.5zm1.5 6c0 .844-.703 1.5-1.5 1.5a1.48 1.48 0 01-1.5-1.5c0-.796.656-1.5 1.5-1.5.797 0 1.5.704 1.5 1.5zm4.5-13.5a1.48 1.48 0 01-1.5-1.5c0-.796.657-1.5 1.5-1.5.797 0 1.5.704 1.5 1.5 0 .844-.703 1.5-1.5 1.5zm1.5 6c0 .844-.703 1.5-1.5 1.5a1.48 1.48 0 01-1.5-1.5c0-.796.657-1.5 1.5-1.5.797 0 1.5.704 1.5 1.5zm-1.5 9a1.48 1.48 0 01-1.5-1.5c0-.796.657-1.5 1.5-1.5.797 0 1.5.704 1.5 1.5 0 .844-.703 1.5-1.5 1.5z" fill="#242B42"></path></svg>
+        <svg width="auto" height="auto" viewBox="0 0 31 25" fill="none" xmlns="http://www.w3.org/2000/svg" color="currentColor" class="pb_custom_icon svg-inline--fa vertical_align_middle  svg_fw"><path d="M12.904 6.355a1.48 1.48 0 01-1.5-1.5c0-.796.656-1.5 1.5-1.5.797 0 1.5.704 1.5 1.5 0 .844-.703 1.5-1.5 1.5zm0 7.5a1.48 1.48 0 01-1.5-1.5c0-.796.656-1.5 1.5-1.5.797 0 1.5.704 1.5 1.5 0 .844-.703 1.5-1.5 1.5zm1.5 6c0 .844-.703 1.5-1.5 1.5a1.48 1.48 0 01-1.5-1.5c0-.796.656-1.5 1.5-1.5.797 0 1.5.704 1.5 1.5zm4.5-13.5a1.48 1.48 0 01-1.5-1.5c0-.796.657-1.5 1.5-1.5.797 0 1.5.704 1.5 1.5 0 .844-.703 1.5-1.5 1.5zm1.5 6c0 .844-.703 1.5-1.5 1.5a1.48 1.48 0 01-1.5-1.5c0-.796.657-1.5 1.5-1.5.797 0 1.5.704 1.5 1.5zm-1.5 9a1.48 1.48 0 01-1.5-1.5c0-.796.657-1.5 1.5-1.5.797 0 1.5.704 1.5 1.5 0 .844-.703 1.5-1.5 1.5z" fill="#242B42"></path></svg>
       <% end %>
     </span>
   <% end %>