CatoTH / antragsgruen

Motions, resolutions and amendments for political conventions
https://motion.tools
GNU Affero General Public License v3.0
94 stars 24 forks source link

Diffs mit Spielraum nach oben #137

Open joriki opened 8 years ago

joriki commented 8 years ago

Ich sammle hier mal suboptimale Diffs, die nicht falsch sind und nicht dringend geändert werden müssen aber wenn wir mal dazu kommen verbessert werden sollten.

https://bdk.antragsgruen.de/39/motion/118/amendment/196

joriki commented 8 years ago

https://bdk.antragsgruen.de/39/motion/118/amendment/289

Diesen beiden ist gemeinsam, dass sie besser aussehen würden, wenn erst die Einfügung und dann die Streichung käme. Das ist übrigens auch, wie es bei Eingeben des Änderungsantrags im Editor angezeigt wird, wenn man Text auswählt und überschreibt -- insofern wäre es vielleicht sinnvoll, allgemein diese Reihenfolge umzukehren und bei Ersetzungen immer zuerst die Einfügung und dann die Streichung zu zeigen -- damit kann man auch zusammenhängender lesen, wie der Text nach der Änderung lauten würde.

joriki commented 8 years ago

Hier ist am Ende unnötig "Bürger*innengesellschaft." gestrichen und eingefügt: https://bdk.antragsgruen.de/39/motion/144/amendment/297

CatoTH commented 8 years ago

Die meisten Probleme hier sollten nun behoben sein.

joriki commented 8 years ago

Ja, sieht jetzt gut aus. Was denkst zu der Frage, ob die Einfügung vor der Streichung stehen sollte?

joriki commented 8 years ago

Die doppelte "Bürger*innengesellschaft" ist inzwischen hierhin umgezogen: https://bdk.antragsgruen.de/39/motion/144/amendment/319

joriki commented 8 years ago

Hier ist "ist" unnötig gestrichen und eingefügt: https://bdk.antragsgruen.de/39/motion/133/amendment/171

CatoTH commented 8 years ago

Die doppelte "Bürger*innengesellschaft" ist inzwischen hierhin umgezogen: https://bdk.antragsgruen.de/39/motion/144/amendment/319

Aber der Fehler ist doch jetzt weg, oder?

Hier ist "ist" unnötig gestrichen und eingefügt: https://bdk.antragsgruen.de/39/motion/133/amendment/171

Ok das ist behoben - auch wenn es jetzt dafür ein expliziter Zeilenumbruch ist, der vorher der Auslöser war...

joriki commented 7 years ago

Hier ein Beispiel, wo das Diff ungünstig ist, weil pro Absatz verglichen wird:

https://antraege.gruene.de/bdk41/Wir_sorgen_fuer_bezahlbare_Wohnungen_und_lebenswerte_Kommunen_-12200/406

Bei der Gelegenheit bin ich mal die alten Beispiele durchgegangen:

https://bdk.antragsgruen.de/39/motion/118/amendment/196 Das ursprüngliche Problem ist gelöst, aber jetzt wird am Ende unnötig ein Punkt gestrichen und eingefügt.

https://bdk.antragsgruen.de/39/motion/133/amendment/171 unverändert -- "ist ," wird gestrichen und eingefügt (Du hattest geschrieben, es sei behoben)

https://bdk.antragsgruen.de/39/motion/144/amendment/319 Das ist jetzt in Ordnung.

Offen ist auch noch die Frage, ob die Einfügung vor der Streichung stehen sollte.

joriki commented 7 years ago

Hier wird "So fordern wir zum Beispiel" unnötig gestrichen und wieder eingefügt: https://antraege.gruene.de/bdk41/Wir_foerdern_eine_innovative_und_faire_Wirtschaft_-61698/1148

joriki commented 7 years ago

Hier wird ein sehr großer Bereich als gestrichen und wieder eingefügt dargestellt: https://antraege.gruene.de/bdk41/Wir_staerken_die_Demokratie_-64428/1610

joriki commented 7 years ago

Auch hier unnötige Streichung und Wiedereinfügung: https://antraege.gruene.de/bdk41/Wir_streiten_fuer_Toleranz_Vielfalt_und_Selbstbestimmung_-12397/2364

joriki commented 7 years ago

Ein etwas anderer Fall: Hier wird die richtige Menge gestrichen, aber nicht erkannt, dass ganze Sätze eingefügt wurden: https://antraege.gruene.de/bdk41/Welt_im_Blick_-29026/3100

CatoTH commented 7 years ago

@joriki Zum letzten Fall (https://antraege.gruene.de/bdk41/Welt_im_Blick_-29026/3100): da sehe ich nur eine Einfügung, gar keine Streichungen - ist das der richtige Link oder versteh ich die Meldung nicht richtig?

joriki commented 7 years ago

@CatoTH Sorry, "richtige Menge gestrichen" war falsch -- ich meinte, hier geht es nicht darum, dass zuviel gestrichen wird. Ja, es ist nur eine Einfügung. Das gedoppelte "Es geht um" sollte vorne eingefügt werden, zusammen mit dem Rest des Satzes, statt hinten im darauffolgenden Satz.

joriki commented 7 years ago

Hier wieder Einfügung von Gestrichenem (wohl wegen Änderung der Absatzstruktur): https://antraege.gruene.de/bdk41/Wir_staerken_die_Demokratie_-64428/913

joriki commented 7 years ago

Ebenso, Einfügung von Gestrichenem wegen Änderung der Absatzstruktur: https://antraege.gruene.de/bdk41/Welt_im_Blick_-29026/363

joriki commented 7 years ago

Ähnlich wie in https://github.com/CatoTH/antragsgruen/issues/137#issuecomment-303621117 wird das "Wir" isoliert am Anfang des nächsten Absatzes eingefügt, während das bestehende "Wir" mitten im eingefügten Absatz erscheint: https://antraege.gruene.de/bdk41/Wir_stehen_ein_fuer_Frieden_und_Menschenrechte_-59186/2807 (<-- hier hatte ich den falschen Link eingesetzt; jetzt ist es der richtige)

joriki commented 7 years ago

"Roma" wird gestrichen und eingefügt: https://antraege.gruene.de/bdk41/Wir_streiten_fuer_Toleranz_Vielfalt_und_Selbstbestimmung_-12397/2489

joriki commented 7 years ago

Noch ein Beispiel (aber jetzt haben wir auch glaub ich schon genug :-) https://antraege.gruene.de/bdk41/Wofuer-wir-Verantwortung-uebernehmen-wollen-17447/3276

CatoTH commented 7 years ago

@joriki nach den neuesten Commits in der 3.7er-Version ist der Diff soweit verbessert, dass die drei genannten Anträge so aussehen (würden):

bildschirmfoto 2017-06-17 um 17 51 43 bildschirmfoto 2017-06-17 um 17 51 51 bildschirmfoto 2017-06-17 um 17 51 55

Allerdings will ich die neue Version eher nicht mehr für die aktuelle BDK einspielen - das wäre mir dann doch zu sehr "Operation bei offenem Herzen"...