Added todos to language/en/mods/info_acp_portal.php & language/en/mods/portal.php;

Fixed how color palette gets displayed and also fixed the small button areas of the color palette; 
When adding a Module, the users now see the Module name rather than the file name;

root/adm/style/portal/acp_portal_welcome.html

@@ -40,7 +40,6 @@
 		<fieldset>
 			<legend>{L_PORTAL_WELCOME}</legend>
 		<dl>
-			<dt></dt>
 			<dd>
 				<script type="text/javascript">
 				// <![CDATA[
@@ -453,7 +452,7 @@
 							{
 								color = String(numberList[r]) + String(numberList[g]) + String(numberList[b]);
 								document.write('<td bgcolor="#' + color + '" style="width: ' + width + 'px; height: ' + height + 'px;">');
-								document.write('<a href="#" onclick="bbfontstyle(\'[color=#' + color + ']\', \'[/color]\'); return false;"><img src="images/spacer.gif" width="' + width + '" height="' + height + '" alt="#' + color + '" title="#' + color + '" /></a>');
+								document.write('<a href="#" onclick="bbfontstyle(\'[color=#' + color + ']\', \'[/color]\'); return false;" style="margin: -4px 0px -4px 2px;"><img src="images/spacer.gif" width="' + width + '" height="' + (height+10) + '" alt="#' + color + '" title="#' + color + '" style="margin: -4px;" /></a>');
 								document.writeln('</td>');
 							}
 
@@ -534,8 +533,15 @@
 						// <![CDATA[
 							function change_palette()
 							{
-								dE('colour_palette');
 								e = document.getElementById('colour_palette');
+								if(e.style.display == 'block')
+								{
+									dE('colour_palette', -1);
+								}
+								else
+								{
+									dE('colour_palette', 1);
+								}
 								
 								if (e.style.display == 'block')
 								{
@@ -547,7 +553,7 @@
 								}
 							}
 
-							colorPalette('h', 5, 5);
+							colorPalette('h', 10, 5);
 						// ]]>
 						</script>
 					</dt>

root/includes/acp/acp_portal.php

@@ -424,19 +424,31 @@ class acp_portal
 								$c_class = new $class();
 								if ($c_class->columns & column_string_const($add_module))
 								{
-									$fileinfo[] = substr($class, 7, -7);
+									if ($c_class->language)
+									{
+										$user->add_lang('mods/portal/' . $c_class->language);
+									}
+									$fileinfo[] = array(
+										'module'		=> substr($class, 7, -7),
+										'name'			=> $user->lang[$c_class->name],
+									);
 								}
 							}
 						}
 					}
 					closedir($dh);
 
-					sort($fileinfo);
+					// we sort the $fileinfo array by the name of the modules
+					foreach($fileinfo as $key => $cur_file)
+					{
+						$name_ary[$key] = $cur_file['name'];
+					}
+					array_multisort($name_ary, SORT_REGULAR, $fileinfo);
 					$options = '';
 
 					foreach ($fileinfo as $module)
 					{
-						$options .= '<option value="' . $module . '">' . $module . '</option>';
+						$options .= '<option value="' . $module['module'] . '">' . $module['name'] . '</option>';
 					}
 
 					$s_hidden_fields = build_hidden_fields(array(

root/language/en/mods/info_acp_portal.php

@@ -35,6 +35,7 @@ if (empty($lang) || !is_array($lang))
 
 // @todo: check for unneeded language variables
 // @todo: change language variables to English ones
+// @todo: merge into mods/portal/
 
 $lang = array_merge($lang, array(
 	// Manage blocks

root/language/en/mods/portal.php

@@ -32,6 +32,7 @@ if (empty($lang) || !is_array($lang))
 // You do not need this where single placeholders are used, e.g. 'Message %d' is fine
 // equally where a string contains only two placeholders which are used to wrap text
 // in a url you again do not need to specify an order e.g., 'Click %sHERE%s' is fine
+// @todo: merge into mods/portal/
 
 // Common
 $lang = array_merge($lang, array(