Fix disabled and readonly on trix field (#3939)

* Initial approach

* Fix Layout/TrailingEmptyLines lint CI failure

* Add tests for the bug fix

* Fix Style/StringLiterals lint CI failure

* Refactor to use component_for_view method

* Fix Lint CI Failure

* Fix failing system and feature tests

* tweaks

* view inquirer

---------

Co-authored-by: Paul Bob <[email protected]>
Co-authored-by: Paul Bob <[email protected]>

Author: zhephyn

lib/avo/fields/trix_field.rb

@@ -37,6 +37,16 @@ def attachments_disabled
         # Disable attachments if attachment_key is not present
         @attachment_key.blank?
       end
+
+      def component_for_view(view)
+        view = Avo::ViewInquirer.new(view)
+
+        if view.edit? && (is_readonly? || is_disabled?)
+          Avo::Fields::TrixField::ShowComponent
+        else
+          super
+        end
+      end
     end
   end
 end

spec/system/avo/trix_field_spec.rb

@@ -125,6 +125,44 @@
       end
     end
   end
+
+  describe "trix field with disabled/readonly set to true" do
+    let!(:body) { "This is test content for disabled state" }
+    let!(:post) { create :post, body: body }
+
+    context "when disabled is set to true" do
+      it "displays show page content when disabled is set to true for a trix field" do
+        Avo::Resources::Post.with_temporary_items do
+          field :body, as: :trix, disabled: true
+        end
+
+        visit "/admin/resources/posts/#{post.id}/edit"
+
+        expect(page).not_to have_selector("trix-editor")
+
+        expect(page).to have_selector(".trix-content")
+        expect(page).to have_text(body)
+
+        Avo::Resources::Post.restore_items_from_backup
+      end
+    end
+    context "when readonly is set to true" do
+      it "displays show page content when readonly is set to true for a trix field" do
+        Avo::Resources::Post.with_temporary_items do
+          field :body, as: :trix, readonly: true
+        end
+
+        visit "/admin/resources/posts/#{post.id}/edit"
+
+        expect(page).not_to have_selector("trix-editor")
+
+        expect(page).to have_selector(".trix-content")
+        expect(page).to have_text(body)
+
+        Avo::Resources::Post.restore_items_from_backup
+      end
+    end
+  end
 end
 
 def fill_in_trix_editor(id, with:)