Greetings,
the AT Commerce theme is one of the best (thanks to the makers). The theming UI giving pretty posibilities (for example shadow control - only for block headings etc.). After some changes in copy of that (as subtheme but without changes of name, description etc.) it is looking exactly according my idea(s).
But - there is (maybe) an issue (if it is not my mystake)...: Breadcrumbs "You are here" disfunction(s).
When breadcrumb is ON, than the "You are here" message is also on - even if not switched so in theme UI (?!). Even worse, the message is positioned at first place in the row of breadcrumbs (it is not moving to the next breadcrumb). Please, see: http://dmdu.netprostor.cz/en/aggregator/sources/1 (" You are hereHome » Feed aggregator » Sources ").
If anyone have any advice o(solution), please let me know...
With best regards,
M. Radan Chrobok Ph.D.
Founder of prestigiois university DMDu
http://DMDU.netprostor.cz
| Comment | File | Size | Author |
|---|---|---|---|
| #10 | at-commerce-breadcrumb-config-1783276-10.patch | 1.65 KB | francescoq |
| #5 | at-commerce-breadcrumb-label-checkbox-1783276-5.patch | 939 bytes | francescoq |
Comments
Comment #1
peterm95018 commentedYou can control the string "You are here" via the template.php file at line 355.
I've had a similar fix today.
peterm
Comment #2
DMDU commentedThank you!
(issue closed as fixed)
Comment #3
stan turyn commentedIt is still a valid bug though.
Comment #4
grabby commentedI agree, it’s still a valid bug, in addition to which the “Append the page title to the breadcrumb trail” option also has no effect. Not to harp on such a relatively minor issue, but it’s there.
Comment #5
francescoq commentedHere's a patch to display the label only if you choose to do this, also added a colon and a space between label and actual breadcrumb :)
Comment #6
Jeff Burnz commentedHarp on guys, nagging is often the best way of getting shit done, cause we forget basically, sounds bad but in a busy day, week, year etc, things go under the radar.
Comment #7
green monkey commented{dragging Harp out of closet and re-stringing}
issue still open
Comment #8
green monkey commentedharp :)
Comment #9
roball commentedCan confirm patch #5 perfectly solves both issues:
1. If the Show the "You are here" label checkbox is unticked in the Breadcrumb Settings, the label will not be shown.
2. If the checkbox is ticked, the label is displayed nicely with a colon and a space.
Comment #10
francescoq commentedI've used AT Commerce in another project, this time i have to display the page name after breadcrumb links but it doesn't work...
Here's the patch.. i don't know if it's the best way to do, but it works :)
edit: includes also #5
Comment #11
Kiendeleo commentedCan we get this tested and committed??
Comment #12
Jeff Burnz commentedOK, thank-you for the harping, much appreciated to bring this to my attention, again :)
Comment #13
Jeff Burnz commentedThis is now fixed in dev, ultimately I removed the custom breadcrumbs function and rely entirely on AT Core to do the heavy lifting for breadcrumb settings.
Comment #14
roball commentedThanks for keeping on this issue. But I cannot see any commit in Git after 11 Dec 2013?
Comment #16
roball commentedComment #17
roball commentedOK, can confirm this is fixed in AT Commerce 7.x-3.0 with AdaptiveTheme 7.x-3.2. Thank you.