ClipboardCopy only needs aria-label when empty

.changeset/good-coats-shout.md

@@ -0,0 +1,5 @@
+---
+'@primer/view-components': patch
+---
+
+Stop requiring aria-label on ClipboardCopy with other content.

app/components/primer/clipboard_copy.rb

@@ -12,7 +12,7 @@ class ClipboardCopy < Primer::Component
     #   <%= render(Primer::ClipboardCopy.new(value: "Text to copy", "aria-label": "Copy text to the system clipboard")) %>
     #
     # @example With text instead of icons
-    #   <%= render(Primer::ClipboardCopy.new(value: "Text to copy", "aria-label": "Copy text to the system clipboard")) do %>
+    #   <%= render(Primer::ClipboardCopy.new(value: "Text to copy")) do %>
     #     Click to copy!
     #   <% end %>
     #
@@ -34,10 +34,14 @@ def initialize(value: nil, **system_arguments)
       @system_arguments[:value] = value if value.present?
     end
 
+    # :nodoc:
+    def before_render
+      validate_aria_label if content.blank?
+    end
+
     private
 
     def validate!
-      validate_aria_label
       raise ArgumentError, "Must provide either `value` or `for`" if @value.nil? && @system_arguments[:for].nil?
     end
   end

test/components/clipboard_copy_test.rb

@@ -14,8 +14,16 @@ def test_renders_simple
     end
   end
 
+  def test_requires_aria_label_when_empty
+    assert_raises(ArgumentError) do
+      render_inline Primer::ClipboardCopy.new(value: "my-branch-name") do
+        nil
+      end
+    end
+  end
+
   def test_renders_with_text_contents
-    render_inline Primer::ClipboardCopy.new(value: "my-branch-name", "aria-label": "Copy branch name to clipboard") do
+    render_inline Primer::ClipboardCopy.new(value: "my-branch-name") do
       "Click to copy!"
     end