New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Conference timetable with indico style: add a link to event home page #3297

Closed
jouvin opened this Issue Apr 6, 2018 · 5 comments

Comments

Projects
None yet
4 participants
@jouvin
Contributor

jouvin commented Apr 6, 2018

Often, people prefer to use the Indico style timetable to the parallel one for conferences without parallel sessions as it is easier to navigate. This removes the conference menu on the left which is fine but has the drawback to remove any link to the event home page. Would it be possible to add a link back to the event page in the timetable in this case?

@bpedersen2

This comment has been minimized.

Contributor

bpedersen2 commented Apr 10, 2018

You can do this via template customisation:
locate indico/modules/events/templates/display/compact.html
copy it to custom/core/events/timetable/display/compact.html in your indico root

Change:
{{ event.title }}
to
<a href="{{event.url}}">{{ event.title }}</a>

This snippet would link the title to the 'normal' page.

@ThiefMaster : Should I create a pull request with this change?

@ThiefMaster

This comment has been minimized.

Member

ThiefMaster commented Apr 10, 2018

I don't think this is only about the "compact" style but rather using meeting themes for conferences in general. While having a link back sounds like a good idea, I'm not sure how useful it would be in an actual meeting/lecture (the link would point to the current page after all).

@jouvin

This comment has been minimized.

Contributor

jouvin commented Apr 10, 2018

@bpedersen2 thanks for snippet, it may help in the short term... but I agree with @ThiefMaster that it would be good if we could ensure that there is always a link back to the main event page from the conference timetables, whatever style is used. At least a useless link is harmless...

@pferreir

This comment has been minimized.

Member

pferreir commented Apr 10, 2018

This is the good old problem of people "abusing" the meeting layout. Unfortunately, Pandora's box was opened long time ago and it's too late to just kill that possibility. Until we come up with a better conference timetable view, I agree that adding a link back won't hurt.

bpedersen2 added a commit to bpedersen2/indico that referenced this issue Apr 10, 2018

Add backlink to conference in title
For conference, if a theme is selected, add a backlink
to the main conference page.

Just using a home icon is not useful here, as the icon already
is used for the link to the indico home.

Fixes: indico#3297
@bpedersen2

This comment has been minimized.

Contributor

bpedersen2 commented Apr 10, 2018

Agreed, this would need to be added to all meeting-like themes if the type is conference.

I think adding it to the header would work universally.

bpedersen2 added a commit to bpedersen2/indico that referenced this issue Apr 10, 2018

Add backlink to conference in title
For conference, if a theme is selected, add a backlink
to the main conference page.

Just using a home icon is not useful here, as the icon already
is used for the link to the indico home.

Fixes: indico#3297

bpedersen2 added a commit to bpedersen2/indico that referenced this issue Apr 10, 2018

Add backlink to conference in title
For conference, if a theme is selected, add a backlink
to the main conference page.

Just using a home icon is not useful here, as the icon already
is used for the link to the indico home.

Fixes: indico#3297

ThiefMaster added a commit to bpedersen2/indico that referenced this issue Apr 18, 2018

Add backlink to conference in title
For conference, if a theme is selected, add a backlink
to the main conference page.

Just using a home icon is not useful here, as the icon already
is used for the link to the indico home.

Fixes: indico#3297
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment