Skip to content

style: redesign settings as a single page - #7

Merged
mviniciusca merged 2 commits into
mainfrom
style/settings-page
Oct 12, 2026
Merged

mviniciusca merged 2 commits into
mainfrom
style/settings-page

Conversation

@mviniciusca

Copy link
Copy Markdown
Owner

Summary

  • One page instead of two layers of tabs: Configurações (SettingResource › EditCompany) is now a single centered page with all sections stacked (Identificação, Contato, Sede, Mapa, Documento PDF, Locação, Modo manutenção, Anti-spam, Privacidade). The separate "Segurança e Acesso" page (EditSecurity) was merged into it.
  • Same pattern as Design e SEO: "Salvar alterações" in the header (Ctrl+S), "Descartar" shown only when there are unsaved changes, a "Tudo salvo / Alterações não salvas" footer and a warning before leaving with unsaved changes. Sections use the aside layout (title on the left, fields on the right).
  • Field convention: every field has a helper text, every field that supports it has a prefix icon, every toggle shows a check icon. The rule is now documented in CLAUDE.md for all admin forms.
  • Fixes:
    • /admin/settings (the "Configurações" link in the user menu) redirected to a page that no longer exists (EditWebsite). It now opens the settings page.
    • Equipment rental showed as off in the form when never configured, while the site treats it as on. The form now matches the site.
    • Saving uses Setting::merge, like Design e SEO, so lists such as the site menu are replaced instead of merged item by item.
  • Breadcrumb and labels are in Portuguese ("Settings" was showing in English).

Breaking

  • /admin/settings/{id}/security no longer exists (it is now a section of the same page).

Test plan

  • php artisan test (new SettingsPageTest: redirect, single page, save keeps other keys, rental default)
  • Checked in the browser: every section, unsaved-changes state and Discard button

🤖 Generated with Claude Code

@mviniciusca
mviniciusca merged commit e234c42 into main Oct 12, 2026
2 checks passed

This branch was successfully deployed

1 active deployment
Testing — 2531ad32 Deployed Oct 12, 2026 by mviniciusca via laravel-tests #24
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.

1 participant