Workspaces toolbar appears very broken

Created on 25 July 2023, over 1 year ago

Problem/Motivation

If I install the Workspaces module (experimental module in Drupal Core), and try to use its toolbar, it appears totally broken (though all the functionality still basically works).

Steps to reproduce

* Install the Workspaces module
* You might need to create a second workspace at /admin/config/workflow/workspaces (I can't remember if you get two workspaces out-of-the-box)
* The "initial state" of the toolbar is okay:

* But after you click on "Stage" the Workspace toolbar appears that looks all wonky:

Proposed resolution

Style to match Gin branding.

๐Ÿ› Bug report
Status

Active

Version

3.0

Component

User interface

Created by

๐Ÿ‡ฒ๐Ÿ‡ฝMexico dalin ๐Ÿ‡จ๐Ÿ‡ฆ, ๐Ÿ‡ฒ๐Ÿ‡ฝ, ๐ŸŒ

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

  • Issue created by @dalin
  • ๐Ÿ‡ฒ๐Ÿ‡ฝMexico dalin ๐Ÿ‡จ๐Ÿ‡ฆ, ๐Ÿ‡ฒ๐Ÿ‡ฝ, ๐ŸŒ
  • ๐Ÿ‡ฎ๐Ÿ‡ณIndia swatidhurandhar

    I am working on it.

  • ๐Ÿ‡ฎ๐Ÿ‡ณIndia swatidhurandhar

    I was able to reproduce this issue in gin theme by installing workspace module and after that on clicking on Live/Stage button on header.
    I have uploaded patch to fix this.
    Thanks

  • Status changed to Needs review over 1 year ago
  • Status changed to Needs work over 1 year ago
  • ๐Ÿ‡ฎ๐Ÿ‡ณIndia Abh1shek

    Hi @swatidhurandhar, I applied your patch, but I found some errors while applying patch. Attaching error screenshot.

  • ๐Ÿ‡ฎ๐Ÿ‡ณIndia swatidhurandhar

    Hi @Abh1shek Chauhan I can see you are inside gin theme, so it is not able to find the path of all the files(which is theme/contrib/gin/dist/css/..). You should be outside theme folder which is 'drupal_OS' in your case and then try to apply the patch. It should work.

  • Status changed to Needs review over 1 year ago
  • Open in Jenkins โ†’ Open on Drupal.org โ†’
    Core: 9.5.x + Environment: PHP 5.3 & MySQL 5.5
    last update over 1 year ago
    Patch Failed to Apply
  • Status changed to RTBC over 1 year ago
  • ๐Ÿ‡ฎ๐Ÿ‡ณIndia mukesh-kumar3 Dharmshala

    Hi swatidhurandhar i have applied your patch and it is working fine for me and now the workspace toolbar appearing nice for me. so moving this to RTBC.

  • Status changed to Needs work over 1 year ago
  • ๐Ÿ‡บ๐Ÿ‡ธUnited States webdrips

    I don't think the patch works in all layouts (see attached).

  • ๐Ÿ‡บ๐Ÿ‡ธUnited States webdrips
  • Open in Jenkins โ†’ Open on Drupal.org โ†’
    Core: 10.1.x + Environment: PHP 5.3 & MySQL 5.5
    last update over 1 year ago
    Patch Failed to Apply
  • ๐Ÿ‡บ๐Ÿ‡ธUnited States davisben NH

    The patch in #4 is working well for me, but the paths should be relative to the theme so it can be applied with composer. Here is a patch with updated paths.

  • First commit to issue fork.
  • Hi @davisben,
    I have tested patch #12 and it has been successfully applied workspace toolbar is looking good to me now.
    Attached screenshot -

  • Status changed to Needs review over 1 year ago
  • Patch files are no longer recommended so I created MR for the above.
    Thanks

  • Pipeline finished with Failed
    over 1 year ago
    Total: 202s
    #65826
  • ๐Ÿ‡บ๐Ÿ‡ธUnited States darren oh Lakeland, Florida

    Darren Oh โ†’ made their first commit to this issueโ€™s fork.

  • Pipeline finished with Failed
    about 1 year ago
    Total: 265s
    #73926
  • First commit to issue fork.
  • Merge request !358gin-3376700: Workspace toolbar css. โ†’ (Open) created by Unnamed author
  • Pipeline finished with Failed
    about 1 year ago
    Total: 240s
    #75467
  • Pipeline finished with Failed
    about 1 year ago
    Total: 209s
    #75486
  • Merge request !359gin-3376700: workspace css fix โ†’ (Closed) created by Unnamed author
  • Pipeline finished with Failed
    about 1 year ago
    Total: 486s
    #75490
  • ๐Ÿ‡ฎ๐Ÿ‡ณIndia ahsannazir

    The one of the piplelines is broken. I tried raising another MR but no luck

  • Pipeline finished with Success
    about 1 year ago
    Total: 287s
    #75492
  • Status changed to Needs work about 1 year ago
  • ๐Ÿ‡ฎ๐Ÿ‡ณIndia djsagar

    Hi @ahsannazir, i applied MR #21, it's working as expected.

    For reference:-
    Before MR

    After MR

    Pipelines issue need to fixed so moving this to NW.

  • Pipeline finished with Failed
    about 1 year ago
    #77195
  • Pipeline finished with Failed
    about 1 year ago
    Total: 179s
    #77198
  • Pipeline finished with Success
    about 1 year ago
    Total: 273s
    #77201
  • Status changed to Needs review about 1 year ago
  • ๐Ÿ‡ฎ๐Ÿ‡ณIndia ahsannazir

    Pipelines are passing now. so moving to NR

  • Status changed to RTBC about 1 year ago
  • ๐Ÿ‡ฎ๐Ÿ‡ณIndia divya.sejekan

    Verified the issue and is fixed. Verified with MR!358 #21
    Testing steps :
    * Install the Workspaces module
    * Create a second workspace at /admin/config/workflow/workspaces
    * click on "Stage" the Workspace toolbar

    Moving to RTBC++

  • Pipeline finished with Success
    about 1 year ago
    Total: 212s
    #96926
  • Pipeline finished with Success
    about 1 year ago
    Total: 337s
    #96935
  • Status changed to Needs review about 1 year ago
  • ๐Ÿ‡บ๐Ÿ‡ธUnited States darren oh Lakeland, Florida

    Added the workspaces icon to the menu tab. Ready for review.

  • Pipeline finished with Success
    about 1 year ago
    Total: 222s
    #96940
  • Status changed to Needs work about 1 year ago
  • The workspaces icon is not fully visible see the screenshot.

  • Status changed to Needs review about 1 year ago
  • ๐Ÿ‡บ๐Ÿ‡ธUnited States darren oh Lakeland, Florida

    Your screen shot shows that your browser is using a cached icon sprite that does not have the workspaces icon.

  • Status changed to Needs work about 1 year ago
  • ๐Ÿ‡ฎ๐Ÿ‡ณIndia Kanchan Bhogade

    Hi @Darren!
    I've tested MR !358 on Drupal 10.2.2.
    Noticed that the workspaces icon isn't fully visible after refreshing the page.
    If there's anything I might have overlooked, especially the Drupal version, please let me know!

    Attaching screenshot

  • Status changed to Needs review about 1 year ago
  • ๐Ÿ‡บ๐Ÿ‡ธUnited States darren oh Lakeland, Florida

    Your screen shot shows the top of the icon sprite, which means your browser is using a cached sprite that does not have the workspaces icon. Try clearing your browser cache or using a private browsing window. I have added testing instructions to the issue summary.

  • Pipeline finished with Failed
    11 months ago
    Total: 246s
    #175454
  • Pipeline finished with Success
    11 months ago
    Total: 277s
    #175481
  • Pipeline finished with Success
    11 months ago
    Total: 216s
    #175483
  • Pipeline finished with Success
    9 months ago
    Total: 483s
    #215113
  • Pipeline finished with Success
    9 months ago
    Total: 210s
    #215164
  • Pipeline finished with Success
    9 months ago
    Total: 182s
    #215165
  • Pipeline finished with Success
    9 months ago
    #215185
  • Pipeline finished with Failed
    8 months ago
    Total: 268s
    #255622
  • Pipeline finished with Success
    8 months ago
    Total: 312s
    #255649
  • Pipeline finished with Failed
    6 months ago
    Total: 1089s
    #300591
  • Pipeline finished with Failed
    6 months ago
    Total: 964s
    #300610
  • Pipeline finished with Failed
    6 months ago
    Total: 601s
    #300715
  • Pipeline finished with Canceled
    6 months ago
    Total: 383s
    #300820
  • Pipeline finished with Failed
    6 months ago
    Total: 588s
    #300822
  • Pipeline finished with Failed
    6 months ago
    Total: 449s
    #307153
  • Pipeline finished with Success
    6 months ago
    #307162
  • Pipeline finished with Failed
    6 months ago
    Total: 513s
    #309414
  • First commit to issue fork.
  • Pipeline finished with Success
    6 months ago
    Total: 211s
    #310752
  • Pipeline finished with Failed
    6 months ago
    Total: 423s
    #311873
  • ๐Ÿ‡บ๐Ÿ‡ธUnited States adamzimmermann

    I updated the MR so it applies to the latest branch cleanly. I regenerated the CSS, but front-end development is not my strong suite, so hopefully I did that correctly.

    However, I'm unable to apply the patch due to it containing binary file changes. I got around that by simply removing those changes in my local patch file, but then I'm missing some icons. How are others testing this?

    A few notes:

    I'm still seeing the issue reported in #29 (likely due to my patching issue).

    Style to match Gin branding.

    This is listed in the issue description, and I think it would be great to try to do this still. If someone can point me in the right direction on the approach we want to take, I can take a swing at it.

    While the MR in its current state is a big improvement. The interface is still very different from anything else rendered in the Gin theme. This seems to be due to the CSS from the core module. Perhaps we need to be more aggressive in overriding it to provide a cohesive admin experience?

    Toolbar with the current state of the MR:

  • Pipeline finished with Failed
    6 months ago
    Total: 627s
    #312344
  • Pipeline finished with Failed
    5 months ago
    Total: 1211s
    #320290
  • Pipeline finished with Success
    5 months ago
    Total: 505s
    #320636
  • ๐Ÿ‡บ๐Ÿ‡ธUnited States adamzimmermann

    For anyone else encountering this, the Workspaces Extra module โ†’ provides a much cleaner interface that "solved" this issue for us.

Production build 0.71.5 2024