-
Notifications
You must be signed in to change notification settings - Fork 5
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
Fixed Issue #18 #19
Fixed Issue #18 #19
Conversation
ShevinuM
commented
Aug 14, 2023
- Added the logo at top.
- Changed the format of the header with date and "Meeting Agenda" heading.
- Markdown doesn't let you change font-family so not sure how i can change the font.
Agendas/Agenda_TEMPLATE.md
Outdated
@@ -1,5 +1,7 @@ | |||
% MUN Computer Science Society |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would keep these like this, it's a part of how pandoc will create the PDF iirc.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
% MUN Computer Science Society % Meeting Agenda % <DATE (such as February 7th, 2019)>
Sorry didn't know that. Do you want the whole thing as it is with just only the logo added?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I just added some instructions to the README for trying the changes locally with Docker. It may not be perfect, but you can pull it into your fork.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
% MUN Computer Science Society % Meeting Agenda % <DATE (such as February 7th, 2019)>
Sorry didn't know that. Do you want the whole thing as it is with just only the logo added?
Right now, this is what it'd look like on your branch. The % signs must be on different lines, not seeing the picture. You can feel free to switch it up entirely or add some small things for #18, up to you. Thanks for contributing!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I will redo it and try out the changes locally.
Assets/mun_cs_society_logo.svg
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you make the SVG black instead of white? This won't show up super well with a white background on the PDF