Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[5.x] Template color scheme configuration #3134

Merged
merged 6 commits into from
Mar 22, 2024

Conversation

tecpromotion
Copy link
Member

Pull Request für Issue #3129 .

Zusammenfassung der Änderungen

Add new strings
translation

Ich habe bewußt "Dark Mode" und "Light Mode" nicht übersetzt. Das sind gesetzte Begriffe und ich würde das gerne so lassen.

Wo wird der Sprachstring angezeigt / Wie kann getestet werden

backend
frontend

@tecpromotion tecpromotion requested a review from a team March 22, 2024 07:58
@tecpromotion tecpromotion self-assigned this Mar 22, 2024
@tecpromotion tecpromotion added this to the 5.1.0v1 milestone Mar 22, 2024
@tecpromotion tecpromotion linked an issue Mar 22, 2024 that may be closed by this pull request
Copy link
Member

@zero-24 zero-24 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ich bin mir nicht sicher ob wir das hier überstzen sollen.

Ich könte mir auch "Dunkler Modus" und "Heller Modus" vorstellen so wie es auch in Windows gemacht wurde:
image

administrator/language/de-DE/com_users.ini Outdated Show resolved Hide resolved
MOD_USER_EDIT_ACCOUNT="Konto bearbeiten"
MOD_USER_LIGHT_MODE="Light Mode"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oben haben wir "Helles Farbschema" genommen. Ich bin mir aber nicht sicher

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wir unterscheiden aber noch mal zwischen Mode und Scheme.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@zero-24 lass mal mit Light und Dark starten und schauen, was als Feedback kommt.

tecpromotion and others added 2 commits March 22, 2024 10:11
Co-authored-by: Tobias Zulauf <zero-24@users.noreply.github.com>
@zero-24
Copy link
Member

zero-24 commented Mar 22, 2024

Wir unterscheiden aber noch mal zwischen Mode und Scheme.

Ok dann lass uns so gehen. Das "verwenden" sollten wir aber einheitlich nutzen, die anderen stellen waren mir nicht aufgefallen beim ersten mal. ggfs können wir aber auch "verwenden" ganz weglassen?

@zero-24 zero-24 merged commit 442735c into joomlagerman:5.1-dev Mar 22, 2024
@zero-24
Copy link
Member

zero-24 commented Mar 22, 2024

Merging thanks 👍

@zero-24 zero-24 deleted the issue-3129 branch March 22, 2024 20:31
@tecpromotion tecpromotion linked an issue Mar 23, 2024 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

[5.1] fix alpha sorting in PR 42221 [5.x] Template color scheme configuration
2 participants