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

95d1eb8 hat falschlicherweise 1pt zu -0.75 geändert #23

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

castilma
Copy link

@castilma castilma commented Apr 4, 2023

Der Vorzeichenwechsel hat zu sich überlappende Ränder geführt. Hier wird das Vorzeichen korrigiert.

Der Vorzeichenwechsel hat zu sich überlappende Ränder geführt. Hier wird das Vorzeichen korrigiert.
@kellertuer
Copy link
Owner

Oh, im Changelog sah es nach einem Vorzeichen aus, magst du einmal posten, dass es mit diesen Vorzeichen korrekt ist, so als Test, der hier dann dokumentiert ist? Das wäre super.

@castilma
Copy link
Author

castilma commented Apr 4, 2023

Ja. Hier gibt es noch das Problem mit dem Wechsel zu scrlayer-scrpage. Aber wenn man den commit auf 5368e8b cherry-picked, hat man das Problem auch gelöst.
front
rear

@kellertuer
Copy link
Owner

In dem Screenshot sieht die Schnittmarke (das gestrichelte) aber nicht richtig aus, da schneidet man sich ja einen Teil des Antwort-Headers weg.

@castilma
Copy link
Author

castilma commented Apr 4, 2023

Ja, das ist das problem mit scrlayer-scrpage. Wenn man diesen Patch auf 5368e8b anwendet, sieht es gut aus. Wenn du einen branch auf dem commit machst, erstelle ich den Pullrequest dagegen.

@kellertuer
Copy link
Owner

hm, magst du dir das Forken und Fixen? Es eilt ja auch nicht, es war schließlich 10 Jahre falsch.

@castilma
Copy link
Author

castilma commented Apr 4, 2023

Ja, hier: https://github.com/castilma/Kartei/tree/borderfix-old
Dort sieht es so aus.
front-good
rear-good

@kellertuer
Copy link
Owner

Das sieht doch gut aus :)

Ich weiß nur noch nicht ganz, wie man das nun hier unterbringt, weil auf dem master mit scrlayer-scrpage ist es ja damit noch nicht gefixt? Ich überleg mal. Vielleicht wirklich mit nem scrpage2-branch.

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.

2 participants