Project

General

Profile

Bug #7381

Eye icon in calendar list view non-reactive

Added by Yannick Warnier about 5 years ago. Updated about 5 years ago.

Status:
Bug resolved
Priority:
Low
Category:
Agenda
Target version:
Start date:
08/11/2014
Due date:
% Done:

100%

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

Description

In the calendar list view, the eye icon doesn't do anything!?

History

#1

Updated by Jan Derriks about 5 years ago

Is this one solved?
We have several issues with the eye-icons since a few months. Creating a new course or changing the intro text sometimes 'disables' all the eye icons on a course frontpage,
It is some javascript code bug but have not found the cause yet.

#2

Updated by Yannick Warnier about 5 years ago

  • Assignee changed from Yannick Warnier to Daniel Barreto

Hi Jan, no, this is not javascript in this case but just the fact that the eye icon is not made to work in the global agenda view (just on the course-specific agenda view).

If you can reproduce the other bug you are talking about (all eye icons being disabled), please register a new bug. I myself have never experienced it.

#3

Updated by Daniel Barreto about 5 years ago

  • % Done changed from 0 to 60

Fix change visibility action when using agenda out of the course tool

Send PR428
https://github.com/chamilo/chamilo-lms/pull/428

#4

Updated by Yannick Warnier about 5 years ago

Comment sent in the PR.
The fix works. It's just a matter of a slight code form review.

#5

Updated by Daniel Barreto about 5 years ago

  • Assignee changed from Daniel Barreto to Yannick Warnier

Replace course_id by cid in PR433
https://github.com/chamilo/chamilo-lms/pull/433

#6

Updated by Yannick Warnier about 5 years ago

  • Status changed from Assigned to Bug resolved
  • Assignee changed from Yannick Warnier to Daniel Barreto
  • % Done changed from 60 to 100

I had to make an additional fix because of one course_id that had been changed to cid but that shouldn't. Apart from that, this is working now.

#7

Updated by Daniel Barreto about 5 years ago

Add PR438 to fix about event types (admin and personal)
https://github.com/chamilo/chamilo-lms/pull/438

Also available in: Atom PDF