Skip to content
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

Regular Paper Page #16

Closed
3 tasks done
hao-fang opened this issue May 25, 2020 · 9 comments · Fixed by #158
Closed
3 tasks done

Regular Paper Page #16

hao-fang opened this issue May 25, 2020 · 9 comments · Fixed by #158
Assignees

Comments

@hao-fang
Copy link

hao-fang commented May 25, 2020

Number of Volunteers: 1

ICLR Layout

image

Update (2020-06-12)

Rre-org the components for the paper webpage. Currently, it looks like this
image

  • Remove the red text "Add content for posters. ...."
  • Put the SlidesLive & RocketChat in the same row.
  • We will not have the poster PDF. Check if there is a way to disable it (rather than deleting the code).
@hao-fang hao-fang changed the title RocketChat Integration RocketChat & SlidesLive Integration May 25, 2020
@hao-fang hao-fang changed the title RocketChat & SlidesLive Integration RocketChat & SlidesLive & Zoom Integration May 28, 2020
@hao-fang hao-fang changed the title RocketChat & SlidesLive & Zoom Integration Regular Paper Page Jun 4, 2020
@hao-fang hao-fang self-assigned this Jun 10, 2020
@salma-elanigri salma-elanigri self-assigned this Jun 11, 2020
@hao-fang
Copy link
Author

@salma-elanigri Thanks for volunteering on this issue. I updated the instruction (see the section "Update (2020-06-12)".

Please check these https://github.com/acl-org/acl-2020-virtual-conference/blob/master/CONTRIBUTING.md and https://github.com/acl-org/acl-2020-virtual-conference/blob/master/CONTRIBUTING-ACL2020.md for setting up your local environment.
You can launch a local server via make run.

Let me know if you have any questions :)

@salma-elanigri
Copy link

Okay, thank you @hao-fang

@hao-fang
Copy link
Author

@salma-elanigri I'll answer your RocketChat question here for visibility so that other people can refer to this in future.

The template is indeed specified by the template/poster.html. This is a Jinja2 template. See details here.
The data paper is populated in main.py here. See the method format_paper for more details.
The by_uid is populated here.

So for this specific issue, you will need to edit the template/poster.html.
To see your changes locally, you can run make run and open your browser to look at the outcome.

Feel free to let me know if you have more questions.

This was referenced Jun 16, 2020
@raosudha89
Copy link

raosudha89 commented Jun 16, 2020

Note from PCs:

  • The paper title should appear in a different color than the 'Live', 'Q&A', 'Cal' links to show that the title is not clickable.
  • Put video + slides first and then the rocket chat window
  • See if we can always show the abstract (without the page becoming too long).
  • See if we can display the time of the paper in the current timezone of the attendee (as used by the schedule calendar).

@jcklie
Copy link
Collaborator

jcklie commented Jun 22, 2020

@salma-elanigri How is your progress so far? I would like to support you on this task but do not want to get in your way. Which tasks are you working on right now and which can I take?

@hao-fang hao-fang linked a pull request Jun 22, 2020 that will close this issue
@hao-fang hao-fang reopened this Jun 22, 2020
@salma-elanigri
Copy link

Hi @jcklie ! I am working on this task " See if we can display the time of the paper in the current timezone of the attendee (as used by the schedule calendar). " I tried to use the moments and moment-timezonejs to convert the session.time to the user time zone but I don't how to make it work inside a Jinja loop.

@jcklie
Copy link
Collaborator

jcklie commented Jun 22, 2020

You can use the static/js/time-extend.js function we already wrote and use throughout the website, e.g. on the sponsor page. You need to format the time in a particular format, then it does the rest for you.

@salma-elanigri
Copy link

thanks I will check this.

@raosudha89
Copy link

Looks like all tasks in this issue are done. Thank you very much @salma-elanigri and @jcklie

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants