Quick feature request - No scroll on popup Quick feature request - No scroll on popup
 

News:

cpg1.5.48 Security release - upgrade mandatory!
The Coppermine development team is releasing a security update for Coppermine in order to counter a recently discovered vulnerability. It is important that all users who run version cpg1.5.46 or older update to this latest version as soon as possible.
[more]

Main Menu

Quick feature request - No scroll on popup

Started by rphMedia, September 21, 2006, 10:29:24 PM

Previous topic - Next topic

0 Members and 1 Guest are viewing this topic.

Print
Go Down Pages1
User actions

rphMedia

September 21, 2006, 10:29:24 PM Last Edit: September 21, 2006, 10:39:07 PM by Nibbler
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. 

rphMedia

#2
September 21, 2006, 10:38:16 PM
I had a feeling ....... eh, quick 'hack' if they want it :)

Joachim Müller

#3
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.

rphMedia

#4
September 23, 2006, 01:48:54 AM
Looking forward to it, thanks, GauGau.
Print
Go Up Pages1
User actions