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

Titles for share pages are not specific #11506

Open
jancborchardt opened this issue Oct 1, 2018 · 13 comments
Open

Titles for share pages are not specific #11506

jancborchardt opened this issue Oct 1, 2018 · 13 comments
Labels
0. Needs triage Pending check for reproducibility or if it fits our roadmap 25-feedback bug feature: files feature: sharing

Comments

@jancborchardt
Copy link
Member

Screenshot of titles for share pages in browser

The titles for share pages are missing some info:

  • Shared files just show Nextcloud
  • Shared folders show Files - Nextcloud

Instead, we should show the actual names, as we also do in the header:

  • The shared file shown above should have Screenshot from 2018-01-07 00-36-09.png - Nextcloud
  • The shared folder shown above should have Screenshots - Nextcloud

It seems the title has to be passed via PHP or so, so cc @nextcloud/sharing
@jancborchardt jancborchardt added bug good first issue Small tasks with clear documentation about how and in which place you need to fix things in. 1. to develop Accepted and waiting to be taken care of feature: sharing labels Oct 1, 2018
@jancborchardt jancborchardt added this to the Nextcloud 15 milestone Oct 1, 2018
@nextcloud nextcloud deleted a comment from nextcloud-bot Oct 1, 2018
@sebastiaanspeck
Copy link

I'll take this

@sebastiaanspeck
Copy link

Where can I find the php-file where this has to be changed?

@MorrisJobke
Copy link
Member

Sadly this is partly done in PHP and then also in JS afterwards 🙈

PHP:

server/core/templates/layout.base.php

Line 6 in b4ea19c

<?php p($theme->getTitle()); ?>

JS:

server/apps/files/js/filelist.js

Lines 976 to 987 in dccd04e

setPageTitle: function(title){
if (title) {
title += ' - ';
} else {
title = '';
}
title += this.appName;
// Sets the page title with the " - Nextcloud" suffix as in templates
window.document.title = title + ' - ' + oc_defaults.title;
return true;
},

@sebastiaanspeck
Copy link

sebastiaanspeck commented Oct 3, 2018 •
edited
Loading

Correct me if I'm wrong, but the PHP-function getTitle() has to be changed and the JS-function setPageTitle has to be changed?

If so:
PHP:

server/apps/theming/lib/ThemingDefaults.php

Line 129 in 893d2da

return strip_tags($this->config->getAppValue('theming', 'name', $this->title));

has to be changed to use the filename or folder

JS:

server/apps/files/js/filelist.js

Line 47 in 893d2da

appName: t('files', 'Files'),

has to be changed to use the filename or folder instead of files/Files

@MorrisJobke
Copy link
Member

has to be changed to use the filename or folder instead of files/Files

I would just change the JS side for now. This should be enough.

@sebastiaanspeck
Copy link

Okay, but how can I get the filename/folder? In the screenshot I see the string (fileId?), but how can I convert that to the filename/folder?

@MorrisJobke
Copy link
Member

Okay, but how can I get the filename/folder? In the screenshot I see the string (fileId?), but how can I convert that to the filename/folder?

Good question. Maybe @skjnldsv has an idea. I thought it is in the data attributes of <html> but it isn't.

@skjnldsv
Copy link
Member

skjnldsv commented Oct 4, 2018

@MorrisJobke I have no idea! :(
@jancborchardt do not put good first issue on stuff that touches the js from files 😬 😜

@nextcloud-bot nextcloud-bot mentioned this issue Nov 6, 2018
Closed
@MorrisJobke MorrisJobke removed Hacktoberfest good first issue Small tasks with clear documentation about how and in which place you need to fix things in. labels Nov 15, 2018
@MorrisJobke
Copy link
Member

Moved to 16.

@jancborchardt jancborchardt mentioned this issue Mar 13, 2019
Closed
62 tasks
@rullzer rullzer modified the milestones: Nextcloud 16, Nextcloud 17 Mar 22, 2019
@juliushaertl juliushaertl mentioned this issue Jun 13, 2019
Closed
@rullzer rullzer removed this from the Nextcloud 17.0.2 milestone Dec 19, 2019
@MorrisJobke MorrisJobke added this to the Nextcloud 20 milestone Aug 20, 2020
@rullzer rullzer modified the milestones: Nextcloud 20, Nextcloud 21 Oct 3, 2020
@rullzer rullzer modified the milestones: Nextcloud 21, Nextcloud 22 Mar 1, 2021
@skjnldsv skjnldsv modified the milestones: Nextcloud 22, Nextcloud 23 Jul 5, 2021
@blizzz blizzz modified the milestones: Nextcloud 23, Nextcloud 24 Nov 30, 2021
@blizzz blizzz modified the milestones: Nextcloud 24, Nextcloud 25 Apr 21, 2022
@blizzz blizzz modified the milestones: Nextcloud 25, Nextcloud 26 Oct 19, 2022
@blizzz

This comment was marked as resolved.

Sign in to view
@szaimen

This comment was marked as resolved.

Sign in to view
@szaimen szaimen added needs info 0. Needs triage Pending check for reproducibility or if it fits our roadmap and removed 1. to develop Accepted and waiting to be taken care of labels Jan 9, 2023
@weasam
Copy link

weasam commented Mar 2, 2023

Still an issue at 25.0.4. when sharing the folder or file, the people received my sharing links in what's app or other chat platform doesn't show the title

@blizzz blizzz modified the milestones: Nextcloud 26, Nextcloud 27 Mar 9, 2023
@c-prompt
Copy link

c-prompt commented Apr 5, 2023

Unclear if this is related...

Sharing files/documents and then opening via Community Document Server/OnlyOffice/pdfviewer/etc. does not properly update document.title. Therefore, when apps like cloud_piwik (matomo) try to track analytics with the page name, the page name is almost always completely incorrect. This might be a Javascript order problem but I haven't been able to figure out as the values I'm seeing in the console for things like document.head.getElementsByTagName('title')[0], document.getElementsByTagName('head')[0].getElementsByTagName('title')[0], and document.getElementsByClassName('header-appname')[0].innerHTML sometimes reflect the appropriate document.title and sometimes not.

As a result, analytics are not accurate.

@skjnldsv skjnldsv removed this from the Nextcloud 27.0.2 milestone Aug 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
0. Needs triage Pending check for reproducibility or if it fits our roadmap 25-feedback bug feature: files feature: sharing
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
10 participants
@rullzer @MorrisJobke @jancborchardt @joshtrichards @blizzz @c-prompt @sebastiaanspeck @weasam @skjnldsv @szaimen