Skip to content
This repository was archived by the owner on Jul 7, 2026. It is now read-only.

fix(reservas): apply ClassLink colors to page title#194

Merged
marpisco merged 2 commits into
mainfrom
codex/update-page-title-styles-for-classlink
Jul 5, 2026
Merged

fix(reservas): apply ClassLink colors to page title#194
marpisco merged 2 commits into
mainfrom
codex/update-page-title-styles-for-classlink

Conversation

@marpisco

@marpisco marpisco commented Jul 5, 2026

Copy link
Copy Markdown
Owner

Motivation

  • Substituir o gradiente inline do título por uma classe CSS simples para alinhar o título com as cores do ClassLink, mantendo legibilidade em light/dark mode sem alterar o espaçamento do cabeçalho nem a descrição.

Description

  • Troquei o span com o gradiente inline por <span class="classlink-page-title">As Minhas Reservas</span> em reservas/index.php.
  • Adicionei regras CSS .classlink-page-title { color: var(--primary-color); font-weight: 700; } e [data-bs-theme="dark"] .classlink-page-title { color: var(--secondary-color); } no bloco <style> em reservas/index.php para usar as variáveis de tema existentes.
  • Mantive o h2 e a descrição Gerir e acompanhar as suas reservas de salas inalterados para preservar o espaçamento e o texto.

Testing

  • Executei php -l reservas/index.php com sucesso sem erros de sintaxe.
  • Executei git diff --check sem encontrar problemas.

Codex Task

Replaces the inline gradient on the reservations page title with a simple CSS class that uses the existing ClassLink theme colors.

This keeps the header spacing and description unchanged while preserving readable title colors in light and dark mode.

Tested with php -l reservas/index.php and git diff --check.

Co-authored-by: Codex <noreply@openai.com>
Copilot AI review requested due to automatic review settings July 5, 2026 03:00

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@github-actions

github-actions Bot commented Jul 5, 2026

Copy link
Copy Markdown

📝 Manual Update Required? It looks like you've modified core logic. Please check if the user manual needs a refresh! @marpisco

@marpisco

marpisco commented Jul 5, 2026

Copy link
Copy Markdown
Owner Author

@codex Change this to fit the rest of the titles like the module, nevermind the original prompt. Take the example of /reservar/index.php. Modify it to fit that font style (not centered, keep it justified to the left).

@chatgpt-codex-connector

Copy link
Copy Markdown

Summary

  • Updated the reservations page title to use the same lightweight module heading style as /reservar/index.php, while keeping it left-aligned in the existing header layout. reservas/index.phpL249-L252
  • Removed the custom .classlink-page-title styling so the title no longer uses the previous bold/color override and instead follows the shared font style. reservas/index.phpL61-L64

Testing

  • php -l reservas/index.php
  • git diff --check

View task →

@github-actions

github-actions Bot commented Jul 5, 2026

Copy link
Copy Markdown

📝 Manual Update Required? It looks like you've modified core logic. Please check if the user manual needs a refresh! @marpisco

@marpisco marpisco merged commit 8b435b8 into main Jul 5, 2026
6 checks passed
@marpisco marpisco deleted the codex/update-page-title-styles-for-classlink branch July 5, 2026 03:06
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants