[RESOLVED] formatting dynamic values

BEFOR

This calendar script prints a dynamic calendar, which defaults to the present month and year, if none is selected on the page.

How can I format:
-the BGCOLOR of todays cell (red?) to show users todays date.
-the font size, face and color of the numbers representing days // when I try the page blanks out.

<?php

//Render the Month and Year values current if not available.
if ((!$Month)&&(!$Year)) {
$Month = date("m");
$Year = date("Y");
}
// Calculate the Month being viewed.
$Timestamp = mktime (0,0,0, $Month, 1, $Year);
$MonthName = date("F", $Timestamp);
// Create a table that presents the month
print("<TABLE BORDER=1 BORDER COLOR=#E8E8E8 CELLPADDING=0 CELLSPACING=0 ALIGN=CENTER>");
print("<TR BGCOLOR=#6F0392><TD COLSPAN=9 ALIGN=CENTER><FONT COLOR=#FFFFFF><B>
$MonthName $Year</B></FONT></TD></TR>");
print ("<TR BGCOLOR=#FFFFFF>
<TD ALIGN=CENTER WIDTH=33><B>
<FONT COLOR=#6F0392>S</FONT></B></TD>
<TD ALIGN=CENTER WIDTH=33><B>
<FONT COLOR=#6F0392>M</FONT></B></TD>
<TD ALIGN=CENTER WIDTH=33><B>
<FONT COLOR=#6F0392>T</FONT></B></TD>
<TD ALIGN=CENTER WIDTH=33><B>
<FONT COLOR=#6F0392>W</FONT></B></TD>
<TD ALIGN=CENTER WIDTH=33><B>
<FONT COLOR=#6F0392>T</FONT></B></TD>
<TD ALIGN=CENTER WIDTH=33><B>
<FONT COLOR=#6F0392>F</FONT></B></TD>
<TD ALIGN=CENTER WIDTH=33><B>
<FONT COLOR=#6F0392>S</FONT></B></TD>
</TR>\n");

$MonthBegin = date("w", $Timestamp);
if ($MonthBegin == 0){
     $MonthBegin=7;
}

$EndDay= date("d", mktime(0,0,0, $Month+1, 0, $Year));
$BeginDate = -$MonthBegin;
for ($k = 1; $k <=6; $k++) {//Print 6 total rows.
print("<TR BGCOLOR=#FFFFFF>");
for ( $i=1; $i <=7; $i++){//With 7 cols
	$BeginDate++;
          if(($BeginDate <=0)||
            ($BeginDate > $EndDay)){
             print ("<TD BGCOLOR=#E8E8E8>
              &nbsp </TD>");   

       }elseif(($BeginDate >=1) &&
        ($BeginDate <=$EndDay)) {
            print ("<TD ALIGN=CENTER>
           $BeginDate </TD>");
	}
     }
      print("</TR>\n");
}
print ("</TABLE>\n");
//Create the form.
?>

maxpup979

get away from font tags--they are depricated. Use CSS instead. It really isnt hard, and it does the things you want quite easily./

BEFOR

where would you apply the class="name" for ea. of the:

$MonthName $Year

since this is different than the

ea of the seven <FONT COLOR=#6F0392>S</FONT> weekdays

which is different from ea of the day numbers being printed.

maxpup979

if need be, <span class="yadda"> is your friend

BEFOR

maxpup,

would you make three different spans, and put them where?

maxpup979

I would probably do a class for the header bar, one for the current day, and one for the other days. But thats just me. but thats one of the beautiful things about CSS...you can make it look pretty much exactly how you want. and change it easily as well...

BEFOR

Here is css, with a little help. This is better but problem is that #calendar td sets not only first row which is the days of the week - but all numbers of the month are the same.

The thd2 colors the background, but still trying to color and size the month #'s
I'm finding it will not allow me to do this, and it forces the same color as #calendar td Is there a way around this?

That, and how can I highlight background of the current day?
The CLASS for the current day,

#calendar {
border: 1px solid #E2E2E2;
margin: 0 auto;
}
#calendar th {
background: #6F0392;
color: #fff;
}
#calendar thd {
background: #FFFF00;
color: #fff;
}
#calendar td {
border: 1px solid #E2E2E2;
width: 30px;
color: #6F0392;
font-weight: bold;
text-align: center;
}

<?php

//Render the Month and Year values current if not available.
if ((!$Month)&&(!$Year)) {
$Month = date("m");
$Year = date("Y");
}
// Calculate the Month being viewed.
$Timestamp = mktime (0,0,0, $Month, 1, $Year);
$MonthName = date("F", $Timestamp);
// Create a table that presents the right month
print("<table id=\"calendar\">");
print("<tr><th colspan=\"9\">$MonthName $Year</th></tr>");
print ("<tr>
<td>S</td>
<td>M</td>
<td>T</td>
<td>W</td>
<td>T</td>
<td>F</td>
<td>S</td>
</tr>\n");

$MonthBegin = date("w", $Timestamp);
if ($MonthBegin == 0){
$MonthBegin=7;
}

$EndDay= date("d", mktime(0,0,0, $Month+1, 0, $Year));
$BeginDate = -$MonthBegin;
for ($k = 1; $k <=6; $k++) {//Print 6 total rows.
print("<tr class=\"thd\" </tr>");
for ( $i=1; $i <=7; $i++){//With 7 cols
$BeginDate++;
if(($BeginDate <=0)||
($BeginDate > $EndDay)){
print ("<TD BGCOLOR=#E8E8E8>
&nbsp </TD>");

}elseif(($BeginDate >=1) &&
($BeginDate <=$EndDay)) {
print ("<TD ALIGN=CENTER>
$BeginDate </TD>");
}
}
print("</TR>\n");
}
print ("</TABLE>\n");
//Create the form.
?>

BEFOR

Okay, almost have it but the BG color I'm applying is kicking today's date out.

This higher on the page -- incorrectly - it needs to be defined right above the rest of this code section:

$today = date(d);
...............................................................................................

Need to integrate this:

if ($BeginDate==$today) echo("<TD BGCOLOR=#CCCCCC></TD>")

with this, into a single statement:

{
print ("<TD ALIGN=CENTER>
$BeginDate </TD>");

....................................

In this portion.

}elseif(($BeginDate >=1) &&
($BeginDate <=$EndDay)) {
print ("<TD ALIGN=CENTER>
$BeginDate </TD>"); if ($BeginDate==$today) echo("<TD BGCOLOR=#CCCCCC></TD>");

}
}
print("</TR>\n");
}
print ("</TABLE>\n");
//Create the form.
?>

Roger_Ramjet

Basic error in your CSS syntax

#calendar thd {

You are trying to mix id and class selectors here and that will not work.

What you need is just to specify the class selector properly

.thd {

Next time try running your CSS through a validator like this one and it will highlight error like that.

BEFOR

thanks, nice validator too.

How can I integrate the green up there with the blue?

BEFOR

Like this:

}elseif(($BeginDate >=1) &&
($BeginDate <=$EndDay)&&($BeginDate!=$today)) {
print ("<TD ALIGN=CENTER>
$BeginDate </TD>");
}elseif(($BeginDate >=1) &&
($BeginDate <=$EndDay)&&($BeginDate==$today)) print("<TD BGCOLOR=#FFC1C1> $BeginDate</TD>")
;

}
}
print("</TR>\n");

print ("</TABLE>\n");
//Create the form.

?>

with two elseif's to account for both scenarios's:

-one if $BeginDate !=$today and then one if they are equal
(which highlights background)

Roger_Ramjet

Use more CSS. Create a class for each table cell format that you want to use and put everything in there: font-size -style -weight, color and background (PS bgcolor is html NOT CSS, and background-color is not as well supported across browsers so just use background in CSS), margins, borders, text-align, etc. The only thing I have found that cannot go into CSS is the table cell-spacing, everything else is better done with CSS.


}
elseif ($BeginDate==$today) {
     echo '<td id="today">'. $begindate .'</td>';
}
elseif (($BeginDate >=1) && ($BeginDate <=$EndDay)) {
    echo '<td class="whatever">'. $begindate .'</td>';
}

I've used an id selector since there can only be 1 cell with today's date.
I've also streamlined your if structure since, again, there will only be 1 case matching $today and so it need not be nested.

Personally, I would be doing this with a case select (switch in php) unless it were impossible - far cleaner and more efficient than complex nested if structures.

BEFOR

This thread is not resolved. - I cannot reverse the "resolved status"

The above code, to highlight the background, checks for: $today = date(d);
the number of which appears within the calendar.

Problem is, this is a dynamic calendar -- users can select many months over a number years-- and the 13th is being highlit on all of them now. Can't believe I didn't think of this.

What value can I check on perhaps that is already given here to ensure only the current month day will be highlit?

How can I parse that timestamp day value and check on it
ex: '13' for today

Roger_Ramjet

Drop the print() from your coding style: ugly, cumbersome and has no added benefits.

Use echo and the single-quoted syntax so you don't have to escape html double-quotes. It means you have to concatenate vars which makes your code more legible and easier to maintain in the long run: and definitely eliminates a lot of typo-syntax errors when first coding/testing it.

Cross-posted each other, sorry.

Roger_Ramjet

Basic cascade of conditions

If == this year then check month Else do the other
If == this month then check day Else do the other
If == today then change color Else do the other

Do not trap yourself into checking every single date against all conditions:

If == this year && == this month && == today

VERY WRONG way to code, totally inefficient

Just cascade down to the specific and bale out at the first opportunity

There are so many ways that you could have set this up that I can not be more specific.

BEFOR

While I know you are right, I've only done it the second way.

If you could please, I don't follow what you say.

"If this...then check" -- with what? How does this differ from AND?

per the page:

$today = date(d);
$Month = date("m");
$Year = date("Y");

BEFOR

Create a class for each table cell -- I got it.

Thanks for the help above.

Roger_Ramjet

pseudo code

If it is not this year then you do not need to check anything else as it cannot be this month or today's date - it is last year. If you check year AND month every time then you are doing a lot of unneccessary processing.