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.

Sign up for GitHub

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

Jump to bottom

[4] New moduleposition below-top, option to switch off logo #1791

Merged
merged 7 commits into from May 31, 2021
Merged

[4] New moduleposition below-top, option to switch off logo #1791

merged 7 commits into from May 31, 2021

Conversation

max123kl
Copy link
Contributor

Pull Request für Issue #1744

Zusammenfassung der Änderungen

Siehe #1744 (comment)

Wo wird der Sprachstring angezeigt / Wie kann getestet werden

@tecpromotion tecpromotion requested a review from a team May 29, 2021 12:19
@tecpromotion tecpromotion added this to the 4.0.0v1 milestone May 29, 2021
@tecpromotion tecpromotion linked an issue May 29, 2021 that may be closed by this pull request
[4] New moduleposition below-top, option to switch off logo #1744
Closed
tecpromotion
tecpromotion requested changes May 29, 2021
View reviewed changes
language/de-DE/tpl_cassiopeia.sys.ini Outdated
@@ -7,6 +7,7 @@
CASSIOPEIA="Cassiopeia Frontend Template"
TPL_CASSIOPEIA_MOD_MENU_LAYOUT_DROPDOWN-METISMENU="Dropdown"
TPL_CASSIOPEIA_POSITION_BANNER="Banner"
TPL_CASSIOPEIA_POSITION_BELOW-TOP="Below Top"
Copy link
Member

Choose a reason for hiding this comment

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

Translation missing

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Richtig, ich hatte in meinem Kommentar schon geschrieben:

Hier fällt mir keine wirklich gute Übersetzung ein:
TPL_CASSIOPEIA_POSITION_BELOW-TOP="Below Top"

Kann das so bleiben?

Copy link
Member

Choose a reason for hiding this comment

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

Nee, dass müssen wir übersetzen.

Copy link
Contributor Author

@max123kl max123kl May 30, 2021 •
edited

Choose a reason for hiding this comment

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

Wird diese Übersetzung noch an anderer Stelle benutzt als beim Auswahlmenü für die Modulposition?
Position_A

Wenn nein, dann kann ich keinen Nutzen erkennen – im Gegenteil, je nach Seitenbreite des Fensters und der Länge der Übersetzung bricht die Zeile unkontrolliert um und erschwert die Lesbarkeit.
Position_B

Ein weiterer Nachteil die Modulpositionen zu übersetzen ist in den nächsten Bildern zu erkennen. Weder in der Vorschau noch in der Modul-Liste wird der übersetzte Begriff verwendet. Zwei Begriffe für die gleiche Position schafft nur Verwirrung.

Vorschau
Modul-Liste

Copy link
Member

Choose a reason for hiding this comment

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

Schaue ich mir an aber guter Einwand. Wird geprüft.

Copy link
Member

Choose a reason for hiding this comment

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

Nachdem man in der Suche nach einer Modulposition auch nach dem "Übersetzten" suchen kann, sollten wir es auf deutsch lassen. Das kennen die Nutzer auch bereits von den bisherigen Core-Templates.

language/de-DE/tpl_cassiopeia.ini Outdated Show resolved Hide resolved
@tecpromotion tecpromotion marked this pull request as draft May 29, 2021 13:35
@tecpromotion
Update language/de-DE/tpl_cassiopeia.sys.ini
dec7dcc 
nach Rücksprache mit @zero-24 übersetzen wir es erstmal so und schauen falls jemand eine bessere Idee hat.
@tecpromotion tecpromotion marked this pull request as ready for review May 31, 2021 17:28
@tecpromotion tecpromotion merged commit e566866 into joomlagerman:4.0-dev May 31, 2021
max123kl
max123kl commented May 31, 2021
View reviewed changes
language/de-DE/tpl_cassiopeia.sys.ini
@@ -7,6 +7,7 @@
CASSIOPEIA="Cassiopeia Frontend Template"
TPL_CASSIOPEIA_MOD_MENU_LAYOUT_DROPDOWN-METISMENU="Dropdown"
TPL_CASSIOPEIA_POSITION_BANNER="Banner"
TPL_CASSIOPEIA_POSITION_BELOW-TOP="unterhalb Oben"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Leider ist "below_top" oberhalb von "Oben-a" und "Oben-b" positioniert
d.h. es ist direkt unter "top-bar" noch oberhalb von "menu"

@max123kl max123kl deleted the Issue-1744 branch May 31, 2021 18:01
@tecpromotion tecpromotion mentioned this pull request May 31, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tecpromotion tecpromotion tecpromotion approved these changes

Assignees

@max123kl max123kl

Labels
core language change Joomla! 4.x
Milestone
4.0.0v1
Development

Successfully merging this pull request may close these issues.

[4] New moduleposition below-top, option to switch off logo
2 participants
@max123kl @tecpromotion