[
https://issues.apache.org/jira/browse/OFBIZ-1096?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12508221
]
Adrian Crum commented on OFBIZ-1096:
------------------------------------
Jeremy,
The nav classes are being used, but they don't do anything at this time. The
idea is to style HTML elements using a set of pre-defined semantic classes so
that - some time in the future - those HTML elements can be styled by updating
the main style sheet.
I just took a quick look at your patch, but I haven't applied it. I noticed you
are using table-based layout - we are trying to avoid that. It would be
preferable to use an unordered list for your navigation links.
Check out the bottom of this page:
http://docs.ofbiz.org/display/OFBADMIN/Best+Practices+Guide
> Enhanced pager for Forms
> ------------------------
>
> Key: OFBIZ-1096
> URL: https://issues.apache.org/jira/browse/OFBIZ-1096
> Project: OFBiz
> Issue Type: Improvement
> Components: framework
> Affects Versions: SVN trunk
> Reporter: Wickersheimer Jeremy
> Priority: Minor
> Attachments: form-dtd.patch, HtmlFormRenderer.patch, ModelForm.patch
>
>
> I am currently working on an enhanced pager. The change affects a few files
> in the framework but i tried to make it as small as possible.
> The main enhancement is to add "First" and "Last" button in the pager as well
> as a javascript combobox to quickly jump to any page.
> It is inspired by the Neogia implementation but rewritten to integrate with
> the existing pager.
> I will attach the patches as i apply the changes to the latest SVN trunk
> version.
> PS: I would like to use this issue to have your comments on another pager
> issue. The current implementation doesn't support pages that display more
> than one list form for the obvious reason that the only the parameters
> VIEW_SIZE and VIEW_INDEX are used in the pager, which lacks a parameter to
> differentiate the form.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
