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" <[email protected]>
Newsgroups gmane.comp.horde.cvs
Message-ID <[email protected]>
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

--
To unsubscribe, mail: [email protected]
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.