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

feat: Set background colour of div that contains the EBD (off-white) #188

Merged
merged 3 commits into from
Jan 23, 2025

Conversation

hf-kklein
Copy link
Contributor

@hf-kklein hf-kklein commented Jan 23, 2025

to the same as the SVG background

@OLILHR das sieth aber irgendwie mist aus :-( dann machen wir den SVG hintergrund in off-white sobald die farbe feststeht.

{193702B0-DEEE-4017-AF65-779C8D3011B5}

Copy link

Azure Static Web Apps: Your stage site is ready! Visit it here: https://white-water-047c71203-188.westeurope.5.azurestaticapps.net

@hf-kklein hf-kklein marked this pull request as draft January 23, 2025 07:41
@OLILHR
Copy link
Member

OLILHR commented Jan 23, 2025

@OLILHR das sieth aber irgendwie mist aus :-( dann machen wir den SVG hintergrund in off-white sobald die farbe feststeht.

was sieht nicht gut aus? Das blau mit der watermark?
ich finde das einfach zu unruhig; die watermark ist viel zu penetrant.

also entweder ja - irgendein off-white oder vielleicht eine schwachere Variante von dem "Blau"

weiß nicht was Annika sagt. Der Kontrast zwischen Watermark und Background muss auf jeden Fall kleiner werden.

image image

was sagt @emmastoeck?

@emmastoeck
Copy link

Aus meiner Sicht ist das Wasserzeichen auch zu prominent in der Version. Das Problem ist dass das Auge nicht weiß, wo der Inhalt ist, der relevant ist (ebds) und, welcher Inhalt eigentlich irrelevant ist für den Benutzer (Wasserzeichen). Zusätzlich verschwindet unsere Fußleiste gänzlich.. An sich aber eine modernere Version als es off-white zu machen. Aber vielleicht mit einem helleren Blauton?

@hf-kklein
Copy link
Contributor Author

Mittlerweile bin ich gedanklich bei off white background hier und in der SVG. Mit ganz weißem watermark

@hf-kklein hf-kklein changed the title feat: Set background colour of div that contains the EBD feat: Set background colour of div that contains the EBD (off-white) Jan 23, 2025
@hf-kklein
Copy link
Contributor Author

ok, next try: wir setzen den hintergrund, wie in teams besprochen auf off-white.

https://teams.microsoft.com/l/message/19:meeting_OTk3YTQ1MTktN2Y5Yy00MzJlLTlkMzEtZTc2YzYwNzhiY2Vj@thread.v2/1737621754378?context=%7B%22contextType%22%3A%22chat%22%7D

die EBD-SVGs ziehen gleich nach: Hochfrequenz/rebdhuhn#337

@hf-kklein hf-kklein requested a review from emmastoeck January 23, 2025 09:15
@hf-kklein hf-kklein marked this pull request as ready for review January 23, 2025 09:15
Copy link

Azure Static Web Apps: Your stage site is ready! Visit it here: https://white-water-047c71203-188.westeurope.5.azurestaticapps.net

1 similar comment
Copy link

Azure Static Web Apps: Your stage site is ready! Visit it here: https://white-water-047c71203-188.westeurope.5.azurestaticapps.net

@hf-kklein hf-kklein merged commit 13ff5f5 into main Jan 23, 2025
8 checks passed
@hf-kklein hf-kklein deleted the bg-colour branch January 23, 2025 09:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants