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

Theming app: Change page title to configured name #341

Closed
mtraeger opened this issue Jul 7, 2016 · 4 comments · Fixed by #371
Closed

Theming app: Change page title to configured name #341

mtraeger opened this issue Jul 7, 2016 · 4 comments · Fixed by #371

Comments

@mtraeger
Copy link

mtraeger commented Jul 7, 2016

For the Theming feature it might be interesting to exchange also the HTML page tile with the new name. At the moment there is Nextcloud or <App> - Nextcloud.

<Name> or <App> - <Name> might be interesting for some users.

@MorrisJobke
Copy link
Member

@schiessle @LukasReschke

@LukasReschke LukasReschke self-assigned this Jul 8, 2016
@nickvergessen
Copy link
Member

I'd even say this is a bug, I expected the Name option to do that. Took me a while to discover that it only changes the text in the footer.

LukasReschke added a commit that referenced this issue Jul 11, 2016
LukasReschke added a commit that referenced this issue Jul 11, 2016
MorrisJobke pushed a commit that referenced this issue Jul 11, 2016
KittieCrash pushed a commit to KittieCrash/server that referenced this issue Jul 15, 2016
@fancycode
Copy link
Member

fancycode commented Jun 13, 2017

I guess I'm a bit late to the party, but this causes some confusion with custom themes where you can implement a getTitle method - which was used for the page title before (NC 10 and 11).
Now with NC 12 the getName is used and the getTitle is ignored. With that, I can no longer just change the page title to my custom string and keep Nextcloud everywhere else...

@fancycode
Copy link
Member

Reading the changes more, this is probably not the issue where my problem was introduced - sorry for the noise, will open a new one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants