Beiträge von basti1012

Valide ja.

Hier schneidest du auch ganz gut ab.

https://developers.google.com/…flohheim.de%2F&tab=mobile

OK, du hast sehr wenig bis gar kein Schnickschnack, da sollten da gute Werte rauskommen.

Responsiv sehe ich gerade auch nichts Negatives

und zum Design kann ich nicht viel sagen

weil ich den Geschmack von keinen treffe.

Aber sonst finde ich das ganz gut

Ajax ist eigentlich nicht schwer.

Man müsste da ein paar Sachen umbauen damit alles ohne Pfusch über Ajax läuft.

Wenn ich nachher etwas Langeweile finde, schaue ich da mal rein, ob es nee einfache schnelle Lösung gibt.

Gibt es eine Anleitung zu dem Spiel?

Wie heißt das Spiel genau?

Ich muss ja wissen, wann es was gibt und so weiter.

Wenn du ein Link zu einer Anleitung hast , oder ein Link zum komplett Download des Spiels, dann schreib mir das mal.

Sorry.

Wenn ich mir deine Seite anschauen muss ich sagen, dass du es nicht verstanden hast , du machst es nur noch schlimmer.

Du versucht dir da irgendwas zusammen zu kopieren, was etwas funktioniert, aber doch alles noch komplizierter und schwieriger macht.

Ich persönlich hätte alles Scripte neu gemacht und auf Ajax umgestellt.

Dieses reingepfusche, was du da versuchst bring nicht viel.

Mach mal ein SQL Dump fertig , wenn ich mal Langeweile habe versuche ich mal was.

Zitat von ButterKnecht

Wie sollte es den aussehen?

Wie es am besten ist kann ich dir auch nicht genau sagen.

Zb so, erst prüfen, ob es $_GET['order'] gibt

und dann den ternary operator nutzen und den Inhalt lesen/ausgeben

if(isset($_GET['order'])){
     $ordertype=$_GET['order']=='desc'? 'desc' : 'asc';
}else{
     $ordertype='asc';//default
}

Bei den $_GET['sort']

würde es sogar mit den coalescing operaor funktionieren

$field =$_GET['sort'] ?? '';

Der prüft, ob es die Variable gibt, ,wenn ja, dann gibt er das aus was da drinnen steht ,

Gibt es die Variable nicht ,wird halt das ausgegeben was in '' steht.

Ob das der beste Weg ist, kann dir sicherlich einer von den PHP Profis sagen

Schön das es jetzt soweit läufz.

Habe noch 2 Fragen,.

Hat das irgendein Sinn

$username = htmlspecialchars_decode($username);
$score = htmlspecialchars_decode($score);

Sehe nicht das es irgendwo benutzt wird.

und wenn du im Link keine Get Variablen hast , sollte hier beim errorreprting Fehler auftreten

$field = $_GET['sort'];
            if($field == ''){
               $field = 'score'; 
            }
und
$ordertype = ($_GET['order'] == 'desc')? 'asc' : 'desc';

Wenn kein GET da ist , sollte im error reporting eigentlich was stehen

Zitat von ButterKnecht

Na klar habe ich was geändert. Ich habe bereits einen Teil rausgeschmissen. Wenn ich deinen Code benutze, funktionieren meine Tabellen Formatierungen nicht mehr richtig.

Das weist darauf hin das deine CSS zusammen gefuscht ist.

Meine Formatierung ist richtig, deine Falsch.

Das kannst du dir aber denken weil ein </a> ohne offnes <a> gibt es nicht

genauso wie <tr></th>

Trotz kaputten HTML nimm hier mal das overflow:hidden raus

.carousel-inner {
    position: relative;
    width: 100%;
   /* overflow: hidden;*/
}

Dann sollte alles wieder da sein.

Man sieht zwar das overflow:hidden fehlt , doch solange das HTML kaputt ist hat das weitere helfen nicht viel Sinn.

Zitat von ButterKnecht

Nein funktioniert leider auch nicht.

Du hast ja auch nichts geändert.

Dein HTML ist immer noch so Schrott.

Wenn man die Vorschläge nicht annehmen will, brauch man auch nicht Hilfe zu rufen.

Das was m.scatello sagte, ist mir gestern gar nicht aufgefallen.

Man könnte sogar den Code weit um 50 % reduzieren.

Kannst du das mal online stellen?

Was zeigt der Firefox ?

Wird das HTML richtig angezeigt ?

Ich glaube das der Firefox bei den Mist durcheinander kommt

echo '<table class="tbl-header" >';
echo '<table class="tbl-content" >';
                echo '<tr>';
                    
                echo '</a></th>'; 
                echo '<th>PLAYER  '; 
                
                echo '</a></th>'; 
                
                echo '<th>Time</th>';  
               
                echo '</a></th>';  
                echo '</tr>';

Da muss der HTML Code schon vollkommen falsch ankommen.

Teste die Seite mal bei https://validator.w3.org/ , der sagt dir dann sehr viel über unclosed Elemente usw

ändere das mal so

if($result->num_rows > 0){
    echo '<table class="tbl-header" ><tbody>
         <tr><th>Player</th><th>Score</th></tr>';
         while($row = $result->fetch_object()){
             echo '<tr>  
                       <td>'.htmlspecialchars($row->username).'</td>
                       <td>'.htmlspecialchars($row->score).'</td>
                   </tr>';
         }
    echo '</tbody></table>';
}

Zitat von Dekiblago

Hab den slid hinbekommen mit height Auto, hab zwar ne Weile gebraucht musste in der CSS bissle was umschreiben. Ist ja garnicht gedacht gewesen das die Transmission mit Height : Auto funktioniert. Jetzt nur noch ein Befehl finden wen das eine gedrückt wird sich das andere schließt

Hättest du dir die Beispiele mal richtig angeschaut ,dann hättest du gesehen das es da auch so wahr.

Die andere Box schließen könnte man mit jS machen.

Dir das aber zu zeigen hat ja eh keinen Sinn, du nimmst ja nichts an was man dir sagt und der Validator schmeißt immer noch alle Fehler raus.

Deine CSS hat immer noch die Anti responsive Werte,

Gerade unclosed Elemente zerschießen dir das Layout, was man mit Pfuschen überdeckt wird.

Mach es gleich richtig und baue es mit Flexbox auf.

Eine checkbox hat ja auch kein Text .

Wenn du aber für an und aus Text haben willst dann ändere es doch einfach in der Ausgabe.

$inarbeit = $row['inarbeit'];
if($inarbeit==1){
     $inarbeit='Primary';
}else{
     $inarbeit='Was anderes';
}

Auserdem solltest du beim erstellen der Tabelle auch gleich gescheckte boxen angelickt lassen.

$inarbeit = $row['inarbeit'];
if($inarbeit==1){
     $inarbeit='Primary';
     $checked="checked='true'";
}else{
     $inarbeit='Was anderes';
     $checked="";
}

echo "<tr>   
 <td> $cd </td>
 <td>$Kundennr </td>
 <td> $Pizza </td>
 <td>$Belag </td>
 <td>$inarbeit</td>
 <td><input type='checkbox' name='check[]' value='$id' $checked></td>
 </tr>";

ungefähr so

EDIT:

Zeile 65 musst du in deinen Code die Checkbox schließen. da fehlt ein >

Ohne Checkbox-Hack und mit den Details Element könnte man es so machen

** Link entfern, weil nicht mehr erreichbar **

Bei seiner jetzigen Navigation und seinen Beiträgen hier denke ich das sowas

<details>
  <summary>Slideshow</summary>
  <p>More info about the details.</p>
</details>

auch sinnvoller ist .

Das sich eine Box in der Navigation öffnet beim Klicken( ob Checkbox oder anders) , finde ich auch etwas fehl am Platz.

Eine Navigation ist ja nicht dafür gedacht , das man sie mit Inhalten voll klatscht

Hast du das auf deiner Seite hochgeladen?

Bei mir sieht das ganz anders aus.

Zitat von Dekiblago

ihr seht die blaue div Box aber die ist nur halb also die schiebt sich hinter Javascript aber nicht das Java Script weg .. was kann ich da machen ?

Ich sehe schon wieder solche Werte

position: relative;
    margin-top: -110px;
float: right;
    width: 40%;
    margin-right: 250px;
    position: relative;
  height: 800px;
    float: left;
clear:both

Das hatte schon ein Grund warum ich die in meinem Beispiel nicht benutzt habe.

Auch hast du wieder alle Fehler beibehalten, bzw. mehr eingebaut

https://validator.w3.org/nu/?s…evic.000webhostapp.com%2F

Teste mal mein Code von Codepen da.

Du hättest von da an weiter bauen sollen, und nicht deinen alten Code noch mehr Fehler einzubauen.

Mit deinem jetzigen Code wird das mit responsivem Design immer schwerer.

Wenn du auf Slideshow klicken möchtest und da drunter soll sich eine Box öffnen, wären solche Elemente

<details>
  <summary>Slideshow</summary>
  <p>More info about the details.</p>
</details>

sinvoller.

Versuch mal den Namen zu ändern.

Sonderzeichen um Umlaute im Link sind nicht so der Hit.

Wenn die Dateien im gleichen Ordne sind , dann weiß ich jetzt auch nicht weiter.

EDIT:

Wenn ich das ö in oe tausche geht es bei mir

Meinst du das so?

Du kannst die boxen , wo neue1 -3 stehen ja selber austauschen

** Link entfern, weil nicht mehr erreichbar **

Zitat von Dekiblago

Die Dropbox habe ich mir aus anderen quel Dateien zusammengeschustert , die "href" links müssen noch abgeändert werden, will einfach nur HTML5/CSS3 und dan nochmal Slideshow zum Slide bringen und nicht auf ein anderen Link verweisen

Verstehe nicht, was du da zum sliden bringen willst?

HTML5/CSS3 klickst du an und es sliden neue Links rein.

Wenn du Slideshow anklickst kann nichts mehr sliden , weil keine Links mehr da sind.

Ich verstehe deine Erklärung nicht.

So wie du es vorhast, geht das auch nicht.

Optimaler wäre das, wenn du das gleich da erstellst, wo hin soll

Du machst das so bei deinem Affenformular System

                      <div id=hell>             
                        <div id='div4' class='zielzone' >
                          <div class='divTableCell1'><div id='blick_feld1'></div></div>
                          <div class='divTableCell2'><div id='blick_feld2'></div></div>
                          <div class='divTableCell3'><div id='blick_feld3'></div></div>
                          <div class='divTableCell4'><div id='blick_feld4'></div></div>
                          <div class='divTableCell5'><div id='blick_feld5'></div></div>
                          <div class='divTableCell4'><div id='blick_feld6'></div></div>
                          <div class='divTableCell3'><div id='blick_feld7'></div></div>
                          <div class='divTableCell2'><div id='blick_feld8'></div></div>
                          <div class='divTableCell1'><div id='blick_feld9'></div></div>
                        </div>
                      </div>
<script>/* irgendwo anders im code */
$("#blick_feld1").html  ("<img src='grafik/images/wand/1_1.png'  draggable='false'>");
$("#blick_feld2").html  ("<img src='grafik/images/wand/2_1.png'  draggable='false'>");
$("#blick_feld3").html  ("<img src='grafik/images/wand/3_1.png'  draggable='false'>");
$("#blick_feld4").html  ("<img src='grafik/images/wand/4_1.png'  draggable='false'>");
$("#blick_feld5").html  ("<img src='grafik/images/wand/5_1.png'  draggable='false'>");
$("#blick_feld6").html  ("<img src='grafik/images/wand/6_1.png'  draggable='false'>");
$("#blick_feld7").html  ("<img src='grafik/images/wand/7_1.png'  draggable='false'>");
$("#blick_feld8").html  ("<img src='grafik/images/wand/8_1.png'  draggable='false'>");
$("#blick_feld9").html  ("<img src='grafik/images/wand/9_1.png'  draggable='false'>");
</script>

Ich würde es so versuchen.

<div id='hell'>             
   <div id='div4' class='zielzone' >
<?php
for($g=1;$g<=9;$g++){
echo "<div class='divTableCell$g'>
        <div id='blick_feld$g'>
           <img src='".$gr."1_".$x1.".png' $hell draggable='false'>
        </div>
     </div>";
}
?>
</div>
</div>

Copy-and-paste hat keinen Sinn, weil es in dein System so auch nicht läuft,

Besser wäre Ajax.

Wo hast du den Code her?

Kann mir einer erklären, was das sein soll?

if("okay"<>"okay") {

WOW.

Ein disaster.

Man könnte den ganzen Code sicherlich auf die Hälfte reduzieren.

Dann wird das ganze HTML über Echo ausgegeben.

Eigentlich sollte man alles umbauen.

Es sind ja nicht nur die Pfeiltasten, auch dein ganzer Zauber Kram sollte man über Ajax machen.

Da wir jetzt ja erstmal nur über die Bilder reden, habe ich auf der schnelle keine Ahnung wie ich auf der schnelle an die Bild Variablen kommen soll mit Ajax.

Da die Bilddaten mitten in der functions.php liegen bin ich gerade etwas überfordert.

Ich müsste dafür einiges umbauen damit alles über Ajax läuft.

Vielleicht hat Sempervivum da nee Idee wie es gehen könnte.

Ich kann dir aber jetzt zeigen wie es mit Ajax aussehen würde

https://basti1012.bplaced.net/katakomben/index.php

Brauchst kein Copy Paste machen , weil funktioniert bei dir so sowieso nicht.

Aber du hast dann mal nee Vorstellung wie schnell es dann läuft.

Nicht wunden das die Bilder nicht passen, ich gebe Random Bilder zurück.

Ja, wie man sieht, führst du immer erst das SELECT aus,

Wenn du jetzt Update klickst , rufst du die Seite neu auf.

Es wird erst das SELECT gemacht, dann dein Update.

Ohne jetzt großartig was dran zu ändern , sollte das eigentlich reichen die Reihenfolge zu tauschen

<?php
try{
    $verbindung = new PDO($server, $username,$passwort);
}catch(Exception $fehler){
    print $fehler->getMessage();
}
if(isset($_POST["inarbeit"])){
    $inarbeit = $_POST["inarbeit"];
    $id=$_POST['id'];
    $stmt = $verbindung->prepare("UPDATE bspdb.pizzabestellung SET inarbeit = ? WHERE id = ?");
    $stmt->execute([$inarbeit,$id]);
}
     //Bestelldaten aus der Datenbank abfragen
$sqlBefehlPizzabestellung = "SELECT * from bspdb.pizzabestellung ORDER BY created_at DESC";
$abfragePizzabestellung = $verbindung->prepare($sqlBefehlPizzabestellung);
$abfragePizzabestellung->execute();
$ergebnisPizzabestellung = $abfragePizzabestellung->fetchAll(); 
$verbindung = null;

?>

In dein fall, würde sich Ajax leicht umsetzen lassen.

Bei einer Änderung der Checkbox, brauch ja nur ein Request mit der ID und den Checkbox Status gesendet werden.

Da du für jede Zeile ein Button hast, brauch man auch nur den Checkbox Status der einen ID zurückgeben.

Du könntest auch nur ein Button für alle Zeilen erstellen.

Dann werden erst alle Checkboxen ausgelesen und dann alles in Request versendet.

Wenn du 30 Zeilen hast, musst du jetzt 30-mal die Seite neu laden, wegen den Affenformular.

Mit Ajax geht das alles mit ein Request