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

Correct the name of logo #36876

Closed

Conversation

JuliaKirschenheuter
Copy link
Contributor

Summary

Correct the name of logo to "Nextcloud - go to dashboard".

Structure:

Screenshot from 2023-02-27 14-28-43

Checklist

Signed-off-by: julia.kirschenheuter <julia.kirschenheuter@nextcloud.com>
@JuliaKirschenheuter
Copy link
Contributor Author

/backport to stable25

@@ -62,7 +62,7 @@
id="nextcloud">
<div class="logo logo-icon">
<span class="hidden-visually">
<?php p($l->t('%s homepage', [$theme->getName()])); ?>
<?php p($l->t('%s - go to dashboard', [$theme->getName()])); ?>
Copy link
Member

Choose a reason for hiding this comment

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

It's not always the dashboard. The app this links to is actually configurable for admins.

/**
* Set the default app to open on login. Use the app names as they appear in the
* URL after clicking them in the Apps menu, such as documents, calendar, and
* gallery. You can use a comma-separated list of app names, so if the first
* app is not enabled for a user then Nextcloud will try the second one, and so
* on. If no enabled apps are found it defaults to the dashboard app.
*
* Defaults to ``dashboard,files``
*/
'defaultapp' => 'dashboard,files',

@JuliaKirschenheuter
Copy link
Contributor Author

This PR makes no sense anymore because a right implementation would take to much time, see #36876 (comment). Closing issue

@JuliaKirschenheuter JuliaKirschenheuter deleted the fix/36753-corect-Nextcloud_logo-name branch February 28, 2023 13:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews
Projects
None yet
4 participants