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

Breadcrumb menu is misaligned on share links #2282

Closed
tribut opened this issue Oct 1, 2019 · 6 comments
Closed

Breadcrumb menu is misaligned on share links #2282

tribut opened this issue Oct 1, 2019 · 6 comments
Assignees
Labels
bug feature: talk-sidebar ⬅️ Sidebar integration of Talk into other apps like sharing and documents regression

Comments

@tribut
Copy link

tribut commented Oct 1, 2019

Steps to reproduce

  1. Upgrade to NC 17.0
  2. Share a link to a folder
  3. Observe breadcrumb menu

Expected behaviour

Menu should be aligned to top, not overlaying header from files app. Grid view toggle should also be aligned to breadcrumb and scroll.

Actual behaviour

1
2

This is a regression from 16.0.4.

Server configuration

Operating system: Debian stretch
Web server: Apache 2.4
Database: MariaDB
PHP version: 7.2
Nextcloud version: 17.0
Updated from an older Nextcloud/ownCloud or fresh install: Upgraded from 16.0.4
Where did you install Nextcloud from: Tarball
Signing status:

Signing status
No errors have been found.

List of activated apps:

App list
Enabled:
  - accessibility: 1.3.0
  - activity: 2.10.1
  - apporder: 0.7.1
  - bruteforcesettings: 1.4.0
  - calendar: 1.7.1
  - cloud_federation_api: 1.0.0
  - comments: 1.7.0
  - contacts: 3.1.4
  - dav: 1.13.0
  - federatedfilesharing: 1.7.0
  - federation: 1.7.0
  - files: 1.12.0
  - files_pdfviewer: 1.6.0
  - files_rightclick: 0.15.1
  - files_sharing: 1.9.0
  - files_trashbin: 1.7.0
  - files_versions: 1.10.0
  - files_videoplayer: 1.6.0
  - firstrunwizard: 2.6.0
  - gallery: 18.4.0
  - logreader: 2.2.0
  - lookup_server_connector: 1.5.0
  - music: 0.10.1
  - nextcloud_announcements: 1.6.0
  - notes: 3.0.3
  - notifications: 2.5.0
  - oauth2: 1.5.0
  - password_policy: 1.7.0
  - previewgenerator: 2.1.0
  - privacy: 1.1.0
  - provisioning_api: 1.7.0
  - recommendations: 0.5.0
  - richdocuments: 3.4.2
  - serverinfo: 1.7.0
  - sharebymail: 1.7.0
  - spreed: 7.0.0
  - support: 1.0.1
  - survey_client: 1.5.0
  - systemtags: 1.7.0
  - tasks: 0.11.3
  - text: 1.1.0
  - theming: 1.8.0
  - twofactor_backupcodes: 1.6.0
  - twofactor_nextcloud_notification: 2.1.1
  - twofactor_totp: 4.0.0
  - updatenotification: 1.7.0
  - viewer: 1.1.0
  - workflowengine: 1.7.0
Disabled:
  - admin_audit
  - encryption
  - files_external
  - user_ldap

Nextcloud configuration:

Config report
{
    "system": {
        "instanceid": "***REMOVED SENSITIVE VALUE***",
        "passwordsalt": "***REMOVED SENSITIVE VALUE***",
        "datadirectory": "***REMOVED SENSITIVE VALUE***",
        "dbtype": "mysql",
        "version": "17.0.0.9",
        "dbname": "***REMOVED SENSITIVE VALUE***",
        "dbhost": "***REMOVED SENSITIVE VALUE***",
        "dbtableprefix": "oc_",
        "dbuser": "***REMOVED SENSITIVE VALUE***",
        "dbpassword": "***REMOVED SENSITIVE VALUE***",
        "installed": true,
        "forcessl": true,
        "forceSSLforSubdomains": true,
        "config_is_read_only": false,
        "memcache.local": "\\OC\\Memcache\\APCu",
        "log_type": "syslog",
        "loglevel": 2,
        "appstoreenabled": true,
        "mail_smtpmode": "smtp",
        "theme": "",
        "maintenance": false,
        "trusted_domains": [
            "***REMOVED SENSITIVE VALUE***"
        ],
        "secret": "***REMOVED SENSITIVE VALUE***",
        "trashbin_retention_obligation": "auto",
        "asset-pipeline.enabled": false,
        "assetdirectory": "***REMOVED SENSITIVE VALUE***",
        "overwrite.cli.url": "***REMOVED SENSITIVE VALUE***",
        "htaccess.RewriteBase": "\/",
        "mail_smtpauthtype": "LOGIN",
        "mail_from_address": "***REMOVED SENSITIVE VALUE***",
        "mail_domain": "***REMOVED SENSITIVE VALUE***",
        "mail_smtphost": "***REMOVED SENSITIVE VALUE***",
        "preview_max_filesize_image": 50,
        "preview_max_x": 16384,
        "preview_max_y": 16384,
        "simpleSignUpLink.shown": false,
        "mysql.utf8mb4": true
    }
}

Are you using external storage, if yes which one: no
Are you using encryption: no
Are you using an external user-backend, if yes which one: no

Client configuration

Browser: FF 69
Operating system: Ubuntu

Logs

n/a

@kesselb
Copy link
Contributor

kesselb commented Oct 1, 2019

cc @skjnldsv @GretaD

@GretaD
Copy link
Contributor

GretaD commented Oct 7, 2019

hey @tribut, can you please check if this problem still exist, I cannot reproduce it anymore. Also I notice that in the screenshot you sent, there's no 'download all' button. Please when you test it, check if the button also will show up

@tribut
Copy link
Author

tribut commented Oct 7, 2019

Hey @GretaD, thanks for checking. Have you tried with NC 17.0? As far as I see this is still the latest release. All the apps are up-to-date according to the Apps page in NC.

Here is a link to an example gallery: https://wolke.tribut.de/s/Yyn8DCY4AEy2yxr

The "Download all" button is there, but it moves to the overflow menu once the screen width crosses a certain threshold.

@GretaD
Copy link
Contributor

GretaD commented Oct 7, 2019

this is a very interesting bug (not in a good way :P ) Yes, you're right, it actually happens also on 18th version. I am going to investigate a bit more and push a PR when I find out what is happening. When i test it locally on my dev env, both versions are ok :/

@danxuliu
Copy link
Member

danxuliu commented Oct 7, 2019

The issue is a regression introduced by the Talk sidebar in public share pages feature added in Talk 7, so I will transfer this to the Talk repository :-)

In order to add the sidebar the default layout is modified, but this modification is applied even when the sidebar is not visible, and unfortunately it breaks the layout with shared folders :-(

It is related to the issue with the toolbar in Text, although in the case of shared folders the best solution might be not to load the Talk sidebar at all.

@danxuliu danxuliu transferred this issue from nextcloud/server Oct 7, 2019
@danxuliu danxuliu self-assigned this Oct 7, 2019
@danxuliu danxuliu added bug feature: talk-sidebar ⬅️ Sidebar integration of Talk into other apps like sharing and documents regression labels Oct 7, 2019
@danxuliu
Copy link
Member

This was fixed in #2340, and therefore in Talk 7.0.2, which was released today :-) (but please note that you also need Nextcloud 17.0.1!)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug feature: talk-sidebar ⬅️ Sidebar integration of Talk into other apps like sharing and documents regression
Projects
None yet
Development

No branches or pull requests

4 participants