Improve the visual indication of the active workspace in the Workspaces listing UI

Created on 5 April 2021, about 3 years ago
Updated 19 March 2024, 3 months ago

Problem/Motivation

On the Workspaces listing screen (/admin/config/workflow/workspaces), we currently try to show which workspace is currently active by highlighting its row in the table:

However, highlighting the row which contains the active workspace is very unclear and it doesn't really achieve its purpose.

Steps to reproduce

Enable the Workspaces module and navigate to /admin/config/workflow/workspaces.

Proposed resolution

This issue was discussed in a usability meeting on Jan 16th 2020 (see #3092553-6: Add a row for switching to the live workspace in the Workspaces listing UI โ†’ ), and the consensus was to replace the table row highlight with a strong emphasis on the workspace label, and add a (active) suffix.

Remaining tasks

Reviews.

User interface changes

Workspaces listing page with the proposed changes:

API changes

Nope.

Data model changes

Nope.

Release notes snippet

Nope.

๐Ÿ› Bug report
Status

Needs work

Version

11.0 ๐Ÿ”ฅ

Component
Workspacesย  โ†’

Last updated 8 days ago

No maintainer
Created by

๐Ÿ‡ท๐Ÿ‡ดRomania amateescu

Live updates comments and jobs are added and updated live.
  • Needs accessibility review

    Used to alert the accessibility topic maintainer(s) that an issue significantly affects (or has the potential to affect) the accessibility of Drupal, and their signoff is needed (see the governance policy draft for more information). Useful links: Drupal's accessibility standards, the Drupal Core accessibility gate.

Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

  • The Needs Review Queue Bot โ†’ tested this issue. It either no longer applies to Drupal core, or fails the Drupal core commit checks. Therefore, this issue status is now "Needs work".

    Apart from a re-roll or rebase, this issue may need more work to address feedback in the issue or MR comments. To progress an issue, incorporate this feedback as part of the process of updating the issue. This helps other contributors to know what is outstanding.

    Consult the Drupal Contributor Guide โ†’ to find step-by-step guides for working with issues.

  • Status changed to Needs review over 1 year ago
  • ๐Ÿ‡ท๐Ÿ‡ดRomania amateescu

    Rerolled the patch for 10.1.x. Still needs accessibility review...

  • ๐Ÿ‡ท๐Ÿ‡ดRomania amateescu

    Fixed that test failure, we need to keep the #title property on each row in the list builder because we use it in the off-canvas switcher.

  • ๐Ÿ‡ฎ๐Ÿ‡ณIndia Nayana Ramakrishnan

    Verified the patch #12 and tested it on Drupal version 10.1.x. The patch works fine. Now the active workspace row is highlighted with a strong emphasis on the workspace label, and add an (active) suffix. I have added the before and after screenshots for reference.

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

    Brought this up in the accessibility channel

    From @rkoller

    well a sighted user gets the success message that the workspace was changed successfully. for one in the upper right corner of the admin toolbar (showing the new active workspace) and within the success message box. in voiceover you got only announced the confirmation step do you want to change the workspace. after pressing cmd option space to click the confirm button you immediately get skip to main content announced. no success message. if i would only rely onto my ears it might be troubeling

    Think we need to announce when a workspace is switched or announce the active one one.

  • ๐Ÿ‡ฉ๐Ÿ‡ชGermany rkoller Nรผrnberg, Germany

    while writing up a comment to another issue in the workspaces queue i realized one detail that is missing for screenreader users. After a brief search i've rediscovered this one (already forgot it existed and i've already commented on)

    One detail that might be considered to be added to the current MR which might help screenreader users tremendously is adding the active workspace to the title tag in the head. Currently the title is:

    [h1] | [site name]

    that might be change to:

    [h1] | [active workspace] workspace | [site name]

    so that you get for example when you are on /admin/structure/types

    Content types | Stage workspace | My Drupal site

    The title might become a bit verbose and lengthy but that way all the necessary information is frontloaded, LTR from the specific to the more general (the screen reader user is able to jump off at any point). that means first you have the h1 then the active workspace and then finally the site name. and since screenreaders get the page title announced when loading a page the overall context would be provided satisfying WCAG 2.2. SC 2.4.2. (i've added also the tag wcag242)

Production build 0.69.0 2024