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

remove Skill codes in condensed version of paper #381

Closed
mhagdorn opened this issue Oct 29, 2024 · 4 comments · Fixed by #382
Closed

remove Skill codes in condensed version of paper #381

mhagdorn opened this issue Oct 29, 2024 · 4 comments · Fixed by #382

Comments

@mhagdorn
Copy link
Collaborator

I think the Skill codes section does not make so much sense in the condensed version of the paper. I reckon we can remove that too.

Do we want to keep the icons for the condensed version?

@jpthiele
Copy link
Collaborator

I am not completely sure what you mean exactly.

@mhagdorn
Copy link
Collaborator Author

I am not completely sure what you mean exactly.

At the end of the condensed paper we have the Sections Glossary, Skill codes and Acronyms. I think we should definitely remove the Glossary, it only has two items. I think we can also drop the Acronyms, just expand them inline (which I think we are already doing). The remaining one Skill codes I am not so sure about. I think in the condensed version of the paper it doesn't make as much sense as in the full paper. So, I tend to dropping that as well.

@jpthiele
Copy link
Collaborator

Ahh, I had that all filed under 'removed glossary' in my mind.
Yes I don't think we need the section in a shorter version of the paper,
while the hover is a nice feature I don't think too many people will notice that anyways.

This was referenced Oct 30, 2024
@mhagdorn
Copy link
Collaborator Author

you are quite right, I didn't realise all three sections were controlled by the glossary. Removing all three was also easy, see PR #382

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 a pull request may close this issue.

2 participants