batpad / jquery-rbox

Responsive lightbox jQuery plugin
Other
0 stars 3 forks source link

Global CSS application on Content #7

Closed moahmed closed 10 years ago

moahmed commented 10 years ago

Ideally the rbox overlay should not have its own css but use what is in the content to be displayed.

For example if i want to show some narrow content in the lightbox, i use max-width:700px but for another link i need max-width:800px ... this will not be possible unless we uses some messy code to change the max-width of the overlay-content.

Stuff like background color of content to be displayed in the lightbox, etc should not be part of the plugin code.

batpad commented 10 years ago

If you have a list of options that would be useful to pass in, let me know. The container of the lightbox has to be defined in the plugin code since it dynamically re-adjusts itself according to size, etc. It should be fairly trivial to add some options for say backgroundColor and maxWidth or so, though. Let me know if this makes sense and if you could work in the code yourself and send a patch, or I can look into implementing options for maxWidth and backgroundColor - anything else you can think of?

Thanks for the issues, btw.