tuankhac / share-extras

Automatically exported from code.google.com/p/share-extras
0 stars 0 forks source link

Media Viewers - only shows the loading gif #120

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
What steps will reproduce the problem?
1. svn co ... 
2. ant
3. cp build/dist/media-viewers-2.0.jar 
.../tomcat/shared/lib/media-viewers-2.0.jar
4. configure Share as per wiki 
tomcat/shared/classes/alfresco/web-extension/org/alfresco/components/preview/web
-preview.get.config.xml
5. start alfresco and activate the module/s
6. navigate to a pdf documents detail page 

What version of the add-on are you using? Please provide version of the
downloaded package, or Subversion revision number.
media-viewers-2.0.jar  (Rev: 1363)

What version of Alfresco are you using? On what application server?
alfresco-4.2.c, tomcat version that comes with the installer

What web browser are you using? Please supply browser name and version.
Ubuntu/Firefox 17, Chromium v_22.0.1229.94 Ubuntu 12.10 (161065)

Please provide any additional information below.
Setting client-debug = true in share-config.xml triggers some magic and 
everything works as expected. I can preview, open, getting search terms 
highlighted ... happy days

Switching off client debugging and afterwards opening the developer consoles 
does not show any javascript errors. But resizing the consoles just a bit 
generates the JS-errors shown in the attachments
------------------

Side note:
There seems to be a problem with the Opera specific gradient CSS config ... 
firefox reports errors

share-extras-read-only/Media Preview$ svn diff
Index: source/web/extras/components/preview/PdfJs.css
===================================================================
--- source/web/extras/components/preview/PdfJs.css  (revision 1363)
+++ source/web/extras/components/preview/PdfJs.css  (working copy)
@@ -267,7 +267,7 @@
 }

 .previewer.PdfJs .sidebar .outlineView>:last-child {
-   margin-bottom: 0.5emt;
+   margin-bottom: 0.5em;
 }

 .previewer.PdfJs .sidebar .searchView input {
@@ -434,4 +434,4 @@

 /* === Printed media overrides === */
 @media print {
-}
\ No newline at end of file
+}
Index: source/web/extras/components/preview/viewer-common.css
===================================================================
--- source/web/extras/components/preview/viewer-common.css  (revision 1363)
+++ source/web/extras/components/preview/viewer-common.css  (working copy)
@@ -30,7 +30,7 @@

 .iframe-view-controls {
   background-color: #eee;
-  background: -o-linear-gradient(bottom,#eee 0%,#fff 100%);
+  /* background: -o-linear-gradient(bottom,#eee 0%,#fff 100%);*/
   background: -moz-linear-gradient(center bottom, #eee 0%, #fff 100%);
   background: -webkit-gradient(linear, left bottom, left top, color-stop(0.0, #ddd), color-stop(1.0, #fff));
   border-bottom: 1px solid #CCC;
@@ -70,4 +70,4 @@
    border: 1px solid #2b6eb5;
    text-decoration: none;
    padding-right: 5px;
-}
\ No newline at end of file
+}

Regards
Andreas

Original issue reported on code.google.com by andreasa...@gmail.com on 25 Dec 2012 at 11:27

Attachments:

GoogleCodeExporter commented 8 years ago
Hi, thanks for the thorough report. Can you confirm what command you used to 
build the add-on? You should be using the project.xml build script.

Original comment by will.abson on 9 Jan 2013 at 4:06

GoogleCodeExporter commented 8 years ago
Hi, looks like I missed that part somehow!
Anyway, building with 'ant -f project.xml dist-jar' helped quite a bit. It is 
now also working with client-debug off. Pdfjs is so much better than the ootb 
flash stuff.
Thanks

Original comment by andreasa...@gmail.com on 9 Jan 2013 at 10:13

GoogleCodeExporter commented 8 years ago
Thanks! Glad it works now. The project.xml file is easy to miss though, so I've 
opened issue 124 to track if something can be done to warn if the wrong file is 
used.

Original comment by will.abson on 10 Jan 2013 at 2:01