Checkout_payment.php update modified by Philip Salerno

This is built upon the work of Kevin Shelton and Brushwood. Thanks to both of them for their previous work. I modified the the placement of the TITLE_PLEASE_SELECT text constant, the Select icon, and the placement of the TEXT_SELECT_PAYMENT_METHOD constant so that the TEXT_SELECT_PAYMENT_METHOD constant does not start at the middle of the box and is not on the first line of the box. This places the TEXT_SELECT_PAYMENT_METHOD constant closer to the left. See the image in the zip file for their placement.

FIND THIS SECTION (approximately line 248-253):

_tr_
_td__?php echo tep_draw_separator('pixel_trans.gif', '10', '1'); ?__/td_
_td class="main" width="50%" valign="top"__?php echo TEXT_SELECT_PAYMENT_METHOD; ?__/td_
_td class="main" width="50%" valign="top" align="right"__b__?php echo TITLE_PLEASE_SELECT; ?__/b__br__?php echo tep_image(DIR_WS_IMAGES . 'arrow_east_south.gif'); ?__/td_
_td__?php echo tep_draw_separator('pixel_trans.gif', '10', '1'); ?__/td_
_/tr_

AND REPLACE WITH THIS:

_tr_
_td__?php echo tep_draw_separator('pixel_trans.gif', '10', '1'); ?__/td_
_td class="main" width="7%" valign="top" align=left__b__?php echo TITLE_PLEASE_SELECT; ?__/b__br__?php echo tep_image(DIR_WS_IMAGES . 'arrow_east_south.gif'); ?__/td_
_td class="main" width="100%" align=left__?php echo TEXT_SELECT_PAYMENT_METHOD; ?__/td_
_td__?php echo tep_draw_separator('pixel_trans.gif', '10', '1'); ?__/td_
_/tr_


Checkout payment update by Kevin L. Shelton

We use the payment methods of credit card and check/money order at our store, and with the default osCommerce placement of the payment selection radio buttons at the far right of the box, well away from the payment title, it seems that at least once a month we were getting a call from a customer who wondered why we were waiting for a check when he had entered his credit card information online. Obviously the customer had checked the button for check/money order instead of the one for credit card because the buttons were far enough away from the payment method title to make it hard to tell which button went with which payment method.

This minor modification moves the radio buttons to select payment method so that they are immediately to the left of the payment method name. That way it will be much more obvious to the customer which method goes with which button.

To install simply replace the /catalog/checkout_payment.php file with the one in this package.

Updated Instructions, by Brushwood, October, 2005: Thanks to Kevin for this excellent fix. The full file included in this contrib is perfect if you have no other modifications and you have a narrow page layout. For those who have modfications or full width sites, try this search and replace:

FIND THIS SECTION:

if (sizeof($selection) _ 1) {
?_
_tr_
_td__?php echo tep_draw_separator('pixel_trans.gif', '10', '1'); ?__/td_
_td class="main" width="50%" valign="top"__?php echo TEXT_SELECT_PAYMENT_METHOD; ?__/td_
_td class="main" width="50%" valign="top" align="right"__b__?php echo TITLE_PLEASE_SELECT; ?__/b__br__?php echo tep_image(DIR_WS_IMAGES . 'arrow_east_south.gif'); ?__/td_
_td__?php echo tep_draw_separator('pixel_trans.gif', '10', '1'); ?__/td_
_/tr_
_?php
} else {
?_
_tr_
_td__?php echo tep_draw_separator('pixel_trans.gif', '10', '1'); ?__/td_
_td class="main" width="100%" colspan="2"__?php echo TEXT_ENTER_PAYMENT_INFORMATION; ?__/td_
_td__?php echo tep_draw_separator('pixel_trans.gif', '10', '1'); ?__/td_
_/tr_
_?php
}

$radio_buttons = 0;
for ($i=0, $n=sizeof($selection); $i_$n; $i++) {
?_
_tr_
_td__?php echo tep_draw_separator('pixel_trans.gif', '10', '1'); ?__/td_
_td colspan="2"__table border="0" width="100%" cellspacing="0" cellpadding="2"_
_?php
if ( ($selection[$i]['id'] == $payment) || ($n == 1) ) {
echo ' _tr id="defaultSelected" class="moduleRowSelected" onmouseover="rowOverEffect(this)" onmouseout="rowOutEffect(this)" onclick="selectRowEffect(this, ' . $radio_buttons . ')"_' . "n";
} else {
echo ' _tr class="moduleRow" onmouseover="rowOverEffect(this)" onmouseout="rowOutEffect(this)" onclick="selectRowEffect(this, ' . $radio_buttons . ')"_' . "n";
}
?_
_td width="10"__?php echo tep_draw_separator('pixel_trans.gif', '10', '1'); ?__/td_
_td class="main" colspan="3"__b__?php echo $selection[$i]['module']; ?__/b__/td_
_td class="main" align="right"_
_?php
if (sizeof($selection) _ 1) {
echo tep_draw_radio_field('payment', $selection[$i]['id']);
} else {
echo tep_draw_hidden_field('payment', $selection[$i]['id']);
}
?_
_/td_
_td width="10"__?php echo tep_draw_separator('pixel_trans.gif', '10', '1'); ?__/td_
_/tr_




AND REPLACE WITH THIS:


if (sizeof($selection) _ 1) {
?_
_tr_
_td__?php echo tep_draw_separator('pixel_trans.gif', '10', '1'); ?__/td_
_td class="main" width="50%" valign="top" align=left__b__?php echo TITLE_PLEASE_SELECT; ?__/b__br__?php echo tep_image(DIR_WS_IMAGES . 'arrow_east_south.gif'); ?__/td_
_td class="main" width="50%" valign="top"__?php echo TEXT_SELECT_PAYMENT_METHOD; ?__/td_
_td__?php echo tep_draw_separator('pixel_trans.gif', '10', '1'); ?__/td_
_/tr_
_?php
} else {
?_
_tr_
_td__?php echo tep_draw_separator('pixel_trans.gif', '10', '1'); ?__/td_
_td class="main" width="100%" colspan="2"__?php echo TEXT_ENTER_PAYMENT_INFORMATION; ?__/td_
_td__?php echo tep_draw_separator('pixel_trans.gif', '10', '1'); ?__/td_
_/tr_
_?php
}

$radio_buttons = 0;
for ($i=0, $n=sizeof($selection); $i_$n; $i++) {
?_
_tr_
_td__?php echo tep_draw_separator('pixel_trans.gif', '10', '1'); ?__/td_
_td colspan="2"__table border="0" width="100%" cellspacing="0" cellpadding="2"_
_?php
if ( ($selection[$i]['id'] == $payment) || ($n == 1) ) {
echo ' _tr id="defaultSelected" class="moduleRowSelected" onmouseover="rowOverEffect(this)" onmouseout="rowOutEffect(this)" onclick="selectRowEffect(this, ' . $radio_buttons . ')"_' . "n";
} else {
echo ' _tr class="moduleRow" onmouseover="rowOverEffect(this)" onmouseout="rowOutEffect(this)" onclick="selectRowEffect(this, ' . $radio_buttons . ')"_' . "n";
}
?_
_td width="10"__?php echo tep_draw_separator('pixel_trans.gif', '10', '1'); ?__/td_
_td class="main" align=left width=20__?php
if (sizeof($selection) _ 1) {
echo tep_draw_radio_field('payment', $selection[$i]['id']);
} else {
echo tep_draw_hidden_field('payment', $selection[$i]['id']);
}
?__/td_
_td class="main" align=left__b__?php echo $selection[$i]['module']; ?__/b__/td_
_td width="10"__?php echo tep_draw_separator('pixel_trans.gif', '10', '1'); ?__/td_
_/tr_

More...