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

Update layout on Cases page #86

Open
wants to merge 8 commits into
base: gh-pages
Choose a base branch
from
Open

Conversation

Arth101
Copy link
Collaborator

@Arth101 Arth101 commented Jan 25, 2023

Opdateret layout til vores Cases side.

Live test version: https://deploy-preview-86--zealous-fermat-77bc7b.netlify.app/cases/

Billederne er primært fra unsplash, men jeg har også fundet nogen fra de forskellige kunder som kunne være nice at bruge (hvis vi må).

Ting jeg er opmærksom på:

  • Tekster er lidt iffy. De skal nok opdateres med nogle gode teaser/headlines.
  • Billeder er relaterede til case, men ikke fra opgaverne/kunderne da jeg ikke ved om vi må bruge dem (pånær det fra KBH billeder, som der ikke er rettigheder på).
  • Logo til K&M/nornorm og KBH stadarkiv vil jeg opdatere.
  • "links" listen til hver case er jeg ikke så glad for, men kan ikke lige finde en god måde at vise de links som er relateret til kunden/opgaven.

Input er velkommen 🙂

Copy link
Collaborator

@haid0109 haid0109 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ingen indvendinger til det nye design, har highlighted grammar stuff som var der fra før.

but good stuff 👍

cases/index.html Outdated Show resolved Hide resolved
cases/index.html Outdated Show resolved Hide resolved
cases/index.html Outdated Show resolved Hide resolved
@hypesystem
Copy link
Contributor

hypesystem commented Jan 26, 2023

Her bliver quotes sgu lidt squished:
image

Og jeg synes layoutet breaker lidt her - måske kan det løses med ny introtekst + flytte teksten der nu ned i summary? skal jeg give det et skud?:
image

@hypesystem
Copy link
Contributor

I cases med ét quote er det også lidt underligt at de er constrained til halv bredde - svært at læse her:
image

@hypesystem
Copy link
Contributor

Overall notes:

  • meget improved layout! generelt stor fan af retningen
  • jeg vil gerne have quotes mere i fokus - kunne godt lide at de var interspersed før for at det ikke bliver en wall of quotes eller en wall of summary - begge bliver lidt voldsomme alene. Så in service of content ville det være nice at blande dem eller i hvert fald breake det lidt op.
  • linkse bliver lidt nemt overset - måske kunne vi gøre dem til en art knapper under billedet eller noget andet sjovt for at give det lidt flavor?
  • smånoter i screenshots ovenfor

@Arth101
Copy link
Collaborator Author

Arth101 commented Jan 27, 2023

@hypesystem

Her bliver quotes sgu lidt squished

Ja, layoutet er nok mest udtænkt fra at der er flere/mange quotes, men i virkeligheden ville det muligvis virke bedre med mindre quotes generelt tror jeg. Jeg tror det ville være en fordel at have en quote pr kunde, så en case er mere overskueligt generelt.

Og jeg synes layoutet breaker lidt her - måske kan det løses med ny introtekst + flytte teksten der nu ned i summary? skal jeg give det et skud?

Ja det er lidt langt og er bare kopieret fra summary anyway. Hvis du kan skrive en god introtekst så ville det være awesome :)

@ITViking
Copy link
Contributor

Mine øjne keder sig lidt fordi der overgangene er lidt kedelige. Jeg tror heller ikke at det hjælper at teksten ikke står i særlig stærk kontrast til baggrunden.

Jeg synes også meget at det er en wall of text hele vejen ned. Jeg kunne ønske mig at det var mere over i retning af et nemt overblik over hvem/hvad vi har lavet, med måske et qoute samt en stærk og koncis linje om vores impact.
Jeg kunne næsten have lyst til at genbruge noget fra team-siden hvor hver boks kunne indeholde et logo, kundens industri, en koncis linje om impact og et link til at læse mere. Hvis man så holder musen henover så kunne den fade til en lidt mere uddybende teaser eller et quote.

Other than that, synes jeg et en stærk forbedring af det vi har og jeg synes der er nogle rigtig gode forklaringer.

@Arth101
Copy link
Collaborator Author

Arth101 commented Feb 1, 2023

Opdateret med nyt layout. Det mangler dog en god løsning til at vise "body" tekst til hver case.
Uploading Screenshot from 2023-02-01 15-39-54.png…

@hypesystem
Copy link
Contributor

@Arth101 dit seneste screenshot virker ikke

@Arth101
Copy link
Collaborator Author

Arth101 commented Feb 13, 2023

@Arth101 dit seneste screenshot virker ikke

Jeg har lavet et nyt så :)
image

@Arth101 Arth101 force-pushed the update-cases-page-layout branch from a00dc4a to 49beba8 Compare September 24, 2024 08:08
…ors and keep the UI more like a list of cases instead of having boxes in a grid
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.

4 participants