jayroh/thredded — pull request #27 by danneu (2012-04-14 05:20:22+00:00)
Title: Breadcrumbs: Removed floating arrows, Added current depth. Truncate long breadcrumbs.

This is my first pull request, ever.

I took it upon myself to address something that's purely aesthetic without discussing it, so I can handle rejection. I don't know why I picked something so subjective. :( But here's my case:

1. The floating arrow at the end of the breadcrumbs looked like a mistake and inconsistant with how breadcrumbs generally work. I removed the floating arrow.

2. Breadcrumbs conventionally show the current depth (current page), so I added a final breadcrumb element that reflects the current page. It's not hyperlinked.

![](http://i.imgur.com/Kx5bL.png)

3. Should the current page title be very long, the final breadcrumb would be too prominent, cluttering the breadcrumbs. At worst, it would stretch on for way to long. I truncated the length of the final breadcrumb to the default 30 chars.

![](http://i.imgur.com/jGEfX.png)