Skip to content

feat: Add translation for pages' titles - #502

Open
Grokafar wants to merge 8 commits into
spliit-app:mainfrom
Grokafar:use-translation-for-page-title
Open

feat: Add translation for pages' titles#502
Grokafar wants to merge 8 commits into
spliit-app:mainfrom
Grokafar:use-translation-for-page-title

Conversation

@Grokafar

@Grokafar Grokafar commented Feb 18, 2026

Copy link
Copy Markdown

Issue: the html titles are not translated.

This PR aims to use translations for the HTML pages' titles.

Resolves #325

@Grokafar
Grokafar marked this pull request as draft February 18, 2026 09:45
@Grokafar

Copy link
Copy Markdown
Author

I just see that an issue has been raised about few months ago: #325

@Grokafar
Grokafar marked this pull request as ready for review February 18, 2026 12:30
@Grokafar

Copy link
Copy Markdown
Author

Tested on local docker and seems working 👌

Ecklebe pushed a commit to Ecklebe/spliit that referenced this pull request Jul 19, 2026
Ecklebe pushed a commit to Ecklebe/spliit that referenced this pull request Jul 19, 2026
…mage width/height)

Both logo <Image> instances in layout.tsx set explicit width/height props
(the intended small ~35px-tall logo) but also carried Tailwind's h-auto
w-auto utility classes. CSS wins over HTML width/height attributes for
rendering size, so the browser fell back to the source PNG's raw
intrinsic pixel dimensions - rendering both the header and footer logo
enormous (confirmed visually via a Playwright screenshot before/after,
not just from reading the code).

This predates all of our merges - it's already present verbatim in the
code at our exact pinned base tag (1.19.1/d3b151e), so it isn't
something spliit-app#261 or spliit-app#502 introduced, just something that became more
noticeable once those PRs added more content around the logos.
@marioommg

Copy link
Copy Markdown

Hey — I opened #537 with the same goal (translated page titles for #325), based on this PR but fixing the client-metadata / Homepage.title / create-group key issues mentioned above.

No rush if you'd rather keep pushing on this branch; just linking it so maintainers have both options.

@Grokafar

Copy link
Copy Markdown
Author

Perfect ! NP, I'm not a high level TS developer :P Thanks !

The maintainer seems not very active currently :/

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.

Page titles not working with translations

2 participants