Fix inconsistent Buy Me A Coffee button styling across all pages#155
Conversation
Agent-Logs-Url: https://github.com/DavidKRK/DavidKRK.github.io/sessions/8dd18ad4-6a79-4462-a55c-856d1697cc3a Co-authored-by: DavidKRK <136035295+DavidKRK@users.noreply.github.com>
|
|
There was a problem hiding this comment.
Pull request overview
This PR standardizes the Buy Me A Coffee button styling in the site footer so it matches the already-updated pattern used on other pages (responsive sizing via em + consistent utility classes).
Changes:
- Updated the Buy Me A Coffee
<a>to includetext-2xl md:text-3xlsizing utilities. - Switched the button image from a fixed
height: 30pxto responsiveheight: 1em(withvertical-align: middle). - Normalized the
shop.htmlmarkup to the single-line pattern used elsewhere.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 5 comments.
| File | Description |
|---|---|
| contact.html | Updates Buy Me A Coffee button to responsive sizing + consistent classes. |
| index.html | Updates Buy Me A Coffee button to responsive sizing + consistent classes. |
| shop.html | Updates Buy Me A Coffee button styling and collapses markup to match other pages. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| <a href="https://www.buymeacoffee.com/davidkrk" target="_blank" aria-label="Buy Me A Coffee" class="social-icon"> | ||
| <img src="https://cdn.buymeacoffee.com/buttons/v2/default-yellow.png" alt="Buy Me A Coffee" style="height: 30px;"> | ||
| </a> | ||
| <a href="https://www.buymeacoffee.com/davidkrk" target="_blank" aria-label="Buy Me A Coffee" class="social-icon text-2xl md:text-3xl"><img src="https://cdn.buymeacoffee.com/buttons/v2/default-yellow.png" alt="Buy Me A Coffee" style="height: 1em; vertical-align: middle;"></a> |
There was a problem hiding this comment.
Links opened with target="_blank" should include rel="noopener noreferrer" to prevent the new page from being able to access window.opener (tabnabbing) and to avoid keeping a reference to the originating window. Please add the rel attribute here (and ideally to the other external links in the same footer for consistency).
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
…fault lang in bio Agent-Logs-Url: https://github.com/DavidKRK/DavidKRK.github.io/sessions/ed06d0b0-1a6f-4799-b542-c27ba4628374 Co-authored-by: DavidKRK <136035295+DavidKRK@users.noreply.github.com>
Agent-Logs-Url: https://github.com/DavidKRK/DavidKRK.github.io/sessions/ed06d0b0-1a6f-4799-b542-c27ba4628374 Co-authored-by: DavidKRK <136035295+DavidKRK@users.noreply.github.com>
Agent-Logs-Url: https://github.com/DavidKRK/DavidKRK.github.io/sessions/f7f3b998-8299-4d2d-bd9e-8af7365cfd1c Co-authored-by: DavidKRK <136035295+DavidKRK@users.noreply.github.com>
…tml and shop.html Agent-Logs-Url: https://github.com/DavidKRK/DavidKRK.github.io/sessions/52ab48c3-abad-4a48-b61d-5d60bbfe6b8e Co-authored-by: DavidKRK <136035295+DavidKRK@users.noreply.github.com>
lang-contenttitre (<h2 class="main-title">) dansindex.html(Accueil/Home/Inicio/Hasiera/الرئيسية/Головна)lang-contenttitre (<h2 class="main-title">) dansshop.html(Boutique/Shop/Tienda/Denda/المتجر/Магазин)