751698f

Needs to be green as the rest of the checkout is.

XS view brings problems with forms. col-xs-* > col-sm-* will cure it

Needs to be swapped

Fixes the display of these tiny icons. No need for them to be responsive.

Outputs body text (div) in the head, which fails html validation.

Would it be better as a header content module?

Get the following error, I tested it twice than I went back to the original installer and works

There was a problem importing the database. The following error had occured:

Column count doesn't match value count at row 1

Please verify the connection parameters and try again.

Introduce BS settings into Admin

Good evening. I was looking for the last hour, file by file and in admin panel how could i change collumnWidth's for boxes like category, manufactures and other, but i did'nt found. Please provide information and help how we could do that? Cheers! ;)

Base it on Addon 8864.

HTML

• Category and Manufacturers HTML Descriptions

SEO Title

• Categories
• Manufacturers
• Products

Meta Data (keywords and description)

• Categories
• Manufacturers
• Products
• Pages

Testimonials

• needs to be split from reviews (admin side)

Reviews

• ability for shopowner to add a review

Other Modules

• Opengraph for Products
• gPublisher (aka Direct Connect)

Search Page

• Plug into the Products Meta Keywords

• Pushed
• Tested

Line 38:

should be

Also, can I submit a possible replacement invoice.php. Something that is very close, but a little more something to look at?

Need updating to Summary Cards. Per https://dev.twitter.com/cards/types/product - "This Card was be DEPRECATED on July 3, 2015 and has been mapped to the Summary Card."

https://github.com/gburton/Responsive-osCommerce/blob/master/includes/modules/address_book_details.php#L163

We have an extra parameter "0"

'country', 
(isset($entry['entry_country_id']) ? $entry['entry_country_id'] : STORE_COUNTRY), 
0,
'required aria-required="true" aria-describedby="atCountry" id="inputCountry"'

How I believe it should be:

'country', 
(isset($entry['entry_country_id']) ? $entry['entry_country_id'] : 0), 
'required aria-required="true" aria-describedby="atCountry" id="inputCountry"'

Will look at older 234 and update issue appropriately.

Needs attention to change to div class="checkbox"

There was a problem importing the database. The following error had occured:

Column count doesn't match value count at row 1

Please verify the connection parameters and try again.

From http://forums.oscommerce.com/topic/396152-bootstrap-3-in-2334-responsive-from-the-get-go/?p=1725341

Change the HEADER_CART_CONTENTS define to:

$plural = ($cart->count_contents() > 1) ? 's' : '';
 define('HEADER_CART_CONTENTS', '<i class="glyphicon glyphicon-shopping-cart"></i> %s item' . $plural . ' <span class="caret"></span>');

This is in file includes/languages/english/modules/content/navigation/cm_navbar.php

Templatise to make it easier for site owners (and designers) to make this module look how they want it to look. /includes/modules/products_new.php

the search box is not showing up in my admin -> Modules -> boxes panel to activate it. Everything else works perfectly for me though.

Needs to be made smaller, as it encompasses too much data.

We are using 1.0.0

https://github.com/gburton/osCommerce-234-bootstrap/blob/master/ext/photoset-grid/jquery.photoset-grid.min.js

In a grid view of a category with 23 products, the list view is correct but the grid view has random gaps.

There is a bug that appears SOMETIMES because there is the async :

have a look at the screenshot:

NOTE: tested only locally

But if I refresh the page then its ok.

I repeat that happens sometimes.

in product_review page needs attention

Required asterisk is misplaced

eg the welcome message on the index page

As per the login modules, pretty the checkout success modules

Title of Product does not show when looking at the products review page.

Used in Admin Side.
Unused in Shop Side.

• Pushed
• Tested

This is not an issuu, but can be helpful to enable scrutinizer-ci alzo on this branch

includes/modules/header_tags/ht_product_colorbox.php : line 49

When viewing a product with only one image using Firefox or Chrome, this line produces a "TypeError: Cannot read property 'substring' of undefined" error in the console. This is due to the following portion of line 49.

var imgid = $(this).attr('id').substring(9);

Surrounding this with an if statement to check if variable exists removes the error.

The system kicked me to the home page after clicking on "Buy Now" button in the product listing page. Item was added properly. "Display Cart After Adding Product" was set to "false" in admin.

tel, email, search, number

If there is code that is obsolete, why do we leave it in the codebase?

https://github.com/gburton/Responsive-osCommerce/blob/master/includes/application_top.php#L38-L39

Why not remove this type of stuff?

Particularly /includes/languages/english.php

In here there are constants that are unused (hence should be removed entirely)
Also constants that are not used site-wide (hence should be moved to the relevant language sub-file)

Showing the entire catalogue looks strange although I believe it is displayed correctly?

Templatise to make it easier for site owners (and designers) to make this module look how they want it to look. /includes/modules/product_listing.php

On line 157 we can see tep_get_country_list() calling with 4 parameters:

echo tep_get_country_list('country', STORE_COUNTRY, 0, 'id="inputCountry" aria-describedby="atCountry"');

However, tep_get_country_list() declaration defines only 3 parameters:
function tep_get_country_list($name, $selected = '', $parameters = '')

@gburton I have discovered this problem while searching a similar problem concerning a third party module.

I think the solution is something like this.

What do you think?

Pretty the Login templates

admin\includes\languages\english.php

must be (!?) close

on last line ?>