Cleanup #11989: Install Tool message colors should match flash message colors
authorFrancois Suter <francois.suter@typo3.org>
Sun, 20 Sep 2009 16:49:55 +0000 (16:49 +0000)
committerFrancois Suter <francois.suter@typo3.org>
Sun, 20 Sep 2009 16:49:55 +0000 (16:49 +0000)
git-svn-id: https://svn.typo3.org/TYPO3v4/Core/trunk@6018 709f56b5-9817-0410-a4d7-c38de5d9e867

ChangeLog
typo3/sysext/install/mod/class.tx_install.php
typo3/sysext/install/mod/install.css

index 870df36..44ba3b5 100755 (executable)
--- a/ChangeLog
+++ b/ChangeLog
@@ -14,6 +14,7 @@
        * Follow-up to UI-Bug #F3282: Changed icon_warning.gif to match flash messages' warning icon
        * Fixed bug #11995: Prompt for keyboard input does not get displayed in CLI scripts
        * Fixed bug #11224: Special menu directory only renders 1st level if special.value is a mount point (Thanks to Xavier Perseguers)
+       * Cleanup #11989: Install Tool message colors should match flash message colors
 
 2009-09-20  Benjamin Mack  <benni@typo3.org>
 
index 8911184..1b91aec 100755 (executable)
@@ -552,16 +552,16 @@ REMOTE_ADDR was '".t3lib_div::getIndpEnv('REMOTE_ADDR')."' (".t3lib_div::getIndp
 
 
                                        $headCode='Header legend';
+                                       $this->message($headCode, 'Notice!', '
+                                       Indicates that something is important to be aware of.
+                                       This does <em>not</em> indicate an error.
+                                       ',1);
                                        $this->message($headCode, 'Just information', '
                                        This is a simple message with some information about something.
                                        ');
                                        $this->message($headCode, 'Check was successful', '
                                        Indicates that something was checked and returned an expected result.
                                        ',-1);
-                                       $this->message($headCode, 'Notice!', '
-                                       Indicates that something is important to be aware of.
-                                       This does <em>not</em> indicate an error.
-                                       ',1);
                                        $this->message($headCode, 'Warning!', '
                                        Indicates that something may very well cause trouble and you should definitely look into it before proceeding.
                                        This indicates a <em>potential</em> error.
@@ -3371,7 +3371,7 @@ From sub-directory:
                $out='';
                $out.='<tr>
                                <td nowrap="nowrap"><strong>'.$this->fw('Update required tables').'</strong></td>
-                               <td'.($action_type=='cmpFile|CURRENT_TABLES'?' class="note"':'').'>'.$this->fw('<a href="'.htmlspecialchars($this->action.'&TYPO3_INSTALL[database_type]=cmpFile|CURRENT_TABLES#bottom').'"><strong>COMPARE</strong></a>').'</td>
+                               <td' . ($action_type == 'cmpFile|CURRENT_TABLES' ? ' class="notice"' : '') . '>' . $this->fw('<a href="' . htmlspecialchars($this->action . '&TYPO3_INSTALL[database_type]=cmpFile|CURRENT_TABLES#bottom') . '"><strong>COMPARE</strong></a>') . '</td>
                                <td>'.$this->fw('&nbsp;').'</td>
                                <td>'.$this->fw('&nbsp;').'</td>
                        </tr>';
@@ -3379,7 +3379,7 @@ From sub-directory:
                $out.='<tr>
                                <td nowrap="nowrap"><strong>'.$this->fw('Dump static data').'</strong></td>
                                <td>'.$this->fw('&nbsp;').'</td>
-                               <td nowrap="nowrap"'.($action_type=='import|CURRENT_STATIC'?' class="note"':'').'>'.$this->fw('<a href="'.htmlspecialchars($this->action.'&TYPO3_INSTALL[database_type]=import|CURRENT_STATIC#bottom').'"><strong>IMPORT</strong></a>').'</td>
+                               <td nowrap="nowrap"' . ($action_type == 'import|CURRENT_STATIC' ? ' class="notice"' : '').'>' . $this->fw('<a href="' . htmlspecialchars($this->action . '&TYPO3_INSTALL[database_type]=import|CURRENT_STATIC#bottom') . '"><strong>IMPORT</strong></a>') . '</td>
                                <td>'.$this->fw('&nbsp;').'</td>
                        </tr>';
 
@@ -3401,27 +3401,27 @@ From sub-directory:
 
                        $out.='<tr>
                                <td nowrap="nowrap">'.$this->fw($fShortName.' ('.t3lib_div::formatSize(filesize($file)).')').'</td>
-                               <td'.($action_type=='cmpFile|'.$file?' class="note"':'').'>'.$this->fw('<a href="'.htmlspecialchars($this->action.'&TYPO3_INSTALL[database_type]=cmpFile|'.rawurlencode($file).'#bottom').'"><strong>COMPARE</strong></a>').'</td>
-                               <td nowrap="nowrap"'.($action_type=='import|'.$file?' class="note"':'').'>'.$this->fw('<a href="'.htmlspecialchars($this->action.'&TYPO3_INSTALL[database_type]=import|'.rawurlencode($file).'#bottom').'"><strong>IMPORT'.$spec1.$spec2.'</strong></a>').'</td>
-                               <td nowrap="nowrap"'.($action_type=='view|'.$file?' class="note"':'').'>'.$this->fw('<a href="'.htmlspecialchars($this->action.'&TYPO3_INSTALL[database_type]=view|'.rawurlencode($file).'#bottom').'"><strong>VIEW'.$spec1.$spec2.'</strong></a>').'</td>
+                               <td' . ($action_type == 'cmpFile|' . $file ? ' class="notice"' : '') . '>' . $this->fw('<a href="' . htmlspecialchars($this->action . '&TYPO3_INSTALL[database_type]=cmpFile|' . rawurlencode($file) . '#bottom') . '"><strong>COMPARE</strong></a>') . '</td>
+                               <td nowrap="nowrap"' . ($action_type == 'import|' . $file ? ' class="notice"' : '') . '>' . $this->fw('<a href="' . htmlspecialchars($this->action . '&TYPO3_INSTALL[database_type]=import|' . rawurlencode($file) . '#bottom') . '"><strong>IMPORT' . $spec1 . $spec2 . '</strong></a>') . '</td>
+                               <td nowrap="nowrap"' . ($action_type == 'view|' . $file ? ' class="notice"' : '') . '>' . $this->fw('<a href="' . htmlspecialchars($this->action . '&TYPO3_INSTALL[database_type]=view|' . rawurlencode($file) . '#bottom') . '"><strong>VIEW' . $spec1 . $spec2 . '</strong></a>') . '</td>
                        </tr>';
                }
                        // TCA
                $out.='<tr>
                        <td></td>
-                       <td colspan="3"'.($action_type=="cmpTCA|"?' class="note"':'').'><strong>'.$this->fw('<a href="'.htmlspecialchars($this->action.'&TYPO3_INSTALL[database_type]=cmpTCA|#bottom').'">Compare with $TCA</a>').'</strong></td>
+                       <td colspan="3"' . ($action_type == "cmpTCA|" ? ' class="notice"' : '') . '><strong>' . $this->fw('<a href="' . htmlspecialchars($this->action . '&TYPO3_INSTALL[database_type]=cmpTCA|#bottom') . '">Compare with $TCA</a>') . '</strong></td>
                </tr>';
                $out.='<tr>
                        <td></td>
-                       <td colspan="3"'.($action_type=="adminUser|"?' class="note"':'').'><strong>'.$this->fw('<a href="'.htmlspecialchars($this->action.'&TYPO3_INSTALL[database_type]=adminUser|#bottom').'">Create "admin" user</a>').'</strong></td>
+                       <td colspan="3"' . ($action_type == "adminUser|" ? ' class="notice"' : '') . '><strong>' . $this->fw('<a href="' . htmlspecialchars($this->action . '&TYPO3_INSTALL[database_type]=adminUser|#bottom') . '">Create "admin" user</a>') . '</strong></td>
                </tr>';
                $out.='<tr>
                        <td></td>
-                       <td colspan="3"'.($action_type=="UC|"?' class="note"':'').'><strong>'.$this->fw('<a href="'.htmlspecialchars($this->action.'&TYPO3_INSTALL[database_type]=UC|#bottom').'">Reset user preferences</a>').'</strong></td>
+                       <td colspan="3"' . ($action_type == "UC|" ? ' class="notice"' : '') . '><strong>' . $this->fw('<a href="' . htmlspecialchars($this->action . '&TYPO3_INSTALL[database_type]=UC|#bottom') . '">Reset user preferences</a>') . '</strong></td>
                </tr>';
                $out.='<tr>
                        <td></td>
-                       <td colspan="3"'.($action_type=="cache|"?' class="note"':'').'><strong>'.$this->fw('<a href="'.htmlspecialchars($this->action.'&TYPO3_INSTALL[database_type]=cache|#bottom').'">Clear tables</a>').'</strong></td>
+                       <td colspan="3"' . ($action_type == "cache|" ? ' class="notice"' : '') . '><strong>' . $this->fw('<a href="' . htmlspecialchars($this->action . '&TYPO3_INSTALL[database_type]=cache|#bottom') . '">Clear tables</a>') . '</strong></td>
                </tr>';
 
                $theForm='<table border="0" cellpadding="2" cellspacing="2">'.$out.'</table>';
@@ -4731,30 +4731,29 @@ $out="
         * @return      void
         */
        function printSection($head, $short_string, $long_string, $type)        {
-               $icon='';
-               $cssClass =' class="note"';
                switch($type)   {
-                       case '3':
-                               $cssClass =' class="error"';
-                               $icon = 'gfx/icon_fatalerror.gif';
-                       break;
-                       case '2':
-                               $cssClass =' class="warning"';
-                               $icon = 'gfx/icon_warning.gif';
+                       case 3:
+                               $cssClass = ' class="typo3-message message-error"';
                        break;
-                       case '1':
-                               $icon = 'gfx/icon_note.gif';
+                       case 2:
+                               $cssClass = ' class="typo3-message message-warning"';
                        break;
-                       case '-1':
-                               $cssClass =' class="success"';
-                               $icon = 'gfx/icon_ok.gif';
+                       case 1:
+                               $cssClass =' class="typo3-message message-notice"';
+                               break;
+                       case 0:
+                               $cssClass = ' class="typo3-message message-information"';
+                               break;
+                       case -1:
+                               $cssClass = ' class="typo3-message message-ok"';
                        break;
                }
+
                if (!trim($short_string))       {
                        $this->sections[$head][]='';
                } else {
                        $this->sections[$head][]='
-                       <tr><td' . $cssClass . ' nowrap="nowrap">' . ($icon ? '<img src="' . $this->backPath . $icon . '" width="18" height="16" align="top" alt="">' : '') . '<strong>' . $this->fw($short_string) . '</strong></td></tr>' . (trim($long_string) ? '
+                       <tr><td' . $cssClass . ' nowrap="nowrap">' . '<strong>' . $this->fw($short_string) . '</strong></td></tr>' . (trim($long_string) ? '
                        <tr><td>' . $this->fw($long_string) . '<br /><br /></td></tr>' : '');
                }
        }
@@ -4815,7 +4814,7 @@ $out="
                while(list($header,$valArray)=each($this->sections))    {
                        $out.='
                        <tr><td>&nbsp;</td></tr>
-                       <tr><td><div align="center"><strong>'.$this->fw($header.':',2).'</strong></div></td></tr>
+                       <tr><td><div align="center"><strong>*'.$this->fw($header.':',2).'</strong></div></td></tr>
                        ';
                        $out.=implode($valArray,chr(10));
                }
@@ -4959,7 +4958,7 @@ $out="
        function note123()      {
                if ($this->mode=='123') {
                        $c='<table border="0" cellpadding="0" cellspacing="0" width="100%">
-                               <tr><td class="note" nowrap="nowrap"><img src="'.$this->backPath.'gfx/icon_note.gif" width="18" height="16" align="top" alt=""><strong>'.$this->fontTag1.'NOTICE: Install Tool is running in \'123\' mode. <a href="'.$this->scriptSelf.'">Click here to disable.</a></span></strong></td></tr>
+                               <tr><td class="notice" nowrap="nowrap"><img src="' . $this->backPath . 'gfx/icon_note.gif" width="18" height="16" align="top" alt=""><strong>' . $this->fontTag1 . 'NOTICE: Install Tool is running in \'123\' mode. <a href="' . $this->scriptSelf . '">Click here to disable.</a></span></strong></td></tr>
                        </table>';
                        return $c;
                }
index 671ef85..7f0c6be 100644 (file)
@@ -3,17 +3,17 @@ body, p, td, a, span, input, .bodytext {
        font-size: 1em;
        color: #000;
        text-align: left;
-       word-wrap: break-word; 
+       word-wrap: break-word;
 }
 
-h1, 
-h2 { 
-       font-family: verdana, arial, helvetica, sans-serif; 
+h1,
+h2 {
+       font-family: verdana, arial, helvetica, sans-serif;
        font-weight: bold;
 }
 
 h1 {
-       font-size: 14pt; 
+       font-size: 14pt;
 }
 
 h2 {
@@ -21,7 +21,7 @@ h2 {
 }
 
 a {
-       color: #600; 
+       color: #600;
        text-decoration: none;
 }
 
@@ -30,7 +30,7 @@ a:active {
 }
 
 a:hover {
-       color: #006; 
+       color: #006;
        text-decoration: underline;
 }
 
@@ -38,9 +38,9 @@ table {
        width: 100%;
 }
 
-th { 
-       font-size: 10pt; 
-       font-weight: bold; 
+th {
+       font-size: 10pt;
+       font-weight: bold;
 }
 
 textarea {
@@ -93,13 +93,6 @@ textarea {
        background-color: black;
 }
 
-.warning {
-       background-color: #ffc67f;
-}
-.success {
-       background-color: #af5;
-}
-
 .editFile {
        text-align: center;
        max-width: 900px;
@@ -111,21 +104,13 @@ textarea {
        width: 900px;
 }
 
-.error {
-       background-color: #f66;
-}
-
 .siteInfo {
        color: navy;
 }
 
-.note {
-       background-color: #d7dbe2;
-}
-
 .updateWizardBoxes {
-       border: 1px solid; 
-       margin: 10px; 
+       border: 1px solid;
+       margin: 10px;
        padding: 0px 10px 10px;
 }
 
@@ -134,6 +119,49 @@ textarea {
        background-color: #b3b7bd;
 }
 
+/* Styles for messages (matching flash messages) */
+
+.typo3-message {
+       padding: 6px;
+       padding-left: 26px;
+       margin-bottom: 4px;
+       background-repeat: no-repeat;
+       background-position: 5px 7px;
+       border: 1px solid;
+}
+.message-notice {
+       background-color: #fff;
+       border-color: #dee0e1;
+}
+.message-information {
+       background-image: url("../../../gfx/information.png");
+       background-color: #dfedf8;
+       border-color: #bccfe4;
+}
+.message-ok {
+       background-image: url("../../../gfx/ok.png");
+       background-color: #dfd;
+       border-color: #beb;
+}
+.message-warning {
+       background-image: url("../../../gfx/warning.png");
+       background-color: #ffd;
+       border-color: #f6ea68;
+}
+.message-error {
+       background-image: url("../../../gfx/error.png");
+       background-color: #fcc;
+       border-color: #ebb;
+}
+/* Colors to match above message styles, but without the icon in the background */
+.warning {
+       background-color: #ffd;
+}
+.notice {
+       background-color: #fff;
+}
+
+
 /* Menus */
 #imageMenu,
 #menu {
@@ -148,10 +176,10 @@ textarea {
        text-decoration: none;
 }
 
-#imageMenu td.generalTableBackground a:hover,  
+#imageMenu td.generalTableBackground a:hover,
 #menu td.generalTableBackground a:hover {
        display: block;
-       background-color: #fa8800;      
+       background-color: #fa8800;
        text-decoration: none;
 }
 
@@ -172,7 +200,7 @@ textarea {
 
 /* phpinfo */
 pre {
-       margin: 0px; 
+       margin: 0px;
        font-family: monospace;
 }
 .phpinfo table {
@@ -181,19 +209,19 @@ pre {
 .center {
        text-align: center;
 }
-.center table { 
-       margin-left: auto; 
-       margin-right: auto; 
+.center table {
+       margin-left: auto;
+       margin-right: auto;
        text-align: left;
 }
 
-.center th { 
-       text-align: center !important; 
+.center th {
+       text-align: center !important;
 }
-.phpinfo td, 
-.phpinfo th { 
-       border: 1px solid #000000; 
-       font-size: 75%; 
+.phpinfo td,
+.phpinfo th {
+       border: 1px solid #000000;
+       font-size: 75%;
        vertical-align: baseline;
 }
 
@@ -210,37 +238,37 @@ h2 {
 }
 
 .e {
-       background-color: #ccf; 
-       font-weight: bold; 
+       background-color: #ccf;
+       font-weight: bold;
        color: #000000;
 }
 
 .h {
-       background-color: #99c; 
-       font-weight: bold; 
+       background-color: #99c;
+       font-weight: bold;
        color: #000000;
 }
 
 .v {
-       background-color: #ccc; 
+       background-color: #ccc;
        color: #000000;
 }
 
 .vr {
-       background-color: #ccc; 
-       text-align: right; 
+       background-color: #ccc;
+       text-align: right;
        color: #000000;
 }
 
 .phpinfo img {
-       float: right; 
+       float: right;
        border: 0px;
 }
 
 .phpinfo hr {
-       width: 600px; 
-       background-color: #ccc; 
-       border: 0px; 
-       height: 1px; 
+       width: 600px;
+       background-color: #ccc;
+       border: 0px;
+       height: 1px;
        color: #000000;
 }