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

Umstieg auf Glossaries-Package #134

Open
wants to merge 20 commits into
base: main
Choose a base branch
from
Open

Umstieg auf Glossaries-Package #134

wants to merge 20 commits into from

Conversation

ZeroPointMax
Copy link
Collaborator

siehe #131, closes #131

Sag mal, ich hab das Issue am 11.04. eröffnet, mir ist echt gar nicht mehr zu helfen...

Ich denke aber, ich hab jetzt alles in einem funktionierenden Zustand?

@ZeroPointMax ZeroPointMax added the enhancement New feature or request label May 12, 2022
@ZeroPointMax ZeroPointMax self-assigned this May 12, 2022
Copy link
Collaborator

@photovoltex photovoltex left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ich würde die ganzen Prefixe (ac:*, jm:*, gls:*) aus der Doku und den Beispielen rausnehmen. Verwirrt meiner Meinung mehr als das es hilft. So kann mit den Prefixen z.B. auch DHGE einmal als Glossar und Abk. Entry vorkommen was ja wahrscheinlich nicht gewollte ist? (Also als zwei Glossareinträge mit jeweils dem selben short aber anderem Prefix)

.vscode/tex_snippets.code-snippets Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
.vscode/tex_snippets.code-snippets Outdated Show resolved Hide resolved
build/template.tex Outdated Show resolved Hide resolved
build/tests/config.tex Outdated Show resolved Hide resolved
config.tex Outdated Show resolved Hide resolved
@ZeroPointMax ZeroPointMax requested a review from photovoltex May 17, 2022 07:28
main.tex Outdated Show resolved Hide resolved
Co-authored-by: Felix Prillwitz <[email protected]>
@photovoltex photovoltex self-requested a review May 18, 2022 08:33
abk.tex Outdated Show resolved Hide resolved
Copy link
Collaborator

@photovoltex photovoltex left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hattte noch überlegt bei Beispielen wo Generic Names verwendet werden z.B. \gls{key}, dass diese evtl noch gegen z.B. \gls{<key>} getauscht werden? Also das der Generic Name vorgehoben wird... sollte aber auch so reichen wahrscheinlich.

Ansonsten sieht es gut aus. Vielen Dank :)

@ZeroPointMax
Copy link
Collaborator Author

Hattte noch überlegt bei Beispielen wo Generic Names verwendet werden z.B. \gls{key}, dass diese evtl noch gegen z.B. \gls{<key>} getauscht werden? Also das der Generic Name vorgehoben wird... sollte aber auch so reichen wahrscheinlich.

Ansonsten sieht es gut aus. Vielen Dank :)

Ah über dieses Thema hadere ich auch jedes Mal. Dann gibt es Nutzer, die die spitzen Klammern evtl. nciht wegmachen. Das wäre dann zu dokumentieren

@ZeroPointMax
Copy link
Collaborator Author

@RvNovae ping

@RvNovae
Copy link
Owner

RvNovae commented May 20, 2022

grafik
Gibts einen Grund, warum der Margin rechts im Glossar so groß ist?

@photovoltex
Copy link
Collaborator

grafik Gibts einen Grund, warum der Margin rechts im Glossar so groß ist?

Das ist mir auch aufgefallen als ich es getestet habe, deswegen hatte ich diese Zeile noch mit vorgeschlagen.

\setlength{\glsdescwidth}{0.65\textwidth}

Allerdings ist es halt keine richtig dynamische Lösung :/

@ZeroPointMax
Copy link
Collaborator Author

Das ist mir auch aufgefallen als ich es getestet habe, deswegen hatte ich diese Zeile noch mit vorgeschlagen.

\setlength{\glsdescwidth}{0.65\textwidth}

Allerdings ist es halt keine richtig dynamische Lösung :/

Hatte ich das nicht sogar gemergt? :O

@ZeroPointMax
Copy link
Collaborator Author

image
Wahrscheinlich, damit sowas hier nicht passiert

@photovoltex photovoltex self-requested a review May 23, 2022 13:59
Copy link
Collaborator

@photovoltex photovoltex left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Siehe Kommentar


% Glossar Setup
\renewcommand{\glsnamefont}[1]{\textbf{#1}}
\setlength{\glsdescwidth}{0.65\textwidth}
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hier Loesung finden.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ZeroPointMax ja das hattest du gemerged... Die Loesung ist auch teilweise okay aber halt nicht voll dynamisch... Evtl muss man das in der Config z.B. anbieten zu setzen.
Ausser natuerlich wir finden noch eine Loesung welche dieses Verhalten automatisch loest.

README.md Outdated Show resolved Hide resolved
@RvNovae
Copy link
Owner

RvNovae commented Jun 13, 2022

Heyo, wie ist denn der Stand hier?
Wollen wir das erstmal undynamisch machen?

Co-authored-by: Felix Prillwitz <[email protected]>
@ZeroPointMax
Copy link
Collaborator Author

Heyo, wie ist denn der Stand hier? Wollen wir das erstmal undynamisch machen?

Ich habe hier leider noch keine Erleuchtung gefunden... Wir haben halt das Problem, dass man recht schnell in eine overfull hbox reinrennt, wenn der Eintrags"titel" zu lang wird.
Meinentwegen können wir das auch in die config.tex auslagern, ich finds halt trotzdem iwi doof 😕

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

Successfully merging this pull request may close these issues.

Ersetzen von Acro durch Glossaries
3 participants