Skip to content

Commit f6b3328

Browse files
sametbasbugclaude
andcommitted
Let the navigation lock count five items instead of four
The İletişim link made the mobile navigation five items and the browser regression suite still expected four. It failed in CI, so the deploy step was skipped and nothing shipped. It passed locally for a bad reason: browser:test reads the built dist/ and I ran it standalone, against a build made before the header changed. The suite was green about the old markup. Only `npm run build` — which builds first — actually tests the change. The count itself is a clamp, not the rule. The checks under it are the ones with teeth: equal flex basis, no overlap, nothing clipped at 360px. They pass with five. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
1 parent 3fae35e commit f6b3328

1 file changed

Lines changed: 4 additions & 1 deletion

File tree

‎scripts/orbit-browser-tests.mjs‎

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -391,7 +391,10 @@ if (errors.length === 0) {
391391
check(await page.locator('.header-mobile-search').isVisible(), `${label}: mobil arama erişimi görünür değil.`);
392392
check(layout.navPosition === 'fixed', `${label}: mobil alt navigasyon fixed değil.`);
393393
check(layout.nav.x >= 0 && layout.nav.right <= layout.innerWidth && layout.nav.bottom <= viewport.height, `${label}: mobil alt navigasyon kırpılıyor.`);
394-
check(layout.navLinks.length === 4, `${label}: mobil navigasyonda dört temel öğe yok.`);
394+
/* Beş oldu: İletişim menüye eklendi. Sayının kendisi bir kural değil,
395+
bir kelepçe — altındaki geometri kontrolleri (eşit flex, çakışma
396+
yok, kırpılma yok) beşin 360 piksele sığıp sığmadığını söylüyor. */
397+
check(layout.navLinks.length === 5, `${label}: mobil navigasyonda beş temel öğe yok.`);
395398
const mobileNavText = (await page.locator('.primary-nav').textContent()) || '';
396399
check(!mobileNavText.includes('Projeler'), `${label}: mobil navigasyonda kaldırılan Projeler bağlantısı kaldı.`);
397400
check(mobileNavText.includes('Konular'), `${label}: mobil navigasyonda Konular bağlantısı yok.`);

0 commit comments

Comments
 (0)