[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