Ticket #767 (closed enhancement: fixed)

Opened 4 months ago

Last modified 4 months ago

Make number of items in form and panel draft lists configurable

Reported by: vakopian Owned by:
Priority: minor Milestone: 2.1
Component: Framework Version: 2.0.2 Stable
Keywords: Cc:

Description

Currently the number of items in the datagrids in the form and panel drafts are hard-coded to 20 and 8. The attached patch makes them configurable through the configuration.inc.php

Attachments

ticket767.patch Download (2.3 KB) - added by vakopian 4 months ago.

Change History

Changed 4 months ago by vakopian

Changed 4 months ago by alex94040

  • status changed from new to in_QA

I really hate that we keep expanding the configuration file to add more and more options. These options are valuable, but they are far from critically important. What do you think about splitting out "most important" config - and optional config - as two separate files?

Changed 4 months ago by vakopian

I don't mind that. We could also just put these unimportant options at the bottom, after a big block of comments, just to avoid the decision of which config file each setting should go, plus the (small) overhead of including two files instead of one. Pick whichever way you prefer.

Changed 4 months ago by alex94040

Fair argument - we can just keep the unimportant stuff at bottom. I don't feel that this is critical enough to argue about :) Go ahead and apply the patch! Thanks for doing this.

Changed 4 months ago by vakopian

  • status changed from in_QA to closed
  • resolution set to fixed

(In [1386]) reviewed by alex. Fixes #767

Note: See TracTickets for help on using tickets.