News:

Rule #1 - Be Patient - Rule #2 - Don't ask when, if you don't contribute - Rule #3 - You have coding skills - LinuxMCE's small brother is available: http://www.agocontrol.com

Main Menu

Scroll bars in web-admin

Started by PeteK, July 20, 2008, 09:46:38 PM

Previous topic - Next topic

PeteK

Hi gang--

I encountered a problem when trying to move a media file via web-admin.  When I select the 'Move file' button on the file's page (through Media Files Sync), It opens a new window to allow you to select a new destination.  Unfortunately, this window does not have scroll bars, and I can't see the items off the bottom of the page.  Is there a way to allow scrolling in this page?

The .php file is: charon-merge/web/pluto-admin/languages/en/moveFile.lang.php

Thanks,
-PeteK

jondecker76

I just took a peak. if you look at line 129 in charon-merge/web/pluto-admin/operations/mediaBrowser/editMediaFile.php you will see this window opened with

<input type="button" class="button" name="moveFile" value="Move file to other directory" onClick="windowOpen(\'index.php?section=moveFile&fileID='.$fileID.'&filePath='.urlencode(stripslashes($rowFile['Path']).'/'.$rowFile['Filename']).'\',\'width=500,height=400,toolbars=true,scrollbars=true,resizable=1\');">

so it appears to be opened with both scrollbars, and being resizable (though I'm not sure if the resizable=1 should be resizable=true. You may want to look at whether changing the "true" to 1, or vice-versa.

I couldn't reproduce because I don't have many storage devices...

Zaerc

Quote from: jondecker76 on July 21, 2008, 02:44:51 AM
I just took a peak. if you look at line 129 in charon-merge/web/pluto-admin/operations/mediaBrowser/editMediaFile.php you will see this window opened with

<input type="button" class="button" name="moveFile" value="Move file to other directory" onClick="windowOpen(\'index.php?section=moveFile&fileID='.$fileID.'&filePath='.urlencode(stripslashes($rowFile['Path']).'/'.$rowFile['Filename']).'\',\'width=500,height=400,toolbars=true,scrollbars=true,resizable=1\');">

so it appears to be opened with both scrollbars, and being resizable (though I'm not sure if the resizable=1 should be resizable=true. You may want to look at whether changing the "true" to 1, or vice-versa.

I couldn't reproduce because I don't have many storage devices...

I think you might be on to something there jondecker76, according to this page: http://www.w3schools.com/htmldom/met_win_open.asp the values for those should be set to "on", "off", "0" or "1" and no mention of "true".  So it could just be just another minor bug to fix.
"Change is inevitable. Progress is optional."
-- Anonymous

[url=http://petition.stopsoftwarepatents.eu/181001941347/][/url]

PeteK

Thanks guys!  I can verify when I get home, if someone doesn't beat me to it.


-PeteK


PeteK

Setting scrollbars=1 worked.

The other change that needs to be made is to replace toolbars=true with: toolbar=1 (note that the correct parameter is 'toolbar,' not 'toolbars'

This was tested with firefox in linux.

I don't have CVS commit permissions, so if one of you could commit the changes, I'd definitely appreciate it.  Or I can bug DanielK for permissions.

Thanks a lot for the help, guys.

-PeteK

PeteK