My first CMSms site

[jftaal]

Hi there,

I'am working on a website for a shopping mall in the netherlands. It's still a work in progress altough its nearing its final stage.
So the design is like 99% ready.

Tell me what you think of the design. Any suggestions are welcome!
http://www.desavorninlohmanplein.nl/index.php
Thanks!

[Dr.CSS]

http://validator.w3.org/check?verbose=1 ... Findex.php

[nuno]

Modules used?

[Rolf]

Hi jftaal,

Nice site, but I would suggest to use Pretty Urls

You should check your page aliases.
For instance the alias of the page for Marskramer is gall-gall-4
The alias for Scholtes Notenshop is thomas-cook
For Thomas Cook the alias is thomas-cook2.

Try to use a table-template for your contact-form

Grtz. Rolf 

[jftaal]

@dr ccs, I know about the validation issue, need to look in to that! Thanks for pointing it
out again for me.

[jftaal]

@Rolf | I will look in to those pretty urls. I did know that is was a option.

The table template for the contactform is in deed nicer. But then I get that rounded border thing above it. I need to loose that ugly thing do you know how? Thanks for the feedback!

[jftaal]

@dr css

Hi there I payed some attention to your comment regarding my XHTML code.
http://validator.w3.org/check?verbose=1&uri=http%3A%2F%2Fwww.desavorninlohmanplein.nl%2Findex.php
As you can see 15 errors are now 2 errors. But the last 2 errors are gennerated by CMSms it self.
Any suggestions how to fix this?

[Rolf]

... You have used the attribute named above in your document, but the document type you are using does not support that attribute for this element. This error is often caused by incorrect use of the "Strict" document type with a document that uses frames (e.g. you must use the "Transitional" document type to get the "target" attribute), or by using vendor proprietary extensions such as "marginheight" (this is usually fixed by using CSS to achieve the desired effect instead)...

[jftaal]

Ah great!

I should read it more carefully!
Do you know how to loose the "contact title" with the borderd box around it? It was made with form builder.
tride a few things but it just mesed up the contact page!

Thanks!

Code: Select all

{* TABLE FORM LAYOUT / Field titles on Top *}
{* next line sets number of columns for things like checkbox groups *}
{assign var="cols" value="3"}
{$fb_form_header}
{if $fb_form_done == 1}
	{* This first section is for displaying submission errors *}
	{if $fb_submission_error}
		<div class="error_message">{$fb_submission_error}</div>
		{if $fb_show_submission_errors}
			<table class="error">
			{foreach from=$fb_submission_error_list item=thisErr}
				<tr><td>{$thisErr}</td></tr>
			{/foreach}
			</table>
		{/if}
	{/if}
{else}
	{* this section is for displaying the form *}
	{* we start with validation errors *}
	{if $fb_form_has_validation_errors}
		<div class="error_message">
		<ul>
		{foreach from=$fb_form_validation_errors item=thisErr}
			<li>{$thisErr}</li>
		{/foreach}
		</ul>
		</div>
	{/if}
	{if $captcha_error}
		<div class="error_message">{$captcha_error}</div>
	{/if}

	{* and now the form itself *}
	{$fb_form_start}
	<div>{$fb_hidden}</div>

    <table{if $css_class != ''} class="{$css_class}"{/if}>
    {if $total_pages gt 1}<tr><td colspan="2">{$title_page_x_of_y}</td></tr>{/if}
    {foreach from=$fields item=entry}
	  {if $entry->display == 1 &&
	    $entry->type != '-Fieldset Start' &&
	    $entry->type != '-Fieldset End' }
	    <tr>
	    	{strip}
	    	<td valign="top"
	    	{if $entry->required == 1 || $entry->css_class != ''} class=" 
	    		{if $entry->required == 1}
	    			required
	    		{/if}
	    		{if $entry->required == 1 && $entry->css_class != ''} {/if}
	    		{if $entry->css_class != ''}
	    			{$entry->css_class}
	    		{/if}
	    		"
	    	{/if}
	    	>
	    	{if $entry->hide_name == 0}
	    		{$entry->name}
	    		{if $entry->required_symbol != ''}
	    			{$entry->required_symbol}
	    		{/if}
	    	{/if}
	    	</td></tr><tr><td align="left" valign="top"{if $entry->css_class != ''} class="{$entry->css_class}"{/if}>
	    	{if $entry->multiple_parts == 1}
    		<table>
					<tr>
				{section name=numloop loop=$entry->input}
	    			<td>{$entry->input[numloop]->input} {$entry->input[numloop]->name}{if $entry->input[numloop]->op} {$entry->input[numloop]->op}{/if}</td>
	    			       {if not ($smarty.section.numloop.rownum mod $cols)}
                				{if not $smarty.section.numloop.last}
                        		</tr><tr>
                				{/if}
        					{/if}
       				{if $smarty.section.numloop.last}
                		{math equation = "n - a % n" n=$cols a=$entry->input|@count assign="cells"}
                		{if $cells ne $cols}
                			{section name=pad loop=$cells}
                        		<td> </td>
                			{/section}
               		 	{/if}
                		</tr>
        			{/if}
	    		{/section}
	    		</table>
	    	{else}
    			{if $entry->smarty_eval == '1'}{eval var=$entry->input}{else}{$entry->input}{/if}
	    	{/if}
	    	{if $entry->valid == 0} <--- {$entry->error}{/if}
	    	</td></tr>
	    	{/strip}
	  {/if}
{/foreach}
{if $has_captcha == 1}
<tr><td>{$graphic_captcha}</td></tr><tr><td>{$input_captcha}<br />{$title_captcha}</td></tr>
{/if}
<tr><td>{$prev}</td></tr><tr><td>{$submit}</td></tr>
</table>
{$fb_form_end}
{/if}
{$fb_form_footer}

[Dr.CSS]

The word contact and the border stuff around it are from the feildset and it's legend, and you are missing the closing fieldset...

This makes it easier to style the forms...

http://forum.cmsmadesimple.org/index.ph ... 386.0.html

[jftaal]

@dr CSS

Thanks for the advise!
I fixed it now, in the formbuilder you can add fields ect. In here you also got the legend field and fieldset. It was just a matter of removing those!

[TheGmus]

Very nice!

En dicht in de buurt voor mij, ik ga er het winkelcentrum zeker eens uitchecken

[ruskin012]

Nice design,
It's perfect and everything is well arranged. It will look more beautiful if left and right spaces are minimized.
Try to do it if possible.