JonnyKukula Posted November 30, 2016 Report Share Posted November 30, 2016 Hello, I have been having an issue with my website. I got this skin and when I go to the pilots list, as shown in picture below, it is all messed up. I have the code for the pilots list here. Does anyone see anything wrong with the code? http://westjetvirtual.com http://westjetvirtual.com/index.php/pilots Thank you for your help, Jonny <div class="clearfix colelem" id="u1543"><!-- group --> <div class="clearfix grpelem" id="u1546"><!-- group --> <p>Pilots</p> </div> </div> <div class="grpelem" id="u1553"><!-- simple frame --></div> </div> <div class="clearfix colelem" id="pu579"><!-- group --> <div class="rgba-background mse_pre_init" id="u579"><!-- simple frame --></div> <div class="museBGSize mse_pre_init" id="u580"><!-- simple frame --></div> <div class="mse_pre_init" id="u581"><!-- simple frame --></div> </div> <div class="clearfix colelem" id="u1556"><!-- group --> <div class="grpelem" id="u1563"><?php if(!$allpilots) { echo 'There are no pilots!'; return; } ?> <table id="id_table"> <tbody><tr> <th>Pilot ID</th> <th>Name</th> <th>Location</th> <th>Rank</th> <th>Flights</th> <th>Hours</th> </tr> <?php foreach($allpilots as $pilot) { /* To include a custom field, use the following example: <td> <?php echo PilotData::GetFieldValue($pilot->pilotid, 'VATSIM ID'); ?> </td> For instance, if you added a field called "IVAO Callsign": echo PilotData::GetFieldValue($pilot->pilotid, 'IVAO Callsign'); */ // To skip a retired pilot, uncomment the next line: //if($pilot->retired == 1) { continue; } ?> <tr> <td><a href="<?php echo url('/profile/view/'.$pilot->pilotid);?>"> <?php echo PilotData::GetPilotCode($pilot->code, $pilot->pilotid)?></a></td> <td><?php echo $pilot->firstname.' '.$pilot->lastname?></td> <td><img src="<?php echo Countries::getCountryImage($pilot->location);?>" /> <?php echo Countries::getCountryName($pilot->location);?></td> <td><img src="<?php echo $pilot->rankimage?>" alt="<?php echo $pilot->rank;?>"/></td> <td><?php echo $pilot->totalflights?></td> <td><?php echo Util::AddTime($pilot->totalhours, $pilot->transferhours); ?></td> </tr> <?php } ?> </tbody></table></div> </div> Quote Link to comment Share on other sites More sharing options...
TAV1702 Posted December 2, 2016 Report Share Posted December 2, 2016 (edited) you need to check your closing div tags I see a couple to many right at the top of the code. <div class="clearfix colelem" id="u1543"><!-- group --> <div class="clearfix grpelem" id="u1546"><!-- group --> <p>Pilots</p> </div> </div> [color=#ff0000]** What is this tag closing?**[/color] <div class="grpelem" id="u1553"><!-- simple frame --></div> </div>[color=#ff0000]** Or this one? I think you need to delete the other one I have highlighted and let this one close your opening div tag.[/color] As for the rest, Ill go look at your site and view the source and see if I can spot anything. I am not the worlds greatest but I get lucky every now and again. Edited December 2, 2016 by TAV1702 Quote Link to comment Share on other sites More sharing options...
TAV1702 Posted December 2, 2016 Report Share Posted December 2, 2016 I just viewed the source for your pilots page, man I think you have a lot of div issues. In my personal way of doing it when I am not sure, I tag each div with a commented number. When it is time to close div tags, I view my numbers and tag my closing divs with numbers as well to make sure each one is closed properly. Then when all is said and done, you can remove comments if you wish. I tend to (not always because i am lazy) leave comments on each opening div or section so I know what it is example <!-- Div 1 About Us --> Add my code here <-- closing div for about us --> Quote Link to comment Share on other sites More sharing options...
Recommended Posts
Join the conversation
You can post now and register later. If you have an account, sign in now to post with your account.