Bilgin, so it seems we are OK with: 1) having the attribute boolean like it is now 2) if a entry-name is specified us this instead of name as sortField 3) David suggests also to have the sort-field attribute true by default to have this automatically show up everywhere. I agree with this but in order to have it working we need to have a default for the asc/desc/normal styles as well. Are we ok to have the styles set by default to: sortorder-asc, sortorder-desc, sortorder ?
Bilgin, could I try to update your patch or you are already on that way? -Bruno 2008/9/16 David E. Jones (JIRA) <[EMAIL PROTECTED]> > > [ > https://issues.apache.org/jira/browse/OFBIZ-1943?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12631392#action_12631392] > > David E. Jones commented on OFBIZ-1943: > --------------------------------------- > > Changing it to a String adds a fair bit of complexity, and makes defaulting > to "true" more difficult... though you could default to the field-name it is > more difficult to represent and such. > > I really don't see the point of being able to sort a certain column by a > field other than what is in the column. If someone has a specific scenario > in mind lets discuss how to accomplish that scenario. Otherwise keep it > simple so it is easier to use, more commonly used, and doesn't have features > that are not grounded in an actual requirement. > > It would be great to keep this super-simple and set the default to true so > that this capability shows up everywhere. > > > List/Multi form column sorting > > ------------------------------ > > > > Key: OFBIZ-1943 > > URL: https://issues.apache.org/jira/browse/OFBIZ-1943 > > Project: OFBiz > > Issue Type: Improvement > > Components: ALL COMPONENTS > > Reporter: Bruno Busco > > Priority: Minor > > Attachments: columnSort.diff, columnSort.rar, > framework_images_webapp_images.zip, sortable_form_column.jpg, > sortable_form_columns.patch > > > > > > Hi, > > I would like to submit, to be reviewed, what I have done to implement > sortable columns in multi and list forms. > > I wonder if there is a simpler and better solution that could, for > instance, be implemented in the form widget itself requiring less code in > the xml file. > > To be reviewed I implemented the sortable column in the FindExample > Screen. > > This solution requires a little change in the framework (included in the > patch). Basically the header-link-style attribute of the field tag needs to > be parsed by FlexibleStringExpander in order to be used with a ${}. > > Two little arrow-up and arrow-down icons are also attached, those are > shown close to the field name indicating which is the sorted column and the > sort order. > > Thank you for reviewing, > > Bruno > > -- > This message is automatically generated by JIRA. > - > You can reply to this email to add a comment to the issue online. > >