Spreed: Conversation menu should not be in the main content

[skjnldsv]

Currently the main menu for a conversation is on the main messages list.
When the sidebar is opened it is confusing as all the other apps on Nextcloud have the menu inside the sidebar (sharing, copy link, rename...)

TODO:

• [ ] Use the same sharing UI as server
  
  
  
  • [ ] Link share copy
  
  
  • [ ] Password protection
  
  
• [ ] Put the rename function in the Sidebar menu (when opened only?)

Already seen with @jancborchardt

[nickvergessen]

Well, we've been there and had that.
I guess once we have the Details tab, we can move some things there. But then again adding all the "share related" settings in the participant list adds lot of bloat again that is almost never changed.

[skjnldsv]

But then again adding all the "share related" settings in the participant list adds lot of bloat again that is almost never changed.

See with Jan, I think it should be the same across NC apps!

[PVince81]

From Talk 11.0 pre-alpha:

As discussed, we can move most of the options to the "Settings" tag on the right sidebar:

• sidebar settings entries:
  
  
  
  • rename conversation: as pencil icon behind the name when moderator ? stays
  
  
  • [x] future section "visibility": see https://github.com/nextcloud/spreed/issues/1763#issuecomment-721008640
    
    
    
    
    
    • listable for regular users
    
    
    
    • listable for guest users (from guest app, when guest app is enabled)
    
    
    
  
  
  • [x] section "share":
    
    
    
    
    
    • [x] share link / allowing guests
    
    
    
    • [x] copy link (but leave the link also in the menu)
    
    
    
    • [x] password protection
    
    
    
  
  
  • [x] section "Restrictions"
    
    
    
    
    
    • [x] lock conversation
    
    
    
    • [x] lobby mode + start time
    
    
    
  
  
  • [x] section SIP
    
    
    
    
    
    • enable SIP dial-in
    
    
    
    • ... potentially other new SIP options ...
    
    
    
  
  
• [x] add menu entry "more settings" that opens the panel

[PVince81]

• if "full screen" stays the only ever option, we could replace the three dots directly with that icon

How to display "potentially dangerous flags" ? A simple auto-saving checkbox feels a bit risky for "lock room" or "visible for guests". Convert some to buttons ?
Alternatively all this (or part of it) could be put into a modal that can be triggered from the menu or settings panel... the modal would then have a "Save" button.

Thoughts ? @nextcloud/designers-talk

[ma12-co]

We came to the conclusion that the 3 dot menu should stay there with at least the following:

• Fullscreen
• Rename
• Copy link if conv is public
• More settings (opens sidebar and conversations settings tab)

The 3 dot menu was moved out of the sidebar deliberately as a first move towards a more usable talk even with the sidebar always collapsed. Deck too has a 3 dot menu in that area just below the header.

[PVince81]

PR here: https://github.com/nextcloud/spreed/pull/4576

[schiessle]

Just as some quick feedback from a users perspective. I always get super confused that the rename operation is so detached from the actual name. Everytime I want to rename a room I hover over the name in the sidebar to see if I can click it in order to edit it or that a "rename icon" appears next to it on hover. It always feels strange that I have to go to the chat/video window to rename something which is displayed in the sidebar. Even that I use Talk every day and create/rename a room almost every day I can't get used to it, so I wonder how confusing it might be for people who use it only occasionally.

[PVince81]

@schiessle for now rename will stay in that menu. Moving it to a better place is a separate topic, and I agree that it's confusing currently. We did a user testing session this week and also observed that people tend to expect this elsewhere.

[PVince81]

as discussed with @ma12-co, more things to move:

• move to the conversation settings dialog:
  
  
  
  • [ ] mattermost settings
  
  
  • [ ] project link ?
  
  
  • [ ] from the left menu: notifications preferences
  
  

[ma12-co]

@PVince81 from the left menu I'd also bring leave/delete conversation.

To address the issue brought up by @schiessle I would go back to the regular AppSidebar renaming but also have a renaming capability in the conversation settigs.

Btw the discussion that we're having here is moving out from the original scope of the issue. Not sure if we should move it to anther one or rename this issue.

[PVince81]

assigning self since I'm working on https://github.com/nextcloud/spreed/pull/4576

however once this is merged we might defer the additional bits as they have less priority

[PVince81]

this https://github.com/nextcloud/spreed/issues/4007#issuecomment-721023598 is done now.

the "listable" section will be next as part of publicly joinable rooms

[PVince81]

the listable section was merged and appears in the dialog