Skip to content

Conversation

olestan
Copy link
Member

@olestan olestan commented Sep 16, 2026

Change

Added responsive styling using plain CSS and updated the layout and styling of the header, weather cards, details dialog, and footer. Some components were also reorganized into dedicated folders.

Closes #14

Verification

  • Used browser inspect/dev tools to test the responsive layout at different pixel widths.
  • Checked that cards, controls, header, footer, and details dialog adapt correctly without unwanted overflow.

Course requirements and AI use

This change addresses the responsive design requirement using plain CSS, primarily with Flexbox and Grid.

AI was used for debugging, suggestions for reasonable design tokens, and help when getting stuck. The suggestions were reviewed and the implementation was manually verified using browser dev tools.

@olestan olestan requested a review from hasanmo September 16, 2026 14:01
@olestan olestan requested a review from a team as a code owner September 16, 2026 14:01
@hasanmo
Copy link
Member

hasanmo commented Sep 16, 2026

Den andre PR-en bør merges til dev før denne PR-en for responsivt design. Design-branchen må deretter hente siste versjon av dev, slik at CSS- og komponentendringene bygges på de nye modellene og aktivitetsdataene. Når den andre PR er godkjent og merget, kan jeg merge dev i responsive design branchen for å sjekke videre. hva synes dere ? @olestan @oliveror @oliverdr

Verification

@olestan olestan marked this pull request as draft September 17, 2026 10:40
@olestan
Copy link
Member Author

olestan commented Sep 17, 2026

Til senere hadde det vært bra om header, footer og detailsoverlay ble refaktorisert til komponenter under /components, men jeg har for dette issuet holdt meg til strukturen slikk den er i App.tsx

@olestan olestan marked this pull request as ready for review September 17, 2026 11:35
@hasanmo hasanmo self-requested a review September 17, 2026 21:38
Copy link
Member

@hasanmo hasanmo left a comment

Choose a reason for hiding this comment

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

Well done, short time left, so we can merge and test and continue on left issues

@hasanmo hasanmo merged commit e57f698 into dev Sep 17, 2026
Sign in to join this conversation on GitHub.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants