Skip to content

fix(login): ajustar background e texto para systemas com light theme - #481

Open
fernanduandrade wants to merge 1 commit into
4.xfrom
fix/login-text
Open

fix(login): ajustar background e texto para systemas com light theme#481
fernanduandrade wants to merge 1 commit into
4.xfrom
fix/login-text

Conversation

@fernanduandrade

@fernanduandrade fernanduandrade commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

Contexto

Esta alteração visa corrigir a omissão do texto "Entrar" na tela de login para usuários que usam light theme. Foi aplicado suporte para o texto ser visivél tanto em light/dark theme

Alterações

  • O login.blade.php foi alterado garantindo suporte para os dois temas.
  • Ajustado background para o span "ou" que também não estava de acordo para usuários light theme

Plano de Testes

  • Executar make check
  • Executar make test
  • Abrir /app/login usando light theme e confirmar que o texto "Entrar" é exibido

Evidências

Antes

image

Depois

image

Issues Relacionadas

Fixes #480

@fernanduandrade fernanduandrade self-assigned this Aug 11, 2026
@fernanduandrade
fernanduandrade requested a review from a team August 11, 2026 23:37
@fernanduandrade fernanduandrade added bug Something isn't working difficulty:easy 1-2 days labels Aug 11, 2026
@coderabbitai

coderabbitai Bot commented Aug 11, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository YAML (base), Central YAML (inherited)

Review profile: CHILL

Plan: Pro Plus

Run ID: d0480e3f-336e-49a3-a468-1a19733f880a

📥 Commits

Reviewing files that changed from the base of the PR and between 021bc45 and 455bdd5.

📒 Files selected for processing (1)
  • app-modules/panel-app/resources/views/auth/login.blade.php

📝 Walkthrough

Walkthrough

The login heading now uses dark text in light mode and white text in dark mode. The divider label now uses matching light-mode and dark-mode background and text colors.

Suggested reviewers: danielhe4rt

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed O título descreve claramente a correção de tema claro na tela de login.
Description check ✅ Passed A descrição inclui contexto, alterações, testes planejados, evidências visuais e a issue relacionada.
Linked Issues check ✅ Passed As alterações atendem ao objetivo da issue #480 ao tornar “Entrar” visível no tema claro e ajustar o divisor “ou”.
Out of Scope Changes check ✅ Passed As alterações estão limitadas ao arquivo login.blade.php e permanecem alinhadas ao objetivo da issue #480.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@Haavertz

Copy link
Copy Markdown

LGTM!

@GabrielFVDev GabrielFVDev 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.

LGTM!!!!!!!

@ranyesantos ranyesantos 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.

LGTM!!

@franSborges franSborges 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.

LGTM ✍️

@ju14x ju14x 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.

LGTM ✍️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working difficulty:easy 1-2 days

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix: login esconde texto para navegador com light theme

6 participants