Reviews Page Not Working Properly

yodapt wrote:

Code: Select all

$('#review .pagination a').live('click', function() {

should be

Code: Select all

$('#review .pagination-bottom a').live('click', function() {

catalog/controller/product/product.php

	$results = $this->model_catalog_review->getReviewsByProductId($this->request->get['product_id'], ($page - 1) * 30, 30);
      		
		foreach ($results as $result) {
        	$this->data['reviews'][] = array(
        		'author'     => $result['author'],
				'rating'     => $result['rating'],
				'text'       => strip_tags($result['text']),
        		'stars'      => sprintf($this->language->get('text_stars'), $result['rating']),
        		'date_added' => date($this->language->get('date_format_short'), strtotime($result['date_added']))
        	);
      	}			
		
		$review_total = $this->model_catalog_review->getTotalReviewsByProductId($this->request->get['product_id']);
			
		$pagination = new Pagination();
		$pagination->total = $review_total;
		$pagination->page = $page;
		$pagination->limit = 30; 

Then, open up /catalog/view/theme/elegium/stylesheet/stylesheet.css and add this:...

gedielson wrote:The yodapt sugestion should solve the issue. BUT, let's try a different approach:
Open up catalog/view/theme/elegium/template/product/review.tpl, find

Code: Select all

<div class="pagination-bottom"><?php echo $pagination; ?></div>

and replace by

Code: Select all

<div class="pagination"><?php echo $pagination; ?></div>

In other words, just replace "pagination-bottom" class by "pagination" only. Then, open up /catalog/view/theme/elegium/stylesheet/stylesheet.css and add this

Code: Select all

.pagination {
	display: inline-block;
	width: 100%;
	padding-bottom: 30px;
}
.pagination-bottom .links {
	float: left;
}
.pagination .links a {
	display: inline-block;
	background: #eee;
	padding: 3px 8px;
	text-decoration: none;
	color: #888;
	-webkit-border-radius: 2px;
	-moz-border-radius: 2px;
	-khtml-border-radius: 2px;
	border-radius: 2px;
}
.pagination .links a:hover {
	color: #666;
	background: #eee;
}
.pagination .links b {
	display: inline-block;
	background: #d6d6d6;
	padding: 3px 8px;
	font-weight: normal;
	text-decoration: none;
	color: #666;
	-webkit-border-radius: 2px;
	-moz-border-radius: 2px;
	-khtml-border-radius: 2px;
	border-radius: 2px;
}
.pagination .results {
	float: right;
	padding-top: 3px;
}

This should do the trick.

gedielson wrote:I got it. You are modifying the wrong files, that are in root folder, instead of the files on dev subdomain. You changed this file:
darkmoontest.com/catalog/view/theme/elegium/stylesheet/stylesheet.css
When you have to change the one in dev subdomain to take effect:
dev.darkmoontest.com/catalog/view/theme/elegium/stylesheet/stylesheet.css

This is why your changes aren't taking effect.

Same with review.tpl

Do the changes in correct files and the issue will be solved.

Cheers.

gedielson wrote:Yeah, we focus so much in probably complex solutions that sometimes the simple ones go unperceived.

gedielson wrote:You need to make this change to all files that you want to display results in this way. But, in my opinion, it wil be weird when, e.g., there are 32 products total and in last page there are 2 products, it will display "Showing 2 of 32 Products [0 products more]". As I said, just my opinion.

gedielson wrote:There are two files: ../catalog/view/theme/*/stylesheet/stylesheet.css, line 372; and ../catalog/view/javascript/bootstrap/css/bootstrap.min.css, this one compressed. Use your editor's search feature (normally Ctrl + F) and search for "pagination" class.

The pagination part in stylesheet.css is practically nothing (...) and whatever changes I do here does not reflect on frontend.

Why are there 2 bootstrap files min and non-min version.

I was under the impression that bootstrap file is needed only for responsive or mobile compliant websites so why is pagination part put in bootstrap whereas in oc1564 it is in stylesheet, please kindly explain this part