You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ofbiz.apache.org by "Wickersheimer Jeremy (JIRA)" <ji...@apache.org> on 2007/06/25 03:43:25 UTC
[jira] Created: (OFBIZ-1096) Enhanced pager for Forms
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
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.
[jira] Updated: (OFBIZ-1096) Enhanced pager for Forms
Posted by "Adrian Crum (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/OFBIZ-1096?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Adrian Crum updated OFBIZ-1096:
-------------------------------
Attachment: pager.patch
Jeremy,
Your latest patches are much improved. Instead of going back and forth in Jira comments, I applied your patches to my local copy and made some corrections. They are in the pager.patch attachment.
A few things still need to be done:
1. HtmlFormRenderer.java - change the \n characters to appendWhitespace calls
2. ModelForm.java and the widget-form.xsd files - the nav-* properties really aren't needed. As long as you have the PaginateStyle available for the outermost container, anyone can style its contained elements.
3. It would be nice to have the select box appear last. That way, the other elements follow the established pattern:
Previous - 1 of x - Next - Page [select box]
Once those changes are made, then you will have an excellent contribution. I look forward to seeing it in the project.
> 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, form-dtd_2.patch, HtmlFormRenderer-2.patch, HtmlFormRenderer.patch, maincss.patch, ModelForm-2.patch, ModelForm.patch, pager.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.
[jira] Commented: (OFBIZ-1096) Enhanced pager for Forms
Posted by "Adrian Crum (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/OFBIZ-1096?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12508202 ]
Adrian Crum commented on OFBIZ-1096:
------------------------------------
Jeremy,
While you are working on the changes, keep in mind that the maincss.css style sheet has semantic css classes for previous and next links. They are near the end of the file. It would be nice if the widget code defaulted to these classes if no style has been specified in the widget XML file.
http://docs.ofbiz.org/display/OFBIZ/OFBiz+maincss.css+HTML+Element+Collection+Styles
> 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.
[jira] Commented: (OFBIZ-1096) Enhanced pager for Forms
Posted by "Jacques Le Roux (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/OFBIZ-1096?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12508774 ]
Jacques Le Roux commented on OFBIZ-1096:
----------------------------------------
Jeremy,
Actually it's easier for us (and also for you) if you keep alwas the same name for a file. Jira takes care of that. Jut reload with the same name and Jira will "grey" the old fine letting us know which files are the good ones.
Thanks for your work.
> 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: pager-2.patch, pager-2.patch, pager-3.patch, pager.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.
[jira] Updated: (OFBIZ-1096) Enhanced pager for Forms
Posted by "Wickersheimer Jeremy (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/OFBIZ-1096?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Wickersheimer Jeremy updated OFBIZ-1096:
----------------------------------------
Attachment: pager-2.patch
I also changed the CSS, firefox did not take input["select"], so ichanged it to select.
I also added a uiLabel for "rows", this means it will need to be translated.
This should be all right, if you think it still needs a minor finish let me know.
> 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, form-dtd_2.patch, HtmlFormRenderer-2.patch, HtmlFormRenderer.patch, maincss.patch, ModelForm-2.patch, ModelForm.patch, pager-2.patch, pager.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.
[jira] Updated: (OFBIZ-1096) Enhanced pager for Forms
Posted by "Wickersheimer Jeremy (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/OFBIZ-1096?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Wickersheimer Jeremy updated OFBIZ-1096:
----------------------------------------
Attachment: (was: ModelForm.patch)
> 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: pager-2.patch, pager-2.patch, pager-3.patch, pager.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.
[jira] Updated: (OFBIZ-1096) Enhanced pager for Forms
Posted by "Wickersheimer Jeremy (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/OFBIZ-1096?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Wickersheimer Jeremy updated OFBIZ-1096:
----------------------------------------
Attachment: ModelForm-2.patch
New modelForm patch, to be used with the new form DTD
> 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, form-dtd_2.patch, HtmlFormRenderer-2.patch, HtmlFormRenderer.patch, maincss.patch, ModelForm-2.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.
[jira] Commented: (OFBIZ-1096) Enhanced pager for Forms
Posted by "Wickersheimer Jeremy (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/OFBIZ-1096?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12509291 ]
Wickersheimer Jeremy commented on OFBIZ-1096:
---------------------------------------------
One step at a time, i will when this one is closed and applied.
Meanwhile this gives me time to test it on our dev server.
> 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: pager-2.patch, pager-2.patch, pager-3.patch, pager-3.patch, pager.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.
[jira] Commented: (OFBIZ-1096) Enhanced pager for Forms
Posted by "Jacques Le Roux (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/OFBIZ-1096?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12515607 ]
Jacques Le Roux commented on OFBIZ-1096:
----------------------------------------
I will have a look and hopefully commit tomorrow
> 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
> Assignee: Jacques Le Roux
> Priority: Minor
> Attachments: pager-2.patch, pager-2.patch, pager-3.patch, pager-3.patch, pager.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.
[jira] Updated: (OFBIZ-1096) Enhanced pager for Forms
Posted by "Wickersheimer Jeremy (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/OFBIZ-1096?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Wickersheimer Jeremy updated OFBIZ-1096:
----------------------------------------
Attachment: form-dtd.patch
This is the patch for : framework/widget/dtd/widget-form.xsd
It adds these attributes to the form:
- paginate-first-label
- paginate-last-label
- paginate-first-style
- paginate-last-style
- paginate-first-disabled-style
- paginate-previous-disabled-style
- paginate-next-disabled-style
- paginate-last-disabled-style
> 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
>
>
> 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.
[jira] Commented: (OFBIZ-1096) Enhanced pager for Forms
Posted by "Wickersheimer Jeremy (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/OFBIZ-1096?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12508725 ]
Wickersheimer Jeremy commented on OFBIZ-1096:
---------------------------------------------
Iknow what you mean.
I like to keep everything related in the same place, classes to style the Forms are in ModelForm, the rendering done in HtmlFormRenderer can then retrieve those classes consistently instead of mixing hard coding and ModeForm getters.
This is easier for maintainers i think. If later we add code that need those styles, we have only one place to look for them and one place to modify them.
If in the future someone want to change the default CSS of the pager elements i will be very easy. If in the future we decide that we need to add the custom styles in the XML, there will be very little to change in the ModelForm code, and no modifications to make anywhere else in the framework.
For the same reason, i kept the public getter instead of retrieving the constant directly .
> 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, form-dtd_2.patch, HtmlFormRenderer-2.patch, HtmlFormRenderer.patch, maincss.patch, ModelForm-2.patch, ModelForm.patch, pager-2.patch, pager.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.
[jira] Assigned: (OFBIZ-1096) Enhanced pager for Forms
Posted by "Jacques Le Roux (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/OFBIZ-1096?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Jacques Le Roux reassigned OFBIZ-1096:
--------------------------------------
Assignee: Jacques Le Roux
> 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
> Assignee: Jacques Le Roux
> Priority: Minor
> Attachments: pager-2.patch, pager-2.patch, pager-3.patch, pager-3.patch, pager.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.
[jira] Updated: (OFBIZ-1096) Enhanced pager for Forms
Posted by "Wickersheimer Jeremy (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/OFBIZ-1096?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Wickersheimer Jeremy updated OFBIZ-1096:
----------------------------------------
Attachment: pager-3.patch
Fixed bad casting in preparePager
> 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: pager-2.patch, pager-2.patch, pager-3.patch, pager-3.patch, pager.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.
[jira] Commented: (OFBIZ-1096) Enhanced pager for Forms
Posted by "Jacques Le Roux (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/OFBIZ-1096?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12516155 ]
Jacques Le Roux commented on OFBIZ-1096:
----------------------------------------
Thanks Jeremy,
Your patch is in trunk rev. 560524
I just added the CommonOf between lines numbers. Tested with LookupPartyName on Firefox, IE7, and last Opera : works fine
> 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
> Assignee: Jacques Le Roux
> Priority: Minor
> Fix For: SVN trunk
>
> Attachments: pager-2.patch, pager-2.patch, pager-3.patch, pager-3.patch, pager.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.
[jira] Commented: (OFBIZ-1096) Enhanced pager for Forms
Posted by "Jacopo Cappellato (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/OFBIZ-1096?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12507957 ]
Jacopo Cappellato commented on OFBIZ-1096:
------------------------------------------
Jeremy,
I've applied your patch (not yet really reviewed) and I like it; some minor issues:
1) minor formatting issues:
// this is not good
if (condition)
action;
// this is good
if (condition) {
action;
}
and also:
// this is not good
if (condition) {
action;
}
else {
action;
}
// this is good
if (condition) {
action;
} else {
action;
}
2) would it be possible to display the links at the top and at the bottom of the list? (right now they are displayed only at the bottom)
What do other think about this patch?
> 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.
[jira] Updated: (OFBIZ-1096) Enhanced pager for Forms
Posted by "Wickersheimer Jeremy (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/OFBIZ-1096?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Wickersheimer Jeremy updated OFBIZ-1096:
----------------------------------------
Attachment: (was: HtmlFormRenderer-2.patch)
> 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: pager-2.patch, pager-2.patch, pager-3.patch, pager.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.
[jira] Updated: (OFBIZ-1096) Enhanced pager for Forms
Posted by "Wickersheimer Jeremy (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/OFBIZ-1096?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Wickersheimer Jeremy updated OFBIZ-1096:
----------------------------------------
Attachment: form-dtd_2.patch
New form DTD (replace the previous patch)
> 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, form-dtd_2.patch, HtmlFormRenderer-2.patch, HtmlFormRenderer.patch, maincss.patch, ModelForm-2.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.
[jira] Updated: (OFBIZ-1096) Enhanced pager for Forms
Posted by "Wickersheimer Jeremy (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/OFBIZ-1096?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Wickersheimer Jeremy updated OFBIZ-1096:
----------------------------------------
Attachment: pager-3.patch
This patch also contains some modifications to have the pager display on top of the list.
This add a new method in ModelForm named preparePager().
This method is perform the computation needed for the pager also done in renderItemRows(). I tried to only extract the necessary code.
I did not remove the computation code from renderItemRows, so the work is done twice. That is because i only call preparePager for a list, and i did not want to break other things.
This patch is a draft to answer one comment on the possibility to have the pager on top, i would prefer to stick with pager-2.patch for now.
I will work on it further and suggest it as a different Improvement.
> 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, form-dtd_2.patch, HtmlFormRenderer-2.patch, HtmlFormRenderer.patch, maincss.patch, ModelForm-2.patch, ModelForm.patch, pager-2.patch, pager-3.patch, pager.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.
[jira] Updated: (OFBIZ-1096) Enhanced pager for Forms
Posted by "Wickersheimer Jeremy (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/OFBIZ-1096?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Wickersheimer Jeremy updated OFBIZ-1096:
----------------------------------------
Attachment: (was: ModelForm-2.patch)
> 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: pager-2.patch, pager-2.patch, pager-3.patch, pager.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.
[jira] Updated: (OFBIZ-1096) Enhanced pager for Forms
Posted by "Wickersheimer Jeremy (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/OFBIZ-1096?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Wickersheimer Jeremy updated OFBIZ-1096:
----------------------------------------
Attachment: (was: maincss.patch)
> 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: pager-2.patch, pager-2.patch, pager-3.patch, pager.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.
[jira] Commented: (OFBIZ-1096) Enhanced pager for Forms
Posted by "Jacques Le Roux (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/OFBIZ-1096?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12507975 ]
Jacques Le Roux commented on OFBIZ-1096:
----------------------------------------
David,
I guess the discussion you think about is http://www.nabble.com/-RFC--Form-Widget-Stylings-element-%7C%7C-attribute-tf3282030.html and the corresponding Jira issue is
https://issues.apache.org/jira/browse/OFBIZ-760
> 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.
[jira] Commented: (OFBIZ-1096) Enhanced pager for Forms
Posted by "Adrian Crum (JIRA)" <ji...@apache.org>.
[ 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.
[jira] Commented: (OFBIZ-1096) Enhanced pager for Forms
Posted by "Jacques Le Roux (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/OFBIZ-1096?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12509233 ]
Jacques Le Roux commented on OFBIZ-1096:
----------------------------------------
I did not review, nor event test these patch. So I have a suggestion that is maybe already covert. I suggest to allow users to be able to set the size of the VIEW_SIZE
> 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: pager-2.patch, pager-2.patch, pager-3.patch, pager-3.patch, pager.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.
[jira] Commented: (OFBIZ-1096) Enhanced pager for Forms
Posted by "Adrian Crum (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/OFBIZ-1096?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12508576 ]
Adrian Crum commented on OFBIZ-1096:
------------------------------------
Maybe something was misunderstood in my point #2.
If you have this line in HtmlFormRenderer.java:
buffer.append("<div class=\"").append(modelForm.getPaginateStyle()).append("\">");
then you don't need lines like this:
buffer.append("<li class=\"").append(modelForm.getPaginateFirstStyle());
Instead, the contained elements can be styled specifically:
buffer.append("<li class=\"nav-first\"");
If someone wants to change the pagination style, they only need to specify the style for the outermost container (the <div> element) because the CSS classes they use will style the contained elements via descendent selectors (like I did in maincss.css). Doing so will eliminate all of the unnecessary form widget properties.
In other words, the only style the xml file, the model form, and the renderer needs to accomodate is the outermost paginate style.
> 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, form-dtd_2.patch, HtmlFormRenderer-2.patch, HtmlFormRenderer.patch, maincss.patch, ModelForm-2.patch, ModelForm.patch, pager-2.patch, pager.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.
[jira] Commented: (OFBIZ-1096) Enhanced pager for Forms
Posted by "Wickersheimer Jeremy (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/OFBIZ-1096?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12508031 ]
Wickersheimer Jeremy commented on OFBIZ-1096:
---------------------------------------------
Jacopo:
It is possible to display it on Top, i will work on it. In fact i already have the call to render the pager on top in my local version, but some elements necessary to the pager are computed when the list is rendered (such as listSize ...)
In Neogia they made the calculation in the pager rendering code which is imho not elegant.
About the attributes, i just added them to be consistent. And i also noticed that labels are hard coded in English, shouldn't we use uiLabels in the Form code ? I think that would remove any use for the label attributes.
For style attributes, i would not use them i think. If that were up to me i would probably prefer to hard code CSS classes in the pager rendering code and let people change their CSS. In my previous version i had these, in the patch only the classes for the the table cells remains in the code (needed to style the pager buttons with arrow images rather than text).
There is also the FTL issue, they won't share this piece of code ... in my local version i had a set of FTL macros that would perform the same result but i had to import code from Opentaps into the framework to support their custom tag for importing FTL files.
But since most of the FTLs comes along with a BSH i would still prefer to see them disappear :)
> 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.
[jira] Closed: (OFBIZ-1096) Enhanced pager for Forms
Posted by "Jacques Le Roux (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/OFBIZ-1096?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Jacques Le Roux closed OFBIZ-1096.
----------------------------------
Resolution: Fixed
Fix Version/s: SVN trunk
> 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
> Assignee: Jacques Le Roux
> Priority: Minor
> Fix For: SVN trunk
>
> Attachments: pager-2.patch, pager-2.patch, pager-3.patch, pager-3.patch, pager.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.
[jira] Commented: (OFBIZ-1096) Enhanced pager for Forms
Posted by "Wickersheimer Jeremy (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/OFBIZ-1096?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12508490 ]
Wickersheimer Jeremy commented on OFBIZ-1096:
---------------------------------------------
Adrian,
I checked the few changes you made, just one issue: you assign the class "disabled" on disabled buttons, this works for text buttons, but won't with images.
If for example i style nav-first with arrow-first.png, i would want to style the corresponding disabled button with arrow-first-disabled.png (which would be a grayed arrow pointing left).
So i will go back to adding "-disabled" to the base class.
About you other points:
1. Yes why not, in fact i would prefer removing all \n and keep the appendWhitespace at the end of the pager rendering.
2. Indeed.
3. Tricky, the combobox represent the current page, the "1 of X" indicates the records displayed, not the pages.
So IMO the current page should remain in the center.
Else i did not use uiLabels, but i guess i will add "Page" before the combobox, and "rows" after the "1 - X of Y", this will make thing clearer i think.
> 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, form-dtd_2.patch, HtmlFormRenderer-2.patch, HtmlFormRenderer.patch, maincss.patch, ModelForm-2.patch, ModelForm.patch, pager.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.
[jira] Commented: (OFBIZ-1096) Enhanced pager for Forms
Posted by "Wickersheimer Jeremy (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/OFBIZ-1096?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12509261 ]
Wickersheimer Jeremy commented on OFBIZ-1096:
---------------------------------------------
Jacques,
No there is no way to set the view size yet, so it uses the one given
in the form definition of the hard coded default (which can be 50 or
100 if i remember well).
This could be done but it is much less useful for the user.
For which patch to take:
- pager-2 is the implementation of the new pager yes.
- pager-3 is a "beta" that adds the pager on top of the list too, but
i would like to discuss it in a separate thread. There is some code to
rewrite in ModelForm to make things better.
> 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: pager-2.patch, pager-2.patch, pager-3.patch, pager-3.patch, pager.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.
[jira] Commented: (OFBIZ-1096) Enhanced pager for Forms
Posted by "David E. Jones (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/OFBIZ-1096?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12507972 ]
David E. Jones commented on OFBIZ-1096:
---------------------------------------
In general I like this enhancement, and it finally gets some things into the form widget that we've had in the ecommerce category pagination for a long time.
As for the new attributes... there was some discussion a little while back about the fact that there are already too many attributes on the form element. It might be good to start putting all new style attributes in a new sub-element, as was discussed before. I tried looking around for the issue but after 15 minutes I'm out of time on this. I'm sure it's in there somewhere, maybe closed and that's why I couldn't find it in the searches I did.
Anyway, that's the main thing that should probably change so we don't get too much (more!) attribute bloat in the form element. The name of the sub-element was in that issue somewhere and we should probably just with that (unless it makes no sense, I don't remember as it was too long ago we discussed that). Something like "form-styles" or just "styles" would be good, and then "-style" can be removed from all of the attribute names.
> 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.
[jira] Commented: (OFBIZ-1096) Enhanced pager for Forms
Posted by "Wickersheimer Jeremy (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/OFBIZ-1096?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12508377 ]
Wickersheimer Jeremy commented on OFBIZ-1096:
---------------------------------------------
Ok i will rewrite the thing a bit with those changes:
- use nav-* CSS classes by default, i also remove the disabled style because i don't think we need 4 attributes for this. Instead i will use (normal style class) + "-disabled"
- use "nav-pager" as the default style for the pager container, and an attribute paginate-style for the user to override. This makes implementing a custom style for the pager easier because only this style will need to be changed.
- rewrite some if blocks to follow the coding format
- change the table structure for:
<div class="..">
<ul>
<li>..</li>
</ul>
</div>
- patch for maincss.css so that the pager looks the same with the new HTML and classes.
> 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.
[jira] Commented: (OFBIZ-1096) Enhanced pager for Forms
Posted by "Jacques Le Roux (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/OFBIZ-1096?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12509234 ]
Jacques Le Roux commented on OFBIZ-1096:
----------------------------------------
BTW which patch is actually to use : pager2 ? Do we need to delete the others to make things clearer ?
> 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: pager-2.patch, pager-2.patch, pager-3.patch, pager-3.patch, pager.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.
[jira] Updated: (OFBIZ-1096) Enhanced pager for Forms
Posted by "Wickersheimer Jeremy (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/OFBIZ-1096?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Wickersheimer Jeremy updated OFBIZ-1096:
----------------------------------------
Attachment: HtmlFormRenderer-2.patch
New HtmlFormRendere patch.
Correct ed if blocks, and changed HTML.
> 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, form-dtd_2.patch, HtmlFormRenderer-2.patch, HtmlFormRenderer.patch, maincss.patch, ModelForm-2.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.
[jira] Updated: (OFBIZ-1096) Enhanced pager for Forms
Posted by "Wickersheimer Jeremy (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/OFBIZ-1096?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Wickersheimer Jeremy updated OFBIZ-1096:
----------------------------------------
Attachment: maincss.patch
CSS patch
> 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, maincss.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.
[jira] Commented: (OFBIZ-1096) Enhanced pager for Forms
Posted by "Adrian Crum (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/OFBIZ-1096?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12508572 ]
Adrian Crum commented on OFBIZ-1096:
------------------------------------
Jeremy,
I still think the pagination elements make more sense with the select box last. I'd like to hear what others think.
> 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, form-dtd_2.patch, HtmlFormRenderer-2.patch, HtmlFormRenderer.patch, maincss.patch, ModelForm-2.patch, ModelForm.patch, pager-2.patch, pager.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.
[jira] Updated: (OFBIZ-1096) Enhanced pager for Forms
Posted by "Wickersheimer Jeremy (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/OFBIZ-1096?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Wickersheimer Jeremy updated OFBIZ-1096:
----------------------------------------
Attachment: pager-2.patch
Reupload because it was missing the dtd patch.
> 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, form-dtd_2.patch, HtmlFormRenderer-2.patch, HtmlFormRenderer.patch, maincss.patch, ModelForm-2.patch, ModelForm.patch, pager-2.patch, pager-2.patch, pager-3.patch, pager.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.
[jira] Updated: (OFBIZ-1096) Enhanced pager for Forms
Posted by "Wickersheimer Jeremy (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/OFBIZ-1096?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Wickersheimer Jeremy updated OFBIZ-1096:
----------------------------------------
Attachment: ModelForm.patch
This patch is for : framework/widget/src/org/ofbiz/widget/form/ModelForm.java
This adds the necessary attributes, initialization code (in initForm) setters and getters for the added attributes in the XSD.
There is no real "new code" here, all additions were adapted from the existing code.
> 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, 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.
[jira] Commented: (OFBIZ-1096) Enhanced pager for Forms
Posted by "Jacques Le Roux (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/OFBIZ-1096?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12507978 ]
Jacques Le Roux commented on OFBIZ-1096:
----------------------------------------
Ha yes, Adrian is right ! (did not see is comment before posting)
> 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.
[jira] Updated: (OFBIZ-1096) Enhanced pager for Forms
Posted by "Wickersheimer Jeremy (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/OFBIZ-1096?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Wickersheimer Jeremy updated OFBIZ-1096:
----------------------------------------
Attachment: (was: form-dtd_2.patch)
> 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: pager-2.patch, pager-2.patch, pager-3.patch, pager.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.
[jira] Commented: (OFBIZ-1096) Enhanced pager for Forms
Posted by "Jacques Le Roux (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/OFBIZ-1096?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12509263 ]
Jacques Le Roux commented on OFBIZ-1096:
----------------------------------------
Jeremy,
Will you mind opening a new Jira issue for pager-3 ? Then I will delete all the deprecated files in this issue to make the things clearer.
> 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: pager-2.patch, pager-2.patch, pager-3.patch, pager-3.patch, pager.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.
[jira] Commented: (OFBIZ-1096) Enhanced pager for Forms
Posted by "Wickersheimer Jeremy (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/OFBIZ-1096?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12508209 ]
Wickersheimer Jeremy commented on OFBIZ-1096:
---------------------------------------------
Adrian:
You mean the nav-* classes. They are currently unused. But yes we
could use them, and we also need to duplicate them for
disabled-buttons. Or agree on something different like.
Basically the only specifications are that the 4 buttons can each have
a different icon, and two states : enabled or disabled.
The container of the pager (here the table) should also have its class.
But this is not really an important issue for me, i am more interested
in having the pager workin. For the HTML and CSS i would rather let a
commiter decide, I am perfectly fine with the text buttons :)
> 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.
[jira] Commented: (OFBIZ-1096) Enhanced pager for Forms
Posted by "Adrian Crum (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/OFBIZ-1096?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12507974 ]
Adrian Crum commented on OFBIZ-1096:
------------------------------------
The issue David is referring to is OFBIZ-762.
> 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.
[jira] Updated: (OFBIZ-1096) Enhanced pager for Forms
Posted by "Wickersheimer Jeremy (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/OFBIZ-1096?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Wickersheimer Jeremy updated OFBIZ-1096:
----------------------------------------
Attachment: (was: form-dtd.patch)
> 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: pager-2.patch, pager-2.patch, pager-3.patch, pager.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.
Re: [jira] Updated: (OFBIZ-1096) Enhanced pager for Forms
Posted by pierre <pi...@nereide.biz>.
Thank you very much to create a patch for this into ofbiz. I should
create a patch fore this feature but i never had the time ...
Many thanks Jeremy
Pierre
Wickersheimer Jeremy (JIRA) wrote:
> [ https://issues.apache.org/jira/browse/OFBIZ-1096?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
>
> Wickersheimer Jeremy updated OFBIZ-1096:
> ----------------------------------------
>
> Attachment: HtmlFormRenderer.patch
>
> This is the patch for : framework/widget/src/org/ofbiz/widget/html/HtmlFormRenderer.java
>
> This is a rewrite of the renderNextPrev method.
>
>
>> 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.
>>
>
>
--
Pierre Gaudin,
Ingénieur Etudes et Développement
NEREIDE
http://www.nereide.biz
33 (0)2 47 50 30 54
[jira] Updated: (OFBIZ-1096) Enhanced pager for Forms
Posted by "Wickersheimer Jeremy (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/OFBIZ-1096?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Wickersheimer Jeremy updated OFBIZ-1096:
----------------------------------------
Attachment: HtmlFormRenderer.patch
This is the patch for : framework/widget/src/org/ofbiz/widget/html/HtmlFormRenderer.java
This is a rewrite of the renderNextPrev method.
> 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.
[jira] Updated: (OFBIZ-1096) Enhanced pager for Forms
Posted by "Wickersheimer Jeremy (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/OFBIZ-1096?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Wickersheimer Jeremy updated OFBIZ-1096:
----------------------------------------
Attachment: (was: HtmlFormRenderer.patch)
> 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: pager-2.patch, pager-2.patch, pager-3.patch, pager.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.
[jira] Commented: (OFBIZ-1096) Enhanced pager for Forms
Posted by "Wickersheimer Jeremy (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/OFBIZ-1096?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12515493 ]
Wickersheimer Jeremy commented on OFBIZ-1096:
---------------------------------------------
Any update on this ? You seem to like the feature so we could have at pager-2.patch in svn
or maybe pager-3 instead because i use it on my server and did not find any issue so far with the little rewrite.
Or any big change i should make ?
> 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: pager-2.patch, pager-2.patch, pager-3.patch, pager-3.patch, pager.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.