CouchCMS • View topic - Repeatable regions

nfwd · **Joined:** Thu Jun 21, 2012 1:44 am **Posts:** 36

You know I love Couch. It is 99% perfect, and I'm being really picky here

1) *BUG* Repeatable regions don't work well with groups in the admin panel. The repeatable table is too wide for the group, and the red X (delete) is half disappearing. I've tried playing with col_width, which is fine for individual columns but doesn't seem to reduce the width of the table enough (in Chrome, anyway).

2) *FEATURE REQUEST* A repeatable row doesn't allow much room in the admin panel. I propose a new tag to allow multiple rows, per repeating row ... if that makes sense. Something like this:

Code:

<cms:repeatable etc>
  <cms:row>
    <cms:editable etc />
    <cms:editable etc />
  </cms:row>
  <cms:row>
    <cms:editable etc />
    <cms:editable etc />
  </cms:row>
</cms:repeatable>

3) *FEATURE REQUEST* Allow us to define minimum and maximum rows per repeatable region.

Code:

<cms:repeatable min_rows='2'> /* Two rows minimum */

Code:

<cms:repeatable max_rows='10'> /* Ten rows maximum */

4) * FEATURE REQUEST * Allow custom config in CKEditor. At the moment I am directly editing /couch/includes/ckeditor/config.js so I can do the following:

Code:

CKEDITOR.editorConfig = function( config )
{
   config.forcePasteAsPlainText = true;
};

This would be easier (and upgrade-proof) if we could have a new parameter in type='richtext' :

Code:

<cms:editable name='blah'
    custom_config="styles/custom_config.js"
    type='richtext'/>

5) * FEATURE REQUEST * Image editor improvements, specifically 'delete images' and view as thumbnails instead of list. If deleting an image will result in a broken path elsewhere, a warning could be displayed. I built a basic CMS a few years ago that displayed the following message:

Code:

You are trying to delete image X, which is currently being used in the following pages:
[Link to page]
[Link to page]
Please remove the image from these pages, and try again.

Or something like that

6) * FEATURE REQUEST * I've mentioned this before, but allowing groups to stay open in the admin panel simultaneously. You've already agreed to add this in a future version anyway.

6) * FEATURE REQUEST * Couch currently spits out loads of whitespace in the source code. It appears to generate a new blank line for each tag, even if there is no output from the tag. It doesn't break anything as such, it just makes html pages larger than they need to be and tough to read. But now I'm being really picky

KK · **Joined:** Wed Dec 01, 2010 5:35 pm **Posts:** 1395

Interesting set of requests these

The 'repeating rows' feature seems very practical.

I've had a similar list from another user today regarding repeatable regions - one additional feature demanded was for the rows to be numbered.

Thank you for your inputs. I'll definitely try and implement as many of them as feasible.

nfwd · **Joined:** Thu Jun 21, 2012 1:44 am **Posts:** 36

Quote:

1) *BUG* Repeatable regions don't work well with groups in the admin panel. The repeatable table is too wide for the group, and the red X (delete) is half disappearing. I've tried playing with col_width, which is fine for individual columns but doesn't seem to reduce the width of the table enough (in Chrome, anyway).

Just so you know, I've fixed this one:

Code:

/couch/addons/repeatable/tablegear/tablegear.css

.rr { background: #fff; border-left: 1px solid #d3d3d3; border-top: 1px solid #d3d3d3; -moz-border-radius: 5px; border-radius: 5px; min-width:720px; table-layout: fixed; }

Removing 'min-width:720px;' fixes this problem.

Lewis

cheesypoof · **Joined:** Thu Oct 13, 2011 2:48 am **Posts:** 262

I don't think you can fault Couch for the white space issue. This is totally dependent on how you arrange your tags and thus can be easily prevented.

nfwd · **Joined:** Thu Jun 21, 2012 1:44 am **Posts:** 36

Quote:

I don't think you can fault Couch for the white space issue. This is totally dependent on how you arrange your tags and thus can be easily prevented.

What I mean, is that if you have a page with a lot of logic (if, each, set etc.) then each tag is generating a new line of white space.

For example purposes :

Code:

<cms:set i='1' />
<cms:set i='2' />
<cms:set i='3' />
<cms:set i='4' />
<cms:set i='5' />
<cms:set i='6' />
<cms:set i='7' />
<cms:set i='8' />

This generates eight lines of white space in the source code.

Code:

<?php $i=1; ?>
<?php $i=2; ?>
<?php $i=3; ?>
<?php $i=4; ?>
<?php $i=5; ?>
<?php $i=6; ?>
<?php $i=7; ?>
<?php $i=8; ?>

This doesn't generate any whitespace.

As you say, arranging your tags differently can fix the problem:

Code:

<cms:set i='1' /><cms:set i='2' /><cms:set i='3' /><cms:set i='4' /><cms:set i='5' /><cms:set i='6' /><cms:set i='7' /><cms:set i='8' />

This only generates a single line of whitespace, but it's not practical.

I only notice it because I've come from Expression Engine which also uses plenty of similar tags but doesn't have the same output.

Anyway, it is such a tiny thing that I didn't really need to mention it. I did say I was being picky

Lewis

cheesypoof · **Joined:** Thu Oct 13, 2011 2:48 am **Posts:** 262

I agree, the way it is handled in Couch there seems to be two less than desirable options: maintain readability and therefore produce excessive white space, or produce no white space and lose readability. Maybe the answer is to introduce a code formatter, that can be toggled by a parameter at the template level:

Code:

<cms:template title='Contact' formatter='minify' />

Another option could be 'beautify'. These would alter the way the code is output, by either adding or removing white space depending on the parameter value.

nfwd · **Joined:** Thu Jun 21, 2012 1:44 am **Posts:** 36

For CSS and JS I have been using this :

http://code.google.com/p/minify/

There is a PHP class that minifies HTML :

http://code.google.com/p/minify/source/ ... y/HTML.php

They recommend that it isn't used on dynamic pages (since it isn't fast) but it could be used in combination with Couch's cache setting. Readability is only useful during development, but for a live site serving up cached HTML pages, I propose this:

Code:

  // 9.
    // If set, CMS will cache generated pages and serve them if possible.
    define( 'K_USE_CACHE', 1 );

  // 9.a
    // Minify cached files.
    define( 'K_MINIFY', 1 );

Lewis

MGMT · **Joined:** Sun Jul 15, 2012 6:50 pm **Posts:** 13

Quote:

5) * FEATURE REQUEST * Image editor improvements, specifically 'delete images' and view as thumbnails instead of list. If deleting an image will result in a broken path elsewhere, a warning could be displayed.

I am looking for this feature as well. It would be great if you could have a thumbnail view and if you could delete images.

Quote:

Code:

You are trying to delete image X, which is currently being used in the following pages:
[Link to page]
[Link to page]
Please remove the image from these pages, and try again.

That would be amazing!