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

NC13b3 regression: Theming glitch with Logo #7575

Closed
nursoda opened this issue Dec 19, 2017 · 7 comments
Closed

NC13b3 regression: Theming glitch with Logo #7575

nursoda opened this issue Dec 19, 2017 · 7 comments
Assignees
Labels
Milestone

Comments

@nursoda
Copy link

nursoda commented Dec 19, 2017

Steps to reproduce

  1. Have 12.0.4 themed, with Logo, like
    https://datenschutz-individuell.de/cloud/index.php/apps/theming/logo?v=7
    grafik

  2. Share a folder by link, protect it with a password, like
    https://datenschutz-individuell.de/cloud/index.php/s/KSitgWbdLfj2BXo

  3. Logo is double/overlayed, see Screenshot
    grafik

Expected behaviour

Logo should be displayed correctly (once).

Actual behaviour

Logo is displayed incorrectly (overlayed, twice).

Server configuration

Operating system: Debian Stretch (up to date)
Web server: Apache 2.4.10
Database: MySQL 5.5.58
PHP version: 5.6.30

Nextcloud version: 13.0.0 Beta 3
Updated from an older Nextcloud/ownCloud or fresh install: Upgraded from 12.0.4
Where did you install Nextcloud from: admin page / installer
Signing status: no errors upon signing: http://example.com/index.php/settings/integrity/failed reports "No errors have been found."

List of activated apps: irrelevant?!
Nextcloud configuration:

{ "system": { "instanceid": "***REMOVED SENSITIVE VALUE***", "trashbin_retention_obligation": "auto,90", "versions_retention_obligation": "90,auto", "memcache.local": "\\OC\\Memcache\\APCu", "memcache.locking": "\\OC\\Memcache\\Redis", "filelocking.enabled": "true", "redis": { "host": "***REMOVED SENSITIVE VALUE***", "port": 6379, "timeout": 0 }, "passwordsalt": "***REMOVED SENSITIVE VALUE***", "secret": "***REMOVED SENSITIVE VALUE***", "trusted_domains": [ "datenschutz-individuell.de" ], "datadirectory": "***REMOVED SENSITIVE VALUE***", "overwrite.cli.url": "https:\/\/datenschutz-individuell.de\/cloud", "dbtype": "mysql", "version": "13.0.0.8", "dbname": "***REMOVED SENSITIVE VALUE***", "dbhost": "***REMOVED SENSITIVE VALUE***", "dbport": "", "dbtableprefix": "oc_", "dbuser": "***REMOVED SENSITIVE VALUE***", "dbpassword": "***REMOVED SENSITIVE VALUE***", "logtimezone": "Europe\/Berlin", "installed": true, "mail_from_address": "***REMOVED SENSITIVE VALUE***", "mail_smtpmode": "php", "mail_domain": "***REMOVED SENSITIVE VALUE***", "defaultapp": "calendar", "maintenance": false, "loglevel": 2, "theme": "", "updater.release.channel": "beta", "mail_smtpauthtype": "LOGIN" } }

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: Firefox 57 / Chrome 63
Operating system: W10

Logs

Web server error log: No entry.

Nextcloud log (data/nextcloud.log): No entry.

Browser log: No entry in Firefox browser console.

@blizzz
Copy link
Member

blizzz commented Dec 19, 2017

@nextcloud/theming

@blizzz blizzz added this to the Nextcloud 13 milestone Dec 19, 2017
@MorrisJobke
Copy link
Member

This is because there is a background image and an <img> tag.

@MorrisJobke
Copy link
Member

Also the CSS looks like it duplicates stuff.
bildschirmfoto 2017-12-19 um 15 23 45

@MorrisJobke MorrisJobke added the design Design, UI, UX, etc. label Dec 19, 2017
@MorrisJobke
Copy link
Member

cc @nextcloud/designers for the HTML/CSS

@pixelipo
Copy link
Contributor

pixelipo commented Dec 19, 2017

Looks rather like a @nextcloud/theming issue - <img /> should not be added AFAIK

@pixelipo
Copy link
Contributor

While testing this I found out that the default logo is not loaded:
image
because the svg image path in CSS is not correct...

@juliusknorr
Copy link
Member

We also use a img tag on the login page to make sure the logo is scaled nicely when it is not in landscape format: https://github.com/nextcloud/server/pull/4632/files#diff-20abc3ec8914f60360246ff607542301R43 I guess we should adapt that to the files_sharing public page as well.

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

No branches or pull requests

5 participants