Hochfrequenz / rebdhuhn

Python library to convert scraped "Entscheidungsbaumdiagramm" tables into machine readable diagrams (e.g. UML)
GNU General Public License v3.0
0 stars 0 forks source link

🎨 Much-needed visual overhaul of SVG output files #272

Open OLILHR opened 1 month ago

OLILHR commented 1 month ago

not sure how far we can go in terms of customizability but here are a first few suggestions:


image
emmastoeck commented 1 month ago
OLILHR commented 1 month ago
  • Adjust padding, border, margin, line spacing.
  • Watermark: Other color than text boxes, too prominent
  • Question: What do we do with trees with branches that go into different directions (e.g. E_0595)?

https://github.com/Hochfrequenz/machine-readable_entscheidungsbaumdiagramme/blob/main/FV2504/E_0595.svg 😬

hf-kklein commented 4 weeks ago

@OLILHR magst du dieses "fix everything"-ticket vllt noch aufteilen in bewältigbare sub-tickets? also z.b. "limit aspect ratio to max. 4:1" und "use sans-serif font" usw. das nimmt glaube ich etwas den druck. wenn ich mir z.b. ein "change font" ticket assigne, fühle ich mich eher so als könnte ich das in endlicher zeit fixen, als wenn ich mir ein "improve entire svg"-ticket schnappe.

hf-kklein commented 4 weeks ago

Watermark: Other color than text boxes, too prominent

das war eine anforderung von "damals": https://github.com/Hochfrequenz/ebd_parser-backend/pull/191 weiß vllt @hf-aschloegl noch, wie prominent das sein sollte?

hf-kklein commented 4 weeks ago

Watermark: [...] too prominent

check out #279

OLILHR commented 4 weeks ago

@OLILHR magst du dieses "fix everything"-ticket vllt noch aufteilen in bewältigbare sub-tickets? also z.b. "limit aspect ratio to max. 4:1" und "use sans-serif font" usw. das nimmt glaube ich etwas den druck. wenn ich mir z.b. ein "change font" ticket assigne, fühle ich mich eher so als könnte ich das in endlicher zeit fixen, als wenn ich mir ein "improve entire svg"-ticket schnappe.

das ist auch als übergeordnetes Issue zu verstehen, was nur erst einmal als Ideen-Sammlung dient. Die genauen Anforderungen müssen wir erst noch festlegen. Anschließend werden diese in Tickets gegossen, denen sich z.B. @emmastoeck annehmen kann.

hf-aschloegl commented 4 weeks ago

Puh, ihr fragt Sachen. Glaube besondere Anforderungen zur Opacity gab es damals nicht. Die Anforderung im Allgemeinen kam von Thorsten, wenn ich mich richtig erinnere und das wichtigste war, dass man es nicht einfach wegschneiden kann und wenn jemand einen Screenshot macht oder es herunterlädt immer noch erkennbar ist, dass das von Hochfrequenz stammt.

OLILHR commented 4 weeks ago

Ich kann auch noch mal mit Thorsten Rücksprache halten. Dass das SVG HF-gebranded sein soll, ist denke ich klar, aber das sollte natürlich nicht auf Kosten von Usability/Accessibility geschehen.

hf-kklein commented 4 weeks ago

lass erstmal das ergebnis abwarten :) wie es dann auf der website aussieht.