adjust close button position and style

Author: DeepDiver1975

3rdparty/pdfjs/viewer.css

@@ -501,7 +501,7 @@ html[dir='ltr'] #toolbarViewerRight,
 html[dir='rtl'] #toolbarViewerLeft {
   position: absolute;
   top: 0;
-  right: 0;
+  right: 25px;
 }
 html[dir='ltr'] #toolbarViewerLeft > *,
 html[dir='ltr'] #toolbarViewerMiddle > *,

js/loader.js

@@ -8,31 +8,28 @@ function hidePDFviewer() {
 	}
 }
 
-function showPDFviewer(dir,filename){
-	if(!showPDFviewer.shown){
+function showPDFviewer(dir,filename) {
+	if(!showPDFviewer.shown) {
 		var $iframe;
 		$("#editor").hide();
 		$('#content table').hide();
 		$("#controls").hide();
-		var oldcontent = $("#content").html();
 		var viewer = OC.linkTo('files_pdfviewer', 'viewer.php')+'?dir='+encodeURIComponent(dir).replace(/%2F/g, '/')+'&file='+encodeURIComponent(filename);
 		$iframe = $('<iframe id="pdframe" style="width:100%;height:100%;display:block;" src="'+viewer+'" /><div id="pdfbar"><a id="close" title="Close">X</a></div>');
-		if ($('#isPublic').val()){
+		if ($('#isPublic').val()) {
 			// force the preview to adjust its height
 			$('#preview').append($iframe).css({height: '100%'});
-		}
-		else{
+		} else {
 			$('#content').append($iframe);
 		}
-		$("#pageWidthOption").attr("selected","selected");	
-		$('#pdfbar').css({position:'absolute',top:'5px',right:'5px'});
+		$("#pageWidthOption").attr("selected","selected");
+		$('#pdfbar').css({position:'absolute',top:'6px',right:'5px'});
 		// if a filelist is present, the PDF viewer can be closed to go back there
-		if ($('#fileList').length){
-			$('#close').css({display:'block',padding:'0 5px',color:'#BBBBBB','font-weight':'900',height:'18px',background:'transparent'}).click(function(){
+		if ($('#fileList').length) {
+				$('#close').css({display:'block',padding:'0 5px',color:'#BBBBBB','font-weight':'900','font-size':'16px',height:'18px',background:'transparent'}).click(function(){
 				hidePDFviewer();
 			});
-		}
-		else{
+		} else {
 			$('#close').css({display:'none'});
 		}
 	}