Skip to content

Add Salazar last name, X/Reddit socials, 3 new projects, expanded About Me, new favicon - #13

Merged
simihablo merged 3 commits into
mainfrom
copilot/add-salazar-to-website
Sep 6, 2026
Merged

Add Salazar last name, X/Reddit socials, 3 new projects, expanded About Me, new favicon#13
simihablo merged 3 commits into
mainfrom
copilot/add-salazar-to-website

Conversation

Copilot AI commented Jun 2, 2026

Copy link
Copy Markdown
Contributor

Personalizes and expands the portfolio site with updated identity info, social presence, and project coverage.

Changes

  • Identity: "Mericio" → "Mericio Salazar" across h1, <title>, meta author/og:title, and footer
  • Favicon: Replaced favicon.ico with favicon.png (GitHub avatar); added <link rel="icon"> in index.html
  • Social links: Added X (x.com/simihablo) and Reddit (reddit.com/user/MelodicClaim890/) buttons with icons in both the hero and contact sections
  • Projects: Added 3 entries — Kubernetes Home Lab, GCP Infrastructure Automation, Open Source DevRel
  • About Me: Expanded with multi-cloud context (GCP/AWS/Azure), tooling focus (Terraform, GitOps), and more personal detail

Copilot AI changed the title Add Salazar last name, social links, new projects, expanded About Me, new favicon Add Salazar last name, X/Reddit socials, 3 new projects, expanded About Me, new favicon Jun 2, 2026
Copilot AI requested a review from simihablo June 2, 2026 20:10
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@simihablo
simihablo marked this pull request as ready for review September 6, 2026 01:22
Copilot AI lite review requested due to automatic review settings September 6, 2026 01:22
@simihablo
simihablo merged commit 634455d into main Sep 6, 2026
2 checks passed
@simihablo
simihablo deleted the copilot/add-salazar-to-website branch September 6, 2026 01:23

Copilot AI 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.

🟡 Changes recommended

External links added via window.open(..., '_blank') should be changed to safe link semantics (e.g., rel="noopener noreferrer") to avoid reverse-tabnabbing.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Pull request overview

Updates the portfolio site’s branding and content to reflect an updated name, expanded “About Me”, additional project entries, new social links, and a new favicon reference in the document head.

Changes:

  • Updated displayed identity to “Mericio Salazar” and refreshed About Me copy.
  • Added X and Reddit social buttons (hero + contact) and added 3 new project entries.
  • Added favicon.png link in index.html and updated related meta tags.
File summaries
File Description
src/pages/Index.tsx Updates name display, expands About Me, adds new projects, and introduces X/Reddit buttons (plus layout tweaks).
index.html Updates title/meta identity fields and adds a PNG favicon link.
Review details

Suppressed comments (4)

src/pages/Index.tsx:4

  • The inline RedditIcon SVG is decorative (the button already has a text label). Mark it as such so screen readers don't announce it as an unlabeled graphic.
  <svg className={className} viewBox="0 0 20 20" fill="currentColor" xmlns="http://www.w3.org/2000/svg">

src/pages/Index.tsx:149

  • Same reverse-tabnabbing concern as above: prefer an anchor with rel="noopener noreferrer" instead of window.open(..., '_blank').
              variant="outline" 
              className="border-amber-400/70 text-amber-200 hover:bg-amber-300 hover:text-slate-950 transition-all duration-300"
              onClick={() => window.open('https://www.reddit.com/user/MelodicClaim890/', '_blank')}
            >
              <RedditIcon className="mr-2 h-4 w-4" />

src/pages/Index.tsx:316

  • Same reverse-tabnabbing concern: replace window.open(..., '_blank') with a link using target="_blank" rel="noopener noreferrer" (via <Button asChild>).
              variant="outline"
              className="border-slate-400 text-slate-200 hover:bg-slate-600 hover:text-white transition-all duration-300 hover:scale-105"
              onClick={() => window.open('https://x.com/simihablo', '_blank')}
            >
              <X className="mr-2 h-5 w-5" />

src/pages/Index.tsx:325

  • Same reverse-tabnabbing concern: replace window.open(..., '_blank') with a link using target="_blank" rel="noopener noreferrer" (via <Button asChild>).
              variant="outline"
              className="border-orange-600 text-orange-300 hover:bg-orange-600 hover:text-white transition-all duration-300 hover:scale-105"
              onClick={() => window.open('https://www.reddit.com/user/MelodicClaim890/', '_blank')}
            >
              <RedditIcon className="mr-2 h-5 w-5" />
  • Files reviewed: 2/3 changed files
  • Comments generated: 3
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread src/pages/Index.tsx
Comment on lines +134 to +142
<Button
size="lg"
variant="outline"
className="border-sky-400/70 text-sky-200 hover:bg-sky-300 hover:text-slate-950 transition-all duration-300"
onClick={() => window.open('https://x.com/simihablo', '_blank')}
>
<X className="mr-2 h-4 w-4" />
X
</Button>
Comment thread src/pages/Index.tsx
Comment on lines +1 to 9
import { Github, Linkedin, ArrowDown, X } from 'lucide-react';

const RedditIcon = ({ className }: { className?: string }) => (
<svg className={className} viewBox="0 0 20 20" fill="currentColor" xmlns="http://www.w3.org/2000/svg">
<circle cx="10" cy="10" r="10" fill="currentColor" fillOpacity="0"/>
<path d="M20 10C20 4.477 15.523 0 10 0S0 4.477 0 10c0 4.992 3.657 9.128 8.438 9.879V12.89h-2.54V10h2.54V7.797c0-2.506 1.492-3.89 3.777-3.89 1.094 0 2.238.195 2.238.195v2.46h-1.26c-1.243 0-1.63.771-1.63 1.562V10h2.773l-.443 2.89h-2.33v6.989C16.343 19.129 20 14.992 20 10z"/>
</svg>
);
import { Button } from '@/components/ui/button';
Comment thread src/pages/Index.tsx
<div className="max-w-6xl mx-auto text-center">
<p className="text-slate-300">
© 2025 Mericio • Infrastructure Engineer • Guatemala 🇬🇹
© 2025 Mericio Salazar • Infrastructure Engineer • Guatemala 🇬🇹
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.

3 participants