[cvs] commit: ansel/docs INSTALL ansel/lib Ansel.php ansel/templates menu.inc ansel/templates/view gallery.inc gallerylightbox.inc image.inc list.inc results.inc slideshow.inc ansel/themes screen.css
Ben Klang
ben at alkaloid.net
Mon Oct 27 03:57:48 UTC 2008
bklang 2008-10-26 23:57:48 EDT
Modified files:
docs INSTALL
lib Ansel.php
templates menu.inc
templates/view gallery.inc gallerylightbox.inc image.inc
list.inc results.inc slideshow.inc
themes screen.css
Log:
Collapse the breadcrumb trail from it's own div into the image title. This was suggested by my users while having them test Ansel's UI. In their opinion the breadcrumb trail was small and the eye tended to skip over it. There was also redundant information between the breadcrumbs and the content header. To address this I collapsed the breadcrumb trail into the content header and added emphasis to the current location. Doing this required reversing the logic of Ansel::getBreadcrumbs() which is why the patch looks so huge. In fact the logic is virtually unchanged, it's just constructed in reverse order (from current location backward up the parents).
Revision Changes Path
1.9 +3 -3 ansel/docs/INSTALL
1.527 +98 -61 ansel/lib/Ansel.php
1.6 +0 -1 ansel/templates/menu.inc
1.178 +2 -2 ansel/templates/view/gallery.inc
1.30 +1 -1 ansel/templates/view/gallerylightbox.inc
1.87 +1 -1 ansel/templates/view/image.inc
1.13 +1 -1 ansel/templates/view/list.inc
1.31 +1 -1 ansel/templates/view/results.inc
1.15 +1 -1 ansel/templates/view/slideshow.inc
1.61 +9 -5 ansel/themes/screen.css
Chora Links:
http://cvs.horde.org/diff.php/ansel/docs/INSTALL?r1=1.8&r2=1.9&ty=u
http://cvs.horde.org/diff.php/ansel/lib/Ansel.php?r1=1.526&r2=1.527&ty=u
http://cvs.horde.org/diff.php/ansel/templates/menu.inc?r1=1.5&r2=1.6&ty=u
http://cvs.horde.org/diff.php/ansel/templates/view/gallery.inc?r1=1.177&r2=1.178&ty=u
http://cvs.horde.org/diff.php/ansel/templates/view/gallerylightbox.inc?r1=1.29&r2=1.30&ty=u
http://cvs.horde.org/diff.php/ansel/templates/view/image.inc?r1=1.86&r2=1.87&ty=u
http://cvs.horde.org/diff.php/ansel/templates/view/list.inc?r1=1.12&r2=1.13&ty=u
http://cvs.horde.org/diff.php/ansel/templates/view/results.inc?r1=1.30&r2=1.31&ty=u
http://cvs.horde.org/diff.php/ansel/templates/view/slideshow.inc?r1=1.14&r2=1.15&ty=u
http://cvs.horde.org/diff.php/ansel/themes/screen.css?r1=1.60&r2=1.61&ty=u
More information about the cvs
mailing list