forum.coppermine-gallery.net

No Support => Feature requests => Topic started by: rphMedia on September 21, 2006, 10:29:24 pm

Title: Quick feature request - No scroll on popup
Post by: rphMedia on September 21, 2006, 10:29:24 pm: Also a quick mod :)

If the popup from the intermediate image is larger than the current screen resolution, scrollbars are definitely needed - no question. But if the popup is smaller than the display res, there shouldn't be the ugly (vertical) scroll bar positioning in the browser. This tiny hack will take care of that and also, just requesting that maybe this be implemented in the future? If it breaks validation somehow, then I totally understand. I just think the popup looks neater (for lack of a better word).

In themes.inc.php find
Code: [Select]
<body>
and replace with

Code: [Select]
<body scroll="auto">
It's that simple.
Title: Re: Quick feature request - No scroll on popup
Post by: Nibbler on September 21, 2006, 10:34:31 pm: It's not valid XHTML.
Title: Re: Quick feature request - No scroll on popup
Post by: rphMedia on September 21, 2006, 10:38:16 pm: I had a feeling ....... eh, quick 'hack' if they want it :)
Title: Re: Quick feature request - No scroll on popup
Post by: Joachim Müller on September 22, 2006, 09:53:03 pm: In cpg1.5.x, there will be two additional options in coppermine's config that will allow you to specify the x and y offset in the pop-up, allowing standards-compliant pop-ups that don't have scroll-bars.
Title: Re: Quick feature request - No scroll on popup
Post by: rphMedia on September 23, 2006, 01:48:54 am: Looking forward to it, thanks, GauGau.