Project

General

Profile

Bug #7381

Eye icon in calendar list view non-reactive

Added by Yannick Warnier over 4 years ago. Updated over 4 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 over 4 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 over 4 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 over 4 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 over 4 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 over 4 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 over 4 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 over 4 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