However I think the current behavior is unintuitive... if there are dynamically sortable columns, maybe the css class should be dynamically set too. Just to give context, I want to tweak my css to not show the unsortable columns as clickable, which I think it is a common choice.

Thanks for this investigation. I know for this and maybe it will stay so.

Do not think that this is a bug. The reason is that you have dynamicall sorting columns. Also using setColProp make this possible.

Regards

Tony

This patch fixes it:

From f2d851e6e93a498a34b0efe32975c75079b873bf Mon Sep 17 00:00:00 2001
From: Paolo Borelli <pborelli@gnome.org>
Date: Thu, 17 Sep 2009 19:53:40 +0200
Subject: [PATCH 1/2] Do not mark unsortable cols with sortable css class

---
 js/grid.base.js |    8 ++++----
 1 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/js/grid.base.js b/js/grid.base.js
index 36365a5..81bb7b6 100644
--- a/js/grid.base.js
+++ b/js/grid.base.js
@@ -1150,11 +1150,11 @@ $.fn.jqGrid = function( p ) {
             sort = ts.p.colModel[j].sortable;
             if( typeof sort !== 'boolean') {ts.p.colModel[j].sortable =  true; sort=true;}
             var nm = ts.p.colModel[j].name;
-            if( !(nm == 'cb' || nm=='subgrid' || nm=='rn') ) {
-                if(ts.p.viewsortcols[2])
-                    $("div",this).addClass('ui-jqgrid-sortable');
-            }
             if(sort) {
+                if( !(nm == 'cb' || nm=='subgrid' || nm=='rn') ) {
+                    if(ts.p.viewsortcols[2])
+                        $("div",this).addClass('ui-jqgrid-sortable');
+                }
                 if(ts.p.viewsortcols[0]) {$("div span.s-ico",this).show(); if(j==ts.p.lastsort){ $("div span.ui-icon-"+ts.p.sortorder,this).removeClass("ui-state-disabled");}}
                 else if( j == ts.p.lastsort) {$("div span.s-ico",this).show();$("div span.ui-icon-"+ts.p.sortorder,this).removeClass("ui-state-disabled");}
             }
--
1.6.2.5