feat: /hire page, drop LinkedIn, About location -> SDU · Denmark#3
Merged
Conversation
…ct reveal - ContactSection: section was invisible because `.reveal` elements never got the `.in` class — added an IntersectionObserver matching AboutSection's pattern. Also fixed the typo'd email (altan.esmer -> esmeraltan). - Nav: hire-me CTA now points to new /hire page instead of #contact. - New /hire page (static) with availability list, mailto CTA, link to contact form, back link. - Footer: removed twitter and linkedin entries (no accounts); added direct email link instead. - AboutSection: 'SDU · Cyprus' -> 'SDU · Denmark', and code-bio `based: 'Cyprus / EU'` -> 'Denmark / EU'. - HeroSection: terminal `"Cyprus / EU"` and 'based in Cyprus' both switched to Denmark. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
AltanEsmer
force-pushed
the
claude/lucid-clarke-945a8b
branch
from
679630e to
7442184
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
/hirepagemailto:CTA, link back homeFooterandContactSectionsocial rows (no account); also drop the never-set Twitter expectation by leaving only GitHub + EmailContactSection: drop duplicateid="contact"(already on the wrapper div), brighten the subtitle for visibility, point Email toesmeraltan@gmail.comAboutSection: change the 2021 timeline org from placeholderUniversitytoSDU · DenmarkTest plan
npm run lint— cleannpx tsc --noEmit— cleannpm run build—/hireshown in route table, static export succeedsnpm run dev, visit/: Contact form visible, Footer shows only GitHub + Email, About 2021 entry reads "SDU · Denmark", nav "Hire me" navigates to/hire/hirepage renders with intro, list, mailto CTA, "Back home" link🤖 Generated with Claude Code