I think we should handle it like the other uicommand components and set the default immediate flag to false. It is quite surprisingly for the user who don´t care about this attribute and expects a different behavior because he thinks that every command component is working like that way.
If the user want´s to do some fine tuning and don´t need the standard behavior he can switch the immediate flag to true. A second thing I came across with was that the usage of the sortheader component is not very intuitive for the user. He has to define 3 attributes - 2 for the datatable and one for the sortheader component. I think it´s better to define a single action like method attribute which expects a special signature like: void <methodBinding>(boolean ascending); For each column to sort the user normaly defines individual action methods. If the user wants a more dynamic way he could use an <sort>ActionListener which can be registered instead of the action. The signature of that listener could be: Void <methodBinding>(SortEvent event); The event contains the component association which can be used to determine the column and the sort direction. That would also make it possible to use the sortheader component in a normal h:datatable To make it backward compatible we should create a x:commandSortHeader2 component. Mathias > -----Original Message----- > From: Manfred Geiler [mailto:[EMAIL PROTECTED] > Sent: Thursday, July 07, 2005 2:52 PM > To: MyFaces Development > Subject: Re: Why is immediate=true default for HtmlCommandSortHeader > > > Hmm, quite a long time ago... > > One thing I have in mind is this: > For simple tables (without input components) the validation, model > update and so on is not necessary. And even for complex tables it is > the question if it is desireable by default to do those things and > display error messages if the user only wanted to resort the table. > i.e. He just wants to change his view. A user clicking on a sort > header does not normally have the intention to submit his entries. > > WDYT? > > -Manfred > > > 2005/7/7, Broekelmann, Mathias <[EMAIL PROTECTED]>: > > Hi, > > > > Does anyone know why the immediate flag defaults to true for the > > sortheader component? IMO we should use the usual default > false for the > > immediate attribute. > > > > Mathias > > >
