On 10/29/07, Michael Baierl mail@mbaierl.com wrote:
till wrote:
Good idea, though I am not sure if I want thickbox because this would add another dependency and thickbox is (apart from some CSS) not very customizable and we technically would need to re-customize it when we update it.
Not sure if it's hard to implement that yourself with jQuery (since that is what we'll be using in the future).
Till
http://jquery.com/demo/thickbox/ is a jQuery extension, so if you start using jQuery you can also use all plugins availabe (i.e. dimensions, UI etc.). And on advice - never ever patch any upstream library, as you already said, any update is going to be hell. Fear it. :)
Yeah, I know what Thickbox is. As I said, I don't think it's suitable because e.g. it lacks localization features and so on. So unless we try to adapt Thickbox and create a Thickbox-Roundcube from it, I'd say, start from scratch. ;-)
Till _______________________________________________ List info: http://lists.roundcube.net/dev/