Commit 0b7b2113 authored by Matt Glaman's avatar Matt Glaman

Issue #2118059: Missing H1 / Title on default taxonomy view

parent 45c1fa72
......@@ -92,10 +92,6 @@ function commerce_kickstart_block_preprocess_page(&$variables) {
// Remove the title from this page.
$variables['title'] = '';
}
if (arg(0) == 'taxonomy' && arg(1) == 'term' && is_numeric(arg(2))) {
// Remove collection_taxonomy_term view title.
$variables['title'] = '';
}
}
/**
......
<?php
/**
* Implements hook_preprocess_page().
*/
function commerce_kickstart_taxonomy_preprocess_page(&$variables) {
// Remove collection_taxonomy_term view title.
$router_item = menu_get_item(current_path());
$machine_name = variable_get('commerce_kickstart_demo_store', FALSE) ? 'collection' : 'product_category';
if ($router_item['path'] == $machine_name . '/%') {
$variables['title'] = '';
}
}
/**
* Implements hook_taxonomy_menu_path().
*/
......
......@@ -23,6 +23,10 @@ Feature: Blog
Then I should see the heading "Social Logins Made Simple"
When I click "Log in"
Scenario: Blog terms display page title (#2118059)
When I am on "/blog-category/cmt"
Then the "#page-title" element should contain "CMT"
@drush
Scenario: Comments should be open for authenticated users
When I am logged in as a user with the "authenticated user" role
......@@ -53,4 +57,4 @@ Feature: Blog
| links |
| delete |
| edit |
| reply |
\ No newline at end of file
| reply |
Feature: Catalog
As a site visitor
I can browse the catalog
To purchase products
Scenario: Catalog taxonomy vocabulary doesn't have duplicate titles (#2118059)
When I am an anonymous user
And I am on "/collection/geek-out"
Then the ".taxonomy-title" element should contain "To Geek Out"
And I should not see an "#page-title" element
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment