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

Typographic fixes #11

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

Typographic fixes #11

wants to merge 3 commits into from

Conversation

m-g-r
Copy link

@m-g-r m-g-r commented Feb 23, 2023

Hi Dominik, wonderful program thanks a lot! Got my class A license on Monday. :)

Here are some fixes to typographic glitches split in two commits. The first are obvious ones.

The second commit is just to fix ugly wraps after the slash in lambda/2 expressions as in this screenshot:
Screenshot_20230125-203411_1_cr

(Sorry for the ugly pen drawing. I did this while learning and that was the only drawing function of the program. hehe)

Just added a third commit...

The link to the original XML of the questions in the readme seems to be outdated. http://oliver-saal.de/software/afutrainer/download.php is not available anymore. It seems to reside here now http://iris-tondok.de/software/afutrainer/download.php

Incidentally, it seems the source XML by Oliver Saal has the same typos. Would you prefer the errors to be fixed in the XML instead?

Cheers, Max

mgr added 3 commits February 23, 2023 17:42
…hone)

so put these (and other) lambda expressions in nowrap spans

Needs testing but as html is used in other cases, this ought to work. :)
(Could also use the Unicode "Zero Width No-Break Space" 
but that might lead to other problems.)
@m-g-r
Copy link
Author

m-g-r commented Feb 23, 2023

Da die Fragen aus dem AFUTrainer von Oliver Saal kommen, habe ich es in dem XML auch gefixed (und ihm per Mail geschickt). Anbei, falls das für den Funktrainier auch sinnvoll sein sollte. Gruß Max

questions-xml.patch.txt
questions.zip

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