Move Workspaces dropdown to the sticky side panel #2920
Service Desk opened 2 months ago

Move the Workspaces dropdown from the top operations/status bar into the sticky right-side info panel on issue and pull request pages. Keep the existing dropdown behavior unchanged (open the dropdown to see workspaces; no always-visible status/count on the control).

Context

Raised from discussion on issue #2919. Checking workspaces currently requires scrolling back to the top of long issue/PR pages. The side info panel is sticky, so placing Workspaces there is more convenient.

Originally this issue also called for showing a workspace count on the control. That was dropped: an always-visible status/count would require keeping a live stream of workspace updates and may hurt performance unnecessarily. Relocate the control only.

Out of scope

  • Do not add an always-visible workspace count/status on the control (see above)
  • Do not bind workspaces to triggering comments:
    1. Workspaces are not necessarily triggered by comments
    2. Comment association would clutter the comment list

Acceptance ideas

  • Workspaces dropdown available in the sticky side info panel (right side)
  • Dropdown behavior remains as today (open to inspect/create/open workspaces)
  • Create/open workspace actions remain reachable from that control
  • Robin Shen commented 2 months ago

    This also applies to pull request workspaces

  • Zak Siddiqui commented 2 months ago

    Thanks @robin

  • Zak Siddiqui commented 2 months ago

    @robin Just one correction I'd like to make.

    I stand corrected on my earlier answer of wanting to always see the status of the workspaces at all times.

    I now believe this is the wrong thing to do because it means that we always need to keep a live stream of the latest information from the workspaces.

    This may negatively impact performance unnecessarily.

    So I think I'd like to keep the workspace's drop-down requirement, meaning move it into the side info panel, but only move it, not change the way it behaves.

    CC @servicedesk

    PS: The reason for my about-turn is because I read a Medium post about how you take OneDev performance very seriously, and I realized that I do not want to turn OneDev into GitLab bloatware. I want to keep it as a highly performant system.

  • Service Desk changed title 2 months ago
    Previous Value Current Value
    Move Workspaces dropdown with count to the sticky side panel
    Move Workspaces dropdown to the sticky side panel
  • Service Desk commented 2 months ago

    @zaakiy Thanks for the correction — noted and agreed.

    Updated scope for this issue:

    1. Move the existing Workspaces dropdown into the sticky side info panel on issue and pull request pages
    2. Keep current dropdown behavior — no always-visible workspace status or count on the control (that would require a live stream of workspace updates and can hurt performance unnecessarily)

    I've updated the issue title and description to match. @robin shout if you want a different direction.

  • Robin Shen changed fields 1 month ago
    Name Previous Value Current Value
    Assignee
    robin
    cwh
  • Unknown referenced from pull request 1 month ago
  • Unknown commented 1 month ago

    Implemented the workspace dropdown relocation for issue and pull request pages. The existing dropdown logic and visibility rules are unchanged, and the control now lives in the sticky side info panel.

    Verified with mvn -Pce -pl server-core -am -DskipTests compile (BUILD SUCCESS).

    Submitted as pr OD-100.

  • OneDev commented 1 month ago

    State changed as build OD-7986 (16.4.0) is successful

  • OneDev changed state to 'Closed' 1 month ago
    Previous Value Current Value
    Open
    Closed
1/1
Type
Improvement
Priority
Normal
Assignee
cwh
Labels
No labels
Issue Votes (0)
Watchers (3)
Reference
OD-2920
Please wait...
Connection lost or session expired, reload to recover
Page is in error, reload to recover