@rip-lang/ui 0.3.44 → 0.3.46

package/components/accordion.rip

@@ -48,7 +48,7 @@ export Accordion = component
         trigger.id = triggerId
         trigger.setAttribute 'aria-expanded', isOpen
         trigger.setAttribute 'aria-controls', panelId
-        trigger.setAttribute 'aria-disabled', isDisabled if isDisabled
+        if isDisabled then trigger.setAttribute 'aria-disabled', true else trigger.removeAttribute 'aria-disabled'
         trigger.tabIndex = if isDisabled then -1 else 0
       if content
         content.id = panelId
@@ -80,7 +80,7 @@ export Accordion = component
       first:  => @_focusTrigger(0)
       last:   => @_focusTrigger(-1)
       select: => @toggle(id) unless disabled
-    }
+    }, 'vertical'
 
   _triggers: ->
     return [] unless @_content

package/components/autocomplete.rip

@@ -93,7 +93,7 @@ export Autocomplete = component
       dismiss: => @close()
       tab:     => @close()
 
-  ~> ARIA.popupDismiss open, (=> @_list), (=> @close()), [=> @_input]
+  ~> ARIA.popupDismiss open, (=> @_list), (=> @close()), [=> @_input], (=> @_position())
 
   mounted: ->
     @_hlIdx = -1

package/components/breadcrumb.rip

@@ -31,6 +31,7 @@ export Breadcrumb = component
     return unless _ready
     items = _items
     return unless items.length
+    @_content?.style.setProperty '--breadcrumb-separator', JSON.stringify(@separator)
     items.forEach (el, idx) =>
       isLast = idx is items.length - 1
       if isLast

package/components/collapsible.rip

@@ -36,7 +36,7 @@ export Collapsible = component
     content = @_root?.querySelector('[data-content]')
     if trigger
       trigger.setAttribute 'aria-expanded', !!@open
-      trigger.setAttribute 'aria-disabled', true if @disabled
+      if @disabled then trigger.setAttribute 'aria-disabled', true else trigger.removeAttribute 'aria-disabled'
       trigger.tabIndex = if @disabled then -1 else 0
     if content
       content.hidden = not @open

package/components/combobox.rip

@@ -86,14 +86,14 @@ export Combobox = component
     len = @getItems().length
     ARIA.listNav e,
       next:    => @openMenu() unless open; highlightedIndex = @_nextEnabled(highlightedIndex,  1); @_scrollToItem()
-      prev:    => highlightedIndex = @_nextEnabled(highlightedIndex, -1); @_scrollToItem()
+      prev:    => @openMenu() unless open; highlightedIndex = @_nextEnabled(highlightedIndex, -1); @_scrollToItem()
       first:   => highlightedIndex = 0;       @_scrollToItem()
       last:    => highlightedIndex = len - 1; @_scrollToItem()
       select:  => if highlightedIndex >= 0 then @selectIndex(highlightedIndex) else if len is 1 then @selectIndex(0)
       dismiss: => if open then @close() else @query = ''
       tab:     => @close()
 
-  ~> ARIA.popupDismiss open, (=> @_list), (=> @close()), [=> @_content]
+  ~> ARIA.popupDismiss open, (=> @_list), (=> @close()), [=> @_content], (=> @_position())
 
   render
     . ref: "_content", $open: open?!

package/components/editable-value.rip

@@ -25,6 +25,7 @@ export EditableValue = component
     return if saving
     saving = true
     @emit 'save'
+    @close()
 
   _onCancel: ->
     editing = false
@@ -50,11 +51,12 @@ export EditableValue = component
     editor.querySelector('input, textarea, select')?.focus()
 
   ~>
+    _editing = editing                                 # track before any early return
     display = @_root?.querySelector('[data-display]')
     editor = @_root?.querySelector('[data-editor]')
     return unless display and editor
-    editor.hidden = not editing
-    if editing
+    editor.hidden = not _editing
+    if _editing
       editor.setAttribute 'data-open', ''
       onDown = (e) =>
         unless @_root?.contains(e.target)

package/components/menu.rip

@@ -110,7 +110,7 @@ export Menu = component
       tab:     => @close()
       char:    => @_typeahead(e.key)
 
-  ~> ARIA.popupDismiss open, (=> @_list), (=> @close()), [=> @_trigger]
+  ~> ARIA.popupDismiss open, (=> @_list), (=> @close()), [=> @_trigger], (=> @_position())
 
   render
     .

package/components/native-select.rip

@@ -17,6 +17,7 @@ export NativeSelect = component
   focused := false
 
   onChange: (e) ->
+    e.isTrusted or return
     @value = e.target.value
     @emit 'change', @value
 

package/components/popover.rip

@@ -99,6 +99,7 @@ export Popover = component
     floating.style.left = "#{x}px"
     floating.style.top = "#{y}px"
     floating.style.zIndex = '50'
+    floating.style.visibility = 'visible'
 
   ~>
     return unless _ready
@@ -109,6 +110,7 @@ export Popover = component
     if floating
       floating.hidden = not open
       if open
+        floating.style.visibility = 'hidden'
         floating.setAttribute 'data-open', ''
         floating.setAttribute 'data-placement', @placement
         heading = floating.querySelector('h1,h2,h3,h4,h5,h6')

package/components/select.rip

@@ -117,7 +117,7 @@ export Select = component
       @_list.style.top = "#{tr.top - fl.height - 4}px"
     @_list.style.visibility = 'visible'
 
-  ~> ARIA.popupDismiss open, (=> @_list), (=> @close()), [=> @_trigger]
+  ~> ARIA.popupDismiss open, (=> @_list), (=> @close()), [=> @_trigger], (=> @_position())
 
   render
     .

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rip-lang/ui",
-  "version": "0.3.44",
+  "version": "0.3.46",
   "author": "Steve Shreeve <steve.shreeve@gmail.com>",
   "repository": {
     "type": "git",
@@ -35,6 +35,6 @@
   "license": "MIT",
   "type": "module",
   "dependencies": {
-    "rip-lang": ">=3.13.98"
+    "rip-lang": ">=3.13.99"
   }
 }