[ http://jira.codehaus.org/browse/DISPL-195?page=comments#action_46742 ] 

Roberta Flashman commented on DISPL-195:
----------------------------------------

There is perhaps a simpler solution to this issue:

Use the existing property in the property file
css.th.sortable={class name from css file for use with anchor in header}

Change the existing code in writeTableHeader():

            // column is sortable, create link
            if (headerCell.getSortable())
            {
                // creates the link for sorting
                Anchor anchor = new Anchor(getSortingHref(headerCell), header);

                // append to buffer
                header = anchor.toString();
            }

to:

            // column is sortable, create link
            if (headerCell.getSortable())
            {
                // creates the link for sorting
                Anchor anchor = new Anchor(getSortingHref(headerCell), header);

                //  when a header cell is sortable, 
                // create a class attribute for the anchor
                // - IE compatibility              
                String cssSortable = this.properties.getCssSortable();
                if(cssSortable != null){
                  anchor.setClass(cssSortable);
                }


                // append to buffer
                header = anchor.toString();
            }


> Allow for customization of Sorting Anchors
> ------------------------------------------
>
>          Key: DISPL-195
>          URL: http://jira.codehaus.org/browse/DISPL-195
>      Project: DisplayTag
>         Type: Improvement
>   Components: Paging/Sorting
>     Reporter: Peter Luttrell

>
>
> DisplayTag provides the ability to override the output of the anchors used 
> for pagination, but not sorting. I need this to be able to use the DisplayTag.
> Here is the code to implement it:
> Add the following to TableProperties.java:
>    /**
>      * property <code>sort.th.link</code>.
>      */
>     public static final String PROPERTY_STRING_SORT_TH_LINK = "sort.th.link"; 
> //$NON-NLS-1$
>   /**
>     * Getter for the <code>PROPERTY_STRING_SORT_TH_LINK</code> property.
>     * @return String
>     */
>    public String getSortThLink()
>    {
>        return getProperty(PROPERTY_STRING_SORT_TH_LINK);
>    }
> Add the following to TableTag.properties:
>    sort.th.link=<a href="{0}">{1}</a>
> In TableTag.java, in the writeTableHeader method, replaced this code:
>    // creates the link for sorting
>    Anchor anchor = new Anchor(getSortingHref(headerCell), header);
>    // append to buffer
>    header = anchor.toString();
> with:
>    // create Object[] including the HREF for the sort and the header text
>    Object[] objects = {getSortingHref(headerCell).toString(), header};
>    // get sort th link
>    String sortedLinkFormat = this.properties.getSortThLink();
>    // merge the final result
>    header = MessageFormat.format(sortedLinkFormat, objects);
> The docs would need to change to reflect the additional configuration 
> parameter.
>    Name: sort.th.link
>    Default: <a href="{0}">{1}</a>
>    Description: Link to get you back to this page. Displayed in the column th 
> for sorting. {0}: Page Url; {1}: Column Title

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira



-------------------------------------------------------
SF.Net email is sponsored by:
Tame your development challenges with Apache's Geronimo App Server. Download
it for free - -and be entered to win a 42" plasma tv or your very own
Sony(tm)PSP.  Click here to play: http://sourceforge.net/geronimo.php
_______________________________________________
displaytag-devel mailing list
displaytag-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/displaytag-devel

Reply via email to