This is a snapshot of Indico's old Trac site. Any information contained herein is most probably outdated. Access our new GitHub site here.

Opened 3 years ago

Closed 3 years ago

Last modified 3 years ago

#1071 closed defect (fixed)

Webcast/Recording Request show talks when select all

Reported by: arescope Owned by: arescope
Priority: normal Milestone: v0.99.0
Component: Video Services Version:
Keywords: Cc:

Description

Sometimes we select the request to all the event and some talks are not webcastable, we should show which ones are going to be recorded.

Change History (17)

comment:1 Changed 3 years ago by arescope

  • Component changed from General to Video Services
  • Owner set to arescope
  • Status changed from new to assigned

comment:2 Changed 3 years ago by arescope

  • Status changed from assigned to awaiting_merge

comment:3 Changed 3 years ago by jbenito

  • Status changed from awaiting_merge to infoneeded

comment:4 Changed 3 years ago by jbenito

  • Status changed from infoneeded to assigned

comment:5 Changed 3 years ago by arescope

  • Status changed from assigned to awaiting_merge

comment:6 Changed 3 years ago by pferreir

  • Status changed from awaiting_merge to assigned

First of all, my changes:

http://indico-software.org/repo/personal/indico-pferreir/commit/?h=1071-imp-show-talks-webcast&id=baf5229f6ce3836bc1811868983606b0506432f6

Then, I wonder whether it's worth showin the contributions that are not in a recordable room more than the ones that are. Normally the user will want to know what is causing the event not to be completely broadcast/recorded, right? Not the opposite...

I added some yellowesque styling to the warning message, but the content still appears very plain. Maybe we should work a bit the way room/contribution lists are shown.

comment:7 Changed 3 years ago by pferreir

OK, after discussing it with Jose, we realized it would be nice to have both things - the contribs that are going to be webcasted/recorded and the ones that are not. Maybe making the numbers clickable and showing some kind of dialog?
We should also use an AJAX request, as pre-loading this data can be too expensive for large events.

comment:8 Changed 3 years ago by arescope

  • Status changed from assigned to awaiting_merge

comment:9 Changed 3 years ago by pferreir

  • Status changed from awaiting_merge to merging

comment:10 Changed 3 years ago by pferreir

  • The dialog breaks if there is a contribution that is unscheduled.
  • Split strings are awful for i18n. Maybe we can try something like this?
    format(
      $T('There are {0} talks in the event. <a class="uncapable">Some of them ({1})</a> ' + 
           'are not in a room capable of recording and thus cannot be recorded while ' +
           '<a class="capable">the remaining ({2})</a> are going to be recorded.'),
           [/* ... */]);
    
    // [...]
    
    $('.warning .uncapable').click( /* ... */);
    
Version 2, edited 3 years ago by pferreir (previous) (next) (diff)

comment:11 Changed 3 years ago by pferreir

  • Status changed from merging to assigned

comment:12 Changed 3 years ago by arescope

  • Status changed from assigned to awaiting_merge

comment:13 Changed 3 years ago by Alberto Resco Perez <alberto.resco.perez@…>

  • Resolution set to fixed
  • Status changed from awaiting_merge to closed

In 666402198882858bbbde99aa80c0072f3e597468/indico:

[IMP] Show webcasted and recorded talks

  • When you select all talks you do not knew which ones are recorded. Now yes.
  • Closes #1071.

comment:14 Changed 3 years ago by Pedro Ferreira <jose.pedro.ferreira@…>

In 0c907a39ba08dfea199262706e36a93a4924d42b/indico:

[IMP] Style and code

  • Replaced inline ifs with proper readable versions;
  • Improved the styling of the warning;
  • see #1071;

comment:15 Changed 3 years ago by Alberto Resco Perez <alberto.resco.perez@…>

In 4669207edcee5678821ca1138eee43004de446e1/indico:

[IMP] Show contributions in a popup

  • Shows the webcast/record-able and the unable.
  • See #1071.

comment:16 Changed 3 years ago by Alberto Resco Perez <alberto.resco.perez@…>

In e3c78f2d0902e26155e91bbafd17755c12480ea9/indico:

[FIX] Dialog crashed when talks not scheduled

comment:17 Changed 3 years ago by Pedro Ferreira <jose.pedro.ferreira@…>

Note: See TracTickets for help on using tickets.