Project

General

Profile

Feature #3754

Remove yoxview for non-images

Added by Yannick Warnier almost 8 years ago. Updated almost 8 years ago.

Status:
Bug resolved
Priority:
High
Assignee:
Category:
-
Target version:
Start date:
27/07/2011
Due date:
% Done:

100%

Estimated time:
0.40 h
Complexity:
Easy
SCRUM pts - complexity:
3

Description

There are too many users report with problems opening things that are not images through yoxview. Yoxview is great for images, but for anything else it's just not the appropriate thing.

So, for 1.8.8.4, yoxview should only be enabled for:
  • png
  • jpg/jpeg
  • gif
  • bmp
  • svg

Nothing more.


Related issues

Related to Chamilo LMS - Feature #1751: Documents tool: The frame for document preview should have adaptive heightNeeds more info22/07/2010

Actions

Associated revisions

Revision cd36dc55 (diff)
Added by Julio Montoya almost 8 years ago

Yoxview only for images and swf see #3754

History

#1

Updated by Julio Montoya almost 8 years ago

  • Status changed from Assigned to Needs more info
  • Assignee changed from Julio Montoya to Yannick Warnier

So ... we will not use yoxview for HTML files? We will have the problem of height when viewing HTML files .... see #1751

If we are going to use a popup method for images we would used jquery dialog or thickbox and not adding a new lib like yoxview ...

#2

Updated by Yannick Warnier almost 8 years ago

  • Status changed from Needs more info to Assigned
  • Assignee changed from Yannick Warnier to Julio Montoya

Just leave yoxview for now. For the next version, we'll add a setting that allows you to enable it at course level.

Too bad we then still have the problem with #1751, but the current solution is worst as it prevents some JavaScript inside the HTML to work (or essential navigation elements to appear).

#3

Updated by Julio Montoya almost 8 years ago

  • Status changed from Assigned to Needs more info
  • Assignee changed from Julio Montoya to Yannick Warnier
  • % Done changed from 0 to 50
#4

Updated by Yannick Warnier almost 8 years ago

  • Status changed from Needs more info to Bug resolved
  • Assignee changed from Yannick Warnier to Julio Montoya
  • % Done changed from 50 to 100

Good for me. Thanks

Also available in: Atom PDF