nextcloud / server

☁️ Nextcloud server, a safe home for all your data
https://nextcloud.com
GNU Affero General Public License v3.0
27.25k stars 4.05k forks source link

Extend manifest to also include /call/ url #36382

Open oktayacikalin opened 1 year ago

oktayacikalin commented 1 year ago

How to use GitHub


Is your feature request related to a problem? Please describe. I like to use Talk as a standalone web app. Therefore, in Brave I'm using the feature to create a shortcut which opens Talk in it's own window without the location bar, buttons etc.. But when I switch into a chat, the URL changes from /apps/spreed/ to /call/, which in turn shows the annoying stripped down location indicator, that wants to tell me that I left my "app".

Describe the solution you'd like Please extend the manifest to also include the /call/ URL or move it /apps/spreed/call/. I do not want to waste screen estate for that annoying bar.

Describe alternatives you've considered If I had the time and expertise in this area, I would try to extend the manifest by myself.

Additional context image

nickvergessen commented 1 year ago

I have no clue what manifest you are talking about. Never heard of that.

Moving the call urls inside apps/spreed is nothing we will do.

oktayacikalin commented 1 year ago

Moving the call urls inside apps/spreed is nothing we will do.

Why are they separated anyway?

oktayacikalin commented 1 year ago

I have no clue what manifest you are talking about. Never heard of that.

I think, it is being generated here: https://github.com/nextcloud/server/blob/master/apps/theming/lib/Controller/ThemingController.php#L392

nickvergessen commented 1 year ago

Why are they separated anyway?

Because we wanted to have shorter and more user friendly URLs for the call links which people might get via email or messengers when being asked to join

nickvergessen commented 1 year ago

I think, it is being generated here:

Okay, then I will move it to that repo, as we here can not control it.