search

infi-nl / the-infi-way

How we like to build software
https://way.infi.nl
Other
9 stars 8 forks source link

Implement new design #28

Closed LucaScorpion closed 2 years ago

LucaScorpion commented 2 years ago

Part of #6

LucaScorpion commented 2 years ago

@jeroenheijmans Dank voor de feedback! :raised_hands:

  1. [Voorstel: design overnemn] De footer mag ook wel naar dark-mode i.p.v. oranje achtergrond.

Eens, hier zal ik naar gaan kijken!

  1. [Voorstel: versie uit PR behouden] Het design bevat (zeer) afgeronde hoeken, maar ik vind de scherpe (of anders in elk geval bijna helemaal scherpe) hoeken beter passen. Zeker op mobile view is de PR versie een stuk beter vind ik.

Top :) Ik vond zelf de afgeronde hoeken ook wel wat hebben, maar dit is meer in lijn met de Infi website stijl, en werkt idd beter op mobile.

  1. [Voorstel: design overnemen]: Kleuren vanaf "gebruikte tech" zijn in het design wat minder donker en wat meer saturated. Zowel in de ol als in de achtergond van de rondjes.

Good point, ik zal zorgen dat ik alle kleuren even goed overneem uit het design.

  1. [Voorstel: design overnemen]: De gekleurde circles hebben wat meer padding (dus tussen het icon en de rand van de cirkel).

Yeah klopt, voordat we dit te veel gaan tweaken wilde ik nog even wachten op de assets van de huisdesigner :grin: Maar kan denk ik voor nu sws de bestaande icons iets kleiner maken/padding tweaken.

  1. [Nog te overwegen]: Het design heeft een meer zwart/wit (#212121 en #121212) dark mode, terwijl de dark mode tinten uit de PR (#21212c en #12121c) wat meer blauw erin lijken te hebben. De design-versie spreekt me persoonlijk ever-so-slightly meer aan, maar ben ook totaal niet tegen de meer blauwige versie als de rest dat een stuk fraaier vindt?

Hier twijfelde ik erg over. Ik heb uiteindelijk de blauwige tinten gekozen op basis van de donkere vlakken op de Infi site zodat het daarmee matcht, maar vind persoonlijk ook het zwart uit het design net wat fraaier. Ik zal het voor nu veranderen naar het zwart, en dan horen we wel wat de rest vindt :)

  1. [Voorstel: andere PR doen]: De subtiele icons op achtergrond laten repeaten uit het design voegt wel wat toe, maar zou ik denk ik in een aparte PR laten, zodat we deze PR sneller af kunnen ronden?

Agreed, ook deze had ik nog niet gedaan omdat ik nog geen assets had ^^ Maar kan inderdaad prima in een andere pr.

LucaScorpion commented 2 years ago

Heb de punten verwerkt!

Ik heb voor de dark footer nu dezelfde grijze kleur als de topic blokken gebruikt ipv de kleur uit het design, die spreekt mij persoonlijk niet zo erg aan, en zo voelde het voor mij iets meer als een geheel.