MODUL: Admin NEW TOP MENU JS

[Tomcraft]

Integriert in r468.

[vr]

Hi,

was haltet ihr davon, die column_left.php übersichtlicher zu machen mit sowas wie

Code: PHP  [Auswählen]

if (!function_exists('show')) {
  function show($flag, $label, $page, $parameters = '') {
  global $admin_access;
  if(empty($flag))
    $cond = true;
  else
    $cond = ($admin_access[$flag] == '1');
  if (($_SESSION['customers_status']['customers_status_id'] == '0') && ($cond))
    echo '<li><a href="' . xtc_href_link($page, $parameters) . '" class="menuBoxContentLink1">' . $label . '</a></li>';
    }
  }
 

Aufruf dann zb so:

Code: PHP  [Auswählen]

...
show('categories', BOX_CATEGORIES, FILENAME_CATEGORIES);
show('new_attributes', BOX_ATTRIBUTES_MANAGER, FILENAME_NEW_ATTRIBUTES);
show('products_attributes', BOX_PRODUCTS_ATTRIBUTES, FILENAME_PRODUCTS_ATTRIBUTES);
...
 

statt

Code: PHP  [Auswählen]

...
if (($_SESSION['customers_status']['customers_status_id'] == '0') && ($admin_access['categories'] == '1')) echo '<li><a href="' . xtc_href_link(FILENAME_CATEGORIES, '', 'NONSSL') . '" class="menuBoxContentLink"> -' . BOX_CATEGORIES . '</a></li>';
  if (($_SESSION['customers_status']['customers_status_id'] == '0') && ($admin_access['new_attributes'] == '1')) echo '<li><a href="' . xtc_href_link(FILENAME_NEW_ATTRIBUTES, '', 'NONSSL') . '" class="menuBoxContentLink"> -'.BOX_ATTRIBUTES_MANAGER.'</a></li>';
  if (($_SESSION['customers_status']['customers_status_id'] == '0') && ($admin_access['products_attributes'] == '1')) echo '<li><a href="' . xtc_href_link(FILENAME_PRODUCTS_ATTRIBUTES, '', 'NONSSL') . '" class="menuBoxContentLink"> -' . BOX_PRODUCTS_ATTRIBUTES . '</a></li>';
...
 

Leider muss getestet werden, ob die Ausgabefunktion definiert ist, da column_left anscheinend mehrfach durchlaufen wird.

Bei Interesse kann ich die umgestellte Datei posten, die könnt ihr dann ja mergen.

Grüße, Volker

[Tomcraft]

Das wär super, wenn du uns die Datei hier mal als ZIP anhängen würdest.

Grüße

Torsten

[web28]

Hallo Volker,
die column_left.php lässt sich mit Sicherheit verbessern.

Aber:
Es gibt unzählige Module mit Einbauanweisungen für die column_left.php.
Wir versuchen so nah wie möglich am Original zu bleiben. Deshalb habe ich trotz des neuen Topmenüs an der column_left.php festgehalten.

Mit Deinen vorgeschlagenen Änderungen würden mit Sicherheit einige User beim Einbau von Erweiterungen überfordert sein.

Trotzdem Danke für Deinen Vorschlag.

Gruss Web28

[vr]

Hi,

hier die column_left. Für User dürfte das sogar leichter zu verstehen sein als vorher.

Grüße, Volker

[Tomcraft]

Kannst du die überarbeitete column_left.php mal inspizieren web28?

Ich werfe da auch nochmal ein Auge drüber mit "meld" (File-Merger unter Ubuntu).

Grüße

Torsten

[vr]

Hallo,

ich habe gerade das neue Admin TOP Menu eingebaut.
Beim Klick auf "Server Info" verschiebt es den Header etwas (er wird höher und die Buttons rutschen nach unten).
Der Content bei Server Info scheint noch etwas zu breit zu sein.

Steffen

Kann ich bestätigen, bei Menupunkt Server Info ist noch irgendwas seltsam, es verhält sich anders als alle anderen Menupunkte.

Die Beschriftung der Icons im Header steht links statt unter den Icons, dadurch wird der Header geringfügig höher. Die Menuzellen des Ausklappmenus haben einen weissen Hintergrund statt dem überall sonst verwendeten grauen Farbverlauf.

Grüße, Volker

[web28]

Vielen Dank für die Info, das sollte aber bereits gefixt sein:

http://www.modified-shop.org/forum/topic.php?id=2222&page=3#post-23178

Falls doch nicht, bitter Screenshot und Browserversion posten.

[koshiro]

Hallo,
kann ich nicht bestätigen, sowohl in beide Live-Shops, als auch im Testshop mit der Aktuellen Trunc Version r544 wird unter Firefox 3.5, Opera 10, und IE8 alles sauber dargestellt, wenn man auf Server Info, oder auf andere Menüpunkte geht.
Gruß
Micha

[vr]

Vielen Dank für die Info, das sollte aber bereits gefixt sein:

http://www.modified-shop.org/forum/topic.php?id=2222&page=3#post-23178

Falls doch nicht, bitter Screenshot und Browserversion posten.

Sorry für den falschen Alarm, ist gefixt, danke.

Grüße, Volker

[vr]

Habt Recht, ich hatte eine alte server_info im Einsatz. Danke und sorry für den Fehlalarm.

Grüße, Volker

[vr]

... und gleich die nächste Sau durchs Dorf:

Was haltet ihr davon, die Anzeige der Merkmale auf den Konfigurationsseiten 3-spaltig statt 2-spaltig zu machen. Es müssen Label, Editierfeld und Beschreibung untergebracht werden, aber bisher stehen Editierkomponente und Beschreibung in einer Zelle untereinander, was viel Platz braucht und nicht so gut lesbar ist.

Ich hab das mal auf 3-spaltig umgestellt, in admin/configuration.php müsste der letzte echo-Befehl durch das hier ersetzt werden:

Code: PHP  [Auswählen]

echo '
  <tr>
    <td width="300" valign="top" class="dataTableContent"><b>'.constant(strtoupper($configuration['configuration_key'].'_TITLE')).'</b></td>
    <td valign="top" style="background-color:#FCF2CF; border: 1px solid; border-color: #CCCCCC;" class="dataTableContent">'.$value_field.'</td>
    <td valign="top" class="dataTableContent">'.constant(strtoupper( $configuration['configuration_key'].'_DESC')).'</td>
  </tr>
  ';
 

Grüße, Volker