[#8929] - Remove class="active" from non-menu item
- Closed
- 19 Jan 2016
- Medium
- Build: staging
- # 8929
- Diff
- uglyeoin:patch-1
- Success Human Test Results: 2 Successful 0 Failed. Details
User tests: Successful: Unsuccessful:
Removed the class="active" from line 17 which is text. In the standard Joomla! install it says "you are here". This should not be active, only the menu item for the page you are on should be active. You can see an example here: http://eoindemo.demojoomla.com/index.php/bread-crumbs-content
Issue posted here: #8593 (comment)
Protostar needs to be updated with this CSS
.breadcrumb > li {
color: #999;
}
This change may break backwards compatibility with other templates. I have fixed Protostar.
Protostar pull request here: #8930
| Status | New | ⇒ | Pending |
| Labels |
Added:
?
|
||
This PR has received new commits.
CC: @wojsmol
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/8929.
This PR has received new commits.
CC: @wojsmol
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/8929.
I have tested this item 
successfully on 540aa6e
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/8929.
I have tested this item successfully on 540aa6e
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/8929.
| Status | Pending | ⇒ | Ready to Commit |
RTC - thanks
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/8929.
| Labels |
Added:
?
|
||
wilsonge
- | Status | Ready to Commit | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2016-01-19 21:59:37 |
| Closed_By | ⇒ | wilsonge |
| Labels |
Removed:
?
|
||
| Milestone |
Added: |
||
I have tested this item
successfully on 21dba53
Together with PR #8930
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/8929.