Resolve "Serialisierung damage und del"

Closes #50 (closed)

assigned to @zeterberg

added 1 commit

• 6ad346e2 - fix(pc): make typing a choice

Compare with previous version

added 2 commits

• f3676471 - feat(word_boundary_separator): serialize pc@type=word_boundary_separator as full stop
• f5ce1636 - fix(id-generation): make id-generation according to element

Compare with previous version

added 1 commit

• 8d4ecdd7 - feat(non-identifiable-signs): serialize pc for non identifiable signs

Compare with previous version

added 1 commit

• 7b7ffdbd - feat(tei-elemente): allow next/prev attributes

Compare with previous version

added 1 commit

• 42fcde5b - feat(tei-elements): serialize next/prev attributes

Compare with previous version

added 1 commit

• 1a94e68d - fix(tei-elements): serialization for author-mode

Compare with previous version

added 1 commit

• 47b15ac4 - fix(generate-id): make id more readable

Compare with previous version

Ich habe das alles stichprobenartig getestet, hat soweit funktioniert. Allerdings werden alle Siglen (".", "x", "...", hochgestellte Fragezeichen, Klammern...) auch kursiv und in der normalen Schriftfarbe dargestellt. Vielleicht liegt es an meinem Browser?

requested review from @tillmann.doenicke

marked this merge request as ready

added 1 commit

• eeec6343 - add new sample

Compare with previous version

Zu "Klammern zwischen zwei gleichen tei-Elementen entfernen" (#50 (closed)): Man könnte das auch nur über CSS lösen, indem man nach den ganzen Regeln für .damage[data-degree=low] und .damage[data-degree=medium] diesen Block einfügt:

/* damage + damage */
.damage[data-degree=low] + .damage[data-degree=low]::before,
.damage[data-degree=medium] + .damage[data-degree=medium]::before,
.damage[data-degree=low]:has(+ .damage[data-degree=low])::after,
.damage[data-degree=medium]:has(+ .damage[data-degree=medium])::after {
    content: "";
}
/* damage + (seg > damage) */
.damage[data-degree=low] + .seg > .damage:first-child[data-degree=low]::before,
.damage[data-degree=medium] + .seg > .damage:first-child[data-degree=medium]::before,
.damage[data-degree=low]:has(+ .seg > .damage:first-child[data-degree=low])::after,
.damage[data-degree=medium]:has(+ .seg > .damage:first-child[data-degree=medium])::after {
    content: "";
}
/* (seg > damage) + damage */
.seg:has(> .damage:last-child[data-degree=low]) + .damage[data-degree=low]::before,
.seg:has(> .damage:last-child[data-degree=medium]) + .damage[data-degree=medium]::before,
.seg:has(+ .damage[data-degree=low]) > .damage:last-child[data-degree=low]::after,
.seg:has(+ .damage[data-degree=medium]) > .damage:last-child[data-degree=medium]::after {
    content: "";
}
/* (seg > damage) + (seg > damage) */
.seg:has(> .damage:last-child[data-degree=low]) + .seg > .damage:first-child[data-degree=low]::before,
.seg:has(> .damage:last-child[data-degree=medium]) + .seg > .damage:first-child[data-degree=medium]::before,
.seg:has(+ .seg > .damage:first-child[data-degree=low]) > .damage:last-child[data-degree=low]::after,
.seg:has(+ .seg > .damage:first-child[data-degree=medium]) > .damage:last-child[data-degree=medium]::after {
    content: "";
}

Das ist jetzt auch nicht das Schickste (und ich bin mir nicht sicher, ob das alle Fälle abdeckt), aber ist das nicht womöglich einfacher (und weniger fehleranfällig), als diese neuen prev- und next-Attribute im XML einfügen zu müssen?

resolved all threads

Statt chaining="next prev" wäre ich sehr dafür, class="next prev" oder data-chaining="next prev" zu verwenden, da chaining="next prev" m. W. kein valides HTML ist.

class="next prev" ist besser, weil man das im CSS über .next und .prev einfacher überprüfen kann. Man könnte auch class="chaining-next chaining-prev" verwenden, wenn man unbedingt das chaining behalten möchte.

Alles klar, klingt sehr sinnvoll!

added 1 commit

• 16044ddd - fix(attributes): make chaining a class instead of a unique attribute

Compare with previous version

added 1 commit

• 1cddadf0 - fix(html-shared-templates): resolve merge conflicts

Compare with previous version

added 6 commits

• 1cddadf0...437b2aa6 - 5 commits from branch master
• e8240e21 - Merge branch 'master' into '50-serialisierung-damage-und-del'

Compare with previous version

@tillmann.doenicke ich habe die Merge-Conflicts resolved, kannst jetzt nochmal reviewen (class="next prev" ist eingebaut).

Sieht für mich so aus, als müsste das CSS noch angepasst werden.

Ach ja, stimmt. Einfach "chaining" durch "class" ersetzen?!

Klassen erreicht man über ., also müsste bspw. .tei.add:not([chaining=prev]) zu .tei.add:not(.prev) werden, denke ich.

gut, hab ich implementiert. Hat lokal bei mir auch funktioniert !50 (c5522629)

added 1 commit

• c5522629 - fix(next-prev): add next and prev classes to CSS

Compare with previous version

Wofür gibt es diese Datei? Ist es relevant, dass hier noch :not([prev]) statt :not(.prev) steht?

Diese Datei steuert den Autor-Modus in Oxygen. Die Stylesheets beziehen sich also auf die XML-Dateien und nicht auf HTML.

approved this merge request

added 14 commits

• c5522629...e5ce9396 - 13 commits from branch master
• c63ee5e7 - Merge branch 'master' into '50-serialisierung-damage-und-del'

Compare with previous version

merged

mentioned in commit 085575e5