[dev] [cvs] commit: ansel .htaccess gallery.php ansel/docs lighttpd-ansel.conf ansel/lib Ansel.php api.php ansel/lib/Views Abstract.php Gallery.php GalleryLightbox.php ansel/templates/view gallery.inc gallerylightbox.inc

[Michael Rubinsky]

Quoting Chuck Hagenbuch <chuck at horde.org>:

> Quoting Michael Rubinsky <mrubinsk at horde.org>:
>
>> I actually did think of going this route, but thought that it would
>> be  ambiguous from a user's point of view...is the second '1' a
>> page, is  it an image? Since we were already doing gallery/x/image/y
>> I thought  that this fit in better.
>
> Well, it'd be clear on the page issue if we did page=x. I think
> regardless of whether or not we can remove the gallery/ part, I'd
> prefer to see something different from more directory levels for
> parameters.

OK. So, should we just leave it as a GET parameter then?  Not sure if  
it's worth parsing the URL just to remove the '?' especially if there  
will be other query parameters tacked on at the end.

Also, do you want to remove the 'image/' level from the URL as well?

ansel/gallery/1/1 would be image 1 of gallery 1?



>
>> I guess we could do something with key=val pairs in the url for a
>> grouping attribute or something (user/category/tag etc...), but IMO,
>>  this defeats the purpose of doing pretty, (more RESTful?) urls.
>
> Well, part of being restful is that a URL should be a resource. It's
> arguable about whether a page in a gallery is a separate resource - my
> inclination would be that it's not.
>
> -chuck
>
> --
> Horde developers mailing list - Join the hunt: http://horde.org/bounties/
> Frequently Asked Questions: http://horde.org/faq/
> To unsubscribe, mail: dev-unsubscribe at lists.horde.org
>



Thanks,
mike

--
The Horde Project (www.horde.org)
mrubinsk at horde.org

"Time just hates me. That's why it made me an adult." - Josh Joplin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: PGP Digital Signature
Url : http://lists.horde.org/archives/dev/attachments/20080526/3d34521b/attachment.bin