Skip to content

Add tjenester page and refresh homepage services section#95

Open
thehoegh wants to merge 7 commits into
mainfrom
feat/tjenester-page
Open

Add tjenester page and refresh homepage services section#95
thehoegh wants to merge 7 commits into
mainfrom
feat/tjenester-page

Conversation

@thehoegh

Copy link
Copy Markdown
Member

Summary

  • Add a dedicated /tjenester page with three anchored service areas: partnerskap, kompetanse, and produktfilosofi.
  • Refresh the homepage with an OffsetProseSection intro, enabled services cards linking to /tjenester, and a rotating client logo band sourced from project content.
  • Improve shared section/layout behavior with optional accent headings, conditional subheadings, and anchor scroll offset below the sticky header.

Test plan

  • Open / and verify the Team section, Tjenester subheading, logo band rotation, and service card links.
  • Confirm logo band shows 3 logos on mobile and 5 on desktop, with smooth swaps and links to /prosjekter/{id}.
  • Open /tjenester and verify hero copy, three sections, anchor navigation, and contact block.
  • Check contact section copy on homepage and tjenester page.
  • Run pnpm astro:check.

Made with Cursor

thehoegh and others added 3 commits June 16, 2026 00:06
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
@netlify

netlify Bot commented Jun 16, 2026

Copy link
Copy Markdown

Deploy Preview for kynd ready!

Name Link
🔨 Latest commit 127b9a2
🔍 Latest deploy log https://app.netlify.com/projects/kynd/deploys/6a324c844ab14a00086f3c57
😎 Deploy Preview https://deploy-preview-95--kynd.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

Comment thread src/components/ClientLogoBand.astro Outdated
Comment thread src/components/ClientLogoBand.astro Outdated
thehoegh and others added 3 commits June 16, 2026 13:11
Rotation was skipped whenever logo count was at most visibleCount (5), but
only three slots are shown on small viewports—so four or five clients never
rotated on mobile. Gate swaps on actually visible slots instead.

Co-authored-by: Cursor <cursoragent@cursor.com>
Track a stable client key on each slot and skip duplicates during initial
pick and rotation, including when multiple projects share one customer.

Co-authored-by: Cursor <cursoragent@cursor.com>
Rotation only updated visible slots on narrow viewports but excluded
hidden slots from the occupied-client set, so widening to desktop could
reveal duplicate customers.

Co-authored-by: Cursor <cursoragent@cursor.com>

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 127b9a2. Configure here.

const slot = activeSlots[slotIndex % activeSlots.length];
if (!slot) return;

const occupiedClients = getVisibleClients(slots);

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Hidden slots block mobile rotation

Medium Severity

On viewports that hide logo slots with CSS, the rotation interval still builds occupiedClients from every .logo-slot, including display: none entries. Clients shown only in hidden slots stay reserved, so the three visible slots cannot rotate through the full client list and takeNext may fall back to a logo whose client is already visible, producing duplicate logos in the band.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 127b9a2. Configure here.

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.

1 participant