Skip to content

Refine translations - #6261

Merged
xet7 merged 2 commits into
wekan:mainfrom
KhaoulaMaleh:fix/translation-refinement
Apr 8, 2026
Merged

Refine translations#6261
xet7 merged 2 commits into
wekan:mainfrom
KhaoulaMaleh:fix/translation-refinement

Conversation

@KhaoulaMaleh

Copy link
Copy Markdown
Contributor

Changes made (Translation Refinement):

Cleared Placeholders: Removed redundant text from comment-placeholder and desc-placeholder in en.i18n.json to ensure input fields start empty.
Clarified Active Labels: Renamed generic "Active" status markers to "Active Team", "Active Organization", and "Active Person" for clearer distinction in the Admin Panel

Fixes #6090

@xet7

xet7 commented Apr 7, 2026

Copy link
Copy Markdown
Member

@KhaoulaMaleh

What is empty placeholder used for? It is empty. Could placeholder related code and translations be removed? Or is there some use case for this?

@KhaoulaMaleh

Copy link
Copy Markdown
Contributor Author

@xet7 The reason I initially kept the placeholder logic was for flexibility it allows us to add helpful cues for new users in the future without needing to refactor the core editor.
However, I agree that code simplicity is more important right now. I have removed the logic and the keys to keep Wekan lean.

@xet7

xet7 commented Apr 7, 2026

Copy link
Copy Markdown
Member

@KhaoulaMaleh

That placeholder code change should be reverted too:
https://github.com/wekan/wekan/pull/6259/changes

@KhaoulaMaleh
KhaoulaMaleh force-pushed the fix/translation-refinement branch from 7fb31ce to 317c82a Compare April 7, 2026 23:53
@KhaoulaMaleh

Copy link
Copy Markdown
Contributor Author

I reverted the placeholder code ,removed desc-placeholder from cardDescription.jade and en.i18n.json,reverted editor.jade to use only the static comment-placeholder.
All input fields are now empty by default. Thanks @xet7

@xet7
xet7 merged commit bdc65ec into wekan:main Apr 8, 2026
9 of 11 checks passed
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.

Enhance translations with new key:value pairs

2 participants