Project

General

Profile

Feature #1357

Learning Path > View > adapt TOC height to window size

Added by Hubert Borderiou about 9 years ago. Updated about 8 years ago.

Status:
Bug resolved
Priority:
Normal
Category:
-
Target version:
Start date:
25/05/2010
Due date:
% Done:

90%

Estimated time:
Complexity:
Normal
SCRUM pts - complexity:
?

Description

Hi,

the TOC height in Learning Path is fixe.
I've modified the TOC height with the window height.
(see attached)
It's Javascript made, and I've tested it with several browsers on windows and MAC OS X and it looks fine.

I've modified files main/newscorm/learnpath.class.php and main/newscorm/lp_view.php

File main/newscorm/learnpath.class.php
Add id attribute for object scorm_title and actions_lp

$html = '<div id="scorm_title" class="scorm_title"><div class="scorm_title_text">'
. Security::remove_XSS(api_convert_encoding($this->get_name(), $this->encoding, $charset)) . '</div></div>';

$html .= '<div id="actions_lp" class="actions_lp">';

File main/newscorm/lp_view.php
(I did some cleaning too.)
See attached

Hope this help.


Files

2010-05-25_1720.png (95.2 KB) 2010-05-25_1720.png screen copy Hubert Borderiou, 25/05/2010 17:26
lp_view.php (16.8 KB) lp_view.php lp_view.php Hubert Borderiou, 25/05/2010 17:26
lp_view.php (18.3 KB) lp_view.php lp_view file for version 1.8.7 Hubert Borderiou, 26/05/2010 10:26
lp_view.php (18.3 KB) lp_view.php lp_view file for version 1.8.7 Hubert Borderiou, 26/05/2010 10:27

Related issues

Related to Chamilo LMS - Bug #1037: Visual corrections in LP tool are neededFeature implemented20/04/2010

Actions

History

#1

Updated by Julio Montoya about 9 years ago

  • Assignee set to Julio Montoya

Going to check it

#2

Updated by Hubert Borderiou about 9 years ago

I've made a mistake, and the file lp_view.php is the 1.8.6.1 one.
Here it is the 1.8.7 lp_view.php file.
See attached.
Regards

#3

Updated by Julio Montoya about 9 years ago

  • Status changed from New to Needs testing
  • Assignee changed from Julio Montoya to Curt Ricardo Rodriguez Salazar
  • % Done changed from 0 to 90
#4

Updated by Julio Montoya about 9 years ago

  • Target version changed from 1.8.7.1 to 1.8.7
#5

Updated by Yannick Warnier about 8 years ago

  • Status changed from Needs testing to Bug resolved

Also available in: Atom PDF