Keyword
Please note that official support for commercial extensions & templates is provided in the Subscriber Help Desk.
Support requests should ONLY be directed there and require an active subscription plan.
This forum board is to be used for archive purposes and knowledge exchange ONLY.

Simple Image Gallery Plugin Pro - Not XHTML Valid?

  • Royal Bhaktapur
  • Royal Bhaktapur's Avatar Topic Author
  • Offline
  • New Member
More
16 years 11 months ago #3310 by Royal Bhaktapur
Hi Guys,

Just found this site with the beautiful plugin. The functions look really nice. Especially for the greybox plugin and the thumbnails.

Only I'm just building my first joomla site, and want to have everything xhtml valid. However when I check some sites and demo's of this addon, it doesn't seems to be that this is xhtml valid. The w3 validator gives a lot of errors and warnings with the script.
Can anyone clarify / confirm this?

I'm thinking about purchasing this addon to use on my site, but I would like to have it xhtml valid.

Thanks.

Please Log in or Create an account to join the conversation.

More
16 years 11 months ago #3311 by JoomlaWorks
SIG Pro is XHTML valid.  ;)

Fotis / JoomlaWorks Support Team
---
Please search the forum before posting a new topic :)

Please Log in or Create an account to join the conversation.

  • Royal Bhaktapur
  • Royal Bhaktapur's Avatar Topic Author
  • Offline
  • New Member
More
16 years 10 months ago #3312 by Royal Bhaktapur
Replied by Royal Bhaktapur on topic Re: Simple Image Gallery Plugin Pro - Not XHTML Valid?
Hmmmm. Seems to be that there is one error in the script. Getting the following error:
Error  Line 131 column 28: document type does not allow element "div" here; missing one of "object", "applet", "map", "iframe", "button", "ins", "del" start-tag.
<div id="sigid0" class="sig"><div class="sig_cont"><div class="sig_thumb"><a hre
The mentioned element is not allowed to appear in the context in which you've placed it; the other mentioned elements are the only ones that are both allowed there and can contain the element mentioned. This might mean that you need a containing element, or possibly that you've forgotten to close a previous element.

One possible cause for this message is that you have attempted to put a block-level element (such as "<p>" or "<table>") inside an inline element (such as "<a>", "<span>", or "<font>").

Please Log in or Create an account to join the conversation.

More
16 years 10 months ago #3313 by JoomlaWorks
URL please?

Fotis / JoomlaWorks Support Team
---
Please search the forum before posting a new topic :)

Please Log in or Create an account to join the conversation.

  • Royal Bhaktapur
  • Royal Bhaktapur's Avatar Topic Author
  • Offline
  • New Member
More
16 years 10 months ago #3314 by Royal Bhaktapur

Please Log in or Create an account to join the conversation.

More
16 years 10 months ago #3315 by JoomlaWorks
mambots/content/plugin_jw_sigpro.php

line 263, change this:
				// if labels don't exist
				if (!$labels) {
					// display nice message?
					if ($displaymessage) {
						if ($popup_engine == 'greybox') {
						$html .= '<b>'.$itemtitle.'</b>'; // remove nice messages for Greybox
						} else {
						$html .= $message.'
<b>'.$itemtitle.'</b>';
						}
					} else {
					$html .= '<b>'.$images[$a].'</b>';
					}
				}
				
				// if labels do exist
				if ($labels[md5($mosConfig_absolute_path.$rootfolder.$_images_dir_.'/'.$images[$a])]!="") {
					$html .= '<b>'.htmlentities($labels[md5($mosConfig_absolute_path.$rootfolder.$_images_dir_.'/'.$images[$a])],ENT_QUOTES).'</b>';
				}

to this
				// if labels don't exist
				if (!$labels) {
					// display nice message?
					if ($displaymessage) {
						if ($popup_engine == 'greybox') {
						$html .= $itemtitle; // remove nice messages for Greybox
						} else {
						$html .= $message.'
'.$itemtitle;
						}
					} else {
					$html .= $images[$a];
					}
				}
				
				// if labels do exist
				if ($labels[md5($mosConfig_absolute_path.$rootfolder.$_images_dir_.'/'.$images[$a])]!="") {
					$html .= htmlentities($labels[md5($mosConfig_absolute_path.$rootfolder.$_images_dir_.'/'.$images[$a])],ENT_QUOTES);
				}

Fotis / JoomlaWorks Support Team
---
Please search the forum before posting a new topic :)

Please Log in or Create an account to join the conversation.

  • Royal Bhaktapur
  • Royal Bhaktapur's Avatar Topic Author
  • Offline
  • New Member
More
16 years 10 months ago #3316 by Royal Bhaktapur
Replied by Royal Bhaktapur on topic Re: Simple Image Gallery Plugin Pro - Not XHTML Valid?
Thanks. I see that you removed the < b > tags to get it xhtml valid. But unfortunately the validator only gives this a warning. The actual error is given by:

Line 129 column 28: document type does not allow element "div" here; missing one of "object", "applet", "map", "iframe", "button", "ins", "del" start-tag.
<div id="sigid0" class="sig"><div class="sig_cont"><div class="sig_thumb"><a hre

This error shows up at all of the pop up engines. Any idea how to fix this?

Please Log in or Create an account to join the conversation.

More
16 years 10 months ago #3317 by JoomlaWorks
This is a validator error, unless you have the gallery inside some <p> tag. Check your content item for that. Either way, validation is a means of making our site better, not a rule to follow constantly...  ;)

Read the following for some nice insight on the matter:

www.456bereastreet.com/archive/200603/validity_does_not_equal_best_practices/
www.modernlifeisrubbish.co.uk/article/web-standards-dont-matter-as-much-as-you-think

Fotis / JoomlaWorks Support Team
---
Please search the forum before posting a new topic :)

Please Log in or Create an account to join the conversation.

  • Royal Bhaktapur
  • Royal Bhaktapur's Avatar Topic Author
  • Offline
  • New Member
More
16 years 10 months ago - 16 years 10 months ago #3318 by Royal Bhaktapur
Replied by Royal Bhaktapur on topic Re: Simple Image Gallery Plugin Pro - Not XHTML Valid?
Thanks for the links and the advice.
Got the page validated now, it was indeed wrapped inside <p> tags. Those tags were automaticly inserted by the wysig editor.

Please Log in or Create an account to join the conversation.

More
16 years 10 months ago #3319 by JoomlaWorks
Look at the editor settings. It should have a p/br option on "Enter". Switch it to br (break).  ;)

Fotis / JoomlaWorks Support Team
---
Please search the forum before posting a new topic :)

Please Log in or Create an account to join the conversation.


Powered by Kunena Forum