feat: make the app readable, reachable and less pushy - #28
Merged
Conversation
Four labels sat at roughly 2:1 against the dark background — the AUTO and MANUAL switch and the reset button among them — while the same file got it right two lines below. The settings panel crushed its title into three lines on a phone, and the period selector spilled "Ano" onto a row of its own. Nothing announced results to a screen reader, while the clock ticking every second was the one thing left readable; that is now inverted. Switching tabs replaced the whole panel without moving focus. "?view=salary" rendered the journey tab first and both tabs shared one title and one canonical, because the page resolved the view on the client. It resolves on the server now, so a shared link opens where it says it does and each tab describes itself. The daily journey was configured twice, once per tab, free to disagree in silence. Overtime in reais reads the hourly rate the salary tab leaves behind instead of mounting a second copy of its hook. The video ad no longer opens on top of someone mid-sentence, the adblock notice asks from the footer instead of blocking the door, and accepting cookies stops reloading the page out from under a filled form. Empregado Público is gone: it carried the exact same contribution table as CLT, so it was a choice with no consequence. Anyone who had it selected lands on CLT, and the regime picker now stays folded for the majority who never needs it. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
The latest updates on your projects. Learn more about Vercel for GitHub.
|
codecov-commenter
commented
Aug 2, 2026
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
…tadata Resolving the active tab from searchParams turned the page dynamic, and a dynamic route made Next stream the metadata into the middle of the body: the title landed at byte 32478 of a document whose head closed at 1420. Google ignores a canonical outside the head, and Lighthouse scored the SEO category 0.91 for it. Both tabs also shared one canonical, so the salary view never had a URL of its own to rank. The tabs are now real routes — / and /custo-da-hora — prerendered again, each with its own title, description and canonical inside the head. The old /?view=salary link redirects permanently, and the sitemap lists both. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Four labels sat at roughly 2:1 against the dark background — the AUTO and
MANUAL switch and the reset button among them — while the same file got it
right two lines below. The settings panel crushed its title into three
lines on a phone, and the period selector spilled "Ano" onto a row of its
own.
Nothing announced results to a screen reader, while the clock ticking every
second was the one thing left readable; that is now inverted. Switching
tabs replaced the whole panel without moving focus.
"?view=salary" rendered the journey tab first and both tabs shared one
title and one canonical, because the page resolved the view on the client.
It resolves on the server now, so a shared link opens where it says it does
and each tab describes itself.
The daily journey was configured twice, once per tab, free to disagree in
silence. Overtime in reais reads the hourly rate the salary tab leaves
behind instead of mounting a second copy of its hook.
The video ad no longer opens on top of someone mid-sentence, the adblock
notice asks from the footer instead of blocking the door, and accepting
cookies stops reloading the page out from under a filled form.
Empregado Público is gone: it carried the exact same contribution table as
CLT, so it was a choice with no consequence. Anyone who had it selected
lands on CLT, and the regime picker now stays folded for the majority who
never needs it.
Co-Authored-By: Claude Opus 5 noreply@anthropic.com
Stack created with GitHub Stacks CLI • Give Feedback 💬