xmpp-web icon indicating copy to clipboard operation
xmpp-web copied to clipboard

Default logo reference should not be absolute

Open guusdk opened this issue 1 year ago • 4 comments

Describe the bug

I'm trying to deploy xmpp-web in a context root of my webserver (eg: https://example/xmppweb).

On the login screen, I'm greeted with a broken image.

It appears that the default location of the image that is used, (when the logoUrl option is left empty) is absolute: /img/icons/android-chrome-192x192.png. This causes the breaking image, as the webapp attempts to download:

https://example/img/icons/android-chrome-192x192.png

instead of:

https://example/xmppweb/img/icons/android-chrome-192x192.png

It is easily worked-around by configuring the relative path in local.js, like this:

  logoUrl: "./img/icons/android-chrome-192x192.png",

... but it'd be nice if this would work out of the box.

Steps to reproduce

No response

Expected behavior

Without specifying a logoUrl, the image that is used by default should be loaded based on a relative, not absolute location.

Relevant log

No response

local.js configuration

No response

XMPP-web version

0.10.0

Installation

Github release archive

XMPP server(s)

Openfire

Browser(s)

Chrome

Device(s)

desktop

Other information

No response

guusdk avatar May 18 '24 14:05 guusdk

Yes, please. This is a problem that's been keeping me from using this application.

virtadpt avatar May 18 '24 21:05 virtadpt

Does the workaround that I described not work for you?

guusdk avatar May 18 '24 21:05 guusdk

Hi, yes the problem is here: https://github.com/nioc/xmpp-web/blob/master/src%2Fcomponents%2FLogin.vue#L99

I can fix it, do you have any others issues with subfolder usage?

nioc avatar May 18 '24 23:05 nioc

It is still early days, but I have not found any other of these issues yet!

guusdk avatar May 19 '24 06:05 guusdk