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

Minor changes to some cover elements #52

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

jtsimoes
Copy link
Member

This is a small PR that removes an incorrect space before comma (sorry, my OCD can't handle it 🙃) on the second page, next to the degree name.

I've also taken the opportunity to fix the title of the new section on acknowledging the use of AI, in accordance with the guidelines (portuguese short form for "AI" is "IA"), and also to add the english version of that same title, in line with the other sections that also have it (jury, acknowledgements, ...).

Before After
Space after degree name 0_before 0_after
Space after degree name (w/ MAP Doctoral program) 1_before 1_after
AI acknowledgement section title 3_before 3_after

@jtsimoes
Copy link
Member Author

jtsimoes commented Aug 31, 2024

If there is interest, I can add one more change to this PR: to allow the introduction of text in PT and EN in these common sections (acknowledgement of use of AI tools and acknowledgments, since there's no need on the jury section). This could solve issue #5

I tried to do it directly in cover.tex with \newline or with \\, to better distinguish the two:

\Acknowledgements{
    Agradeço toda a ajuda a todos os meus colegas e companheiros.
    \newline
    \newline
    I would like to thank all my colleagues and colleagues for their help.
}

But it gives an error, probably because it is not allowed to use commands inside another command. That's why I think I need to split the command in two parameters, to allow formatting.

Like so:

\newcommand{\MAPProgram}[1]{\def\ciMAPProgram{#1}}
\newcommand{\GrantText}[2]{\def\ciGrantFirst{#1}\def\ciGrantSecond{#2}}
- \newcommand{\Acknowledgements}[1]{\def\ciAcknowledgements{#1}}
+ \newcommand{\Acknowledgements}[2]{\def\ciAcknowledgementsPT{#1}\def\ciAcknowledgementsEN{#2}}
\newcommand{\JuriMember}[2]{
    \listadd{\ciJuriMember}{\JURI{\JURICATEGORY}{#1}{#2}}
}

...

%
% Acknowledgement page
%
%
-\ifdefined \ciAcknowledgements
+\ifdefined \ciAcknowledgementsPT
\TitlePage
  \vspace*{95mm}
  \SECTION{\textbf{agradecimentos~/\newline acknowledgements}}
-       {\ciAcknowledgements}
+       {\ciAcknowledgementsPT \newline \newline \ciAcknowledgementsEN}
\EndTitlePage
\fi

The EN form would be optional, of course.

I know it's not mandatory for the submission, but I'm doing it on mine, so it wouldn't cost me anything to migrate those changes here too

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

Successfully merging this pull request may close these issues.

1 participant