Project

General

Profile

Actions

Bug #22172

closed

"CANCEL" menu item is all caps, looks weird

Added by Brett Smith about 2 months ago. Updated 17 days ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
Workbench2
Story points:
-
Release:
Release relationship:
Auto

Description

The context menu for a running process lists the "CANCEL" action in all caps. I saw this on pirca, but it seems to come from source:services/workbench2/src/components/multiselect-toolbar/ms-menu-actions.ts and still be true in main:

This seems weird to me. I'm not aware of anything else in Workbench that does this. If we want to emphasize the destructiveness of this action, it seems like it would be more appropriate to use color and/or have a confirmation dialog.


Files

clipboard-202410071022-5yxfj.png (39.4 KB) clipboard-202410071022-5yxfj.png Brett Smith, 10/07/2024 02:22 PM

Subtasks 1 (0 open1 closed)

Task #22250: Review 22172-cancel-button-capitalizationResolvedStephen Smith10/30/2024Actions
Actions #1

Updated by Peter Amstutz 29 days ago

  • Target version set to Development 2024-11-06 sprint
Actions #2

Updated by Peter Amstutz 29 days ago

  • Assigned To set to Stephen Smith
Actions #3

Updated by Stephen Smith 23 days ago

Changes at arvados|6dca2d4c6fd19a6979b763592f5166515df5eaa6 branch 22172-cancel-button-capitalization
Tests developer-run-tests-services-workbench2: #1323

  • All agreed upon points are implemented / addressed.
    • Changed all instances of cancel menu option capitalization to "Cancel"
    • I left the cancel button next to the status badge as all caps since that's not next to other menu options
  • Anything not implemented (discovered or discussed during work) has a follow-up story.
    • none
  • Code is tested and passing, both automated and manual, what manual testing was done is described
    • passed normal tests
  • Documentation has been updated.
    • n/a
  • Behaves appropriately at the intended scale (describe intended scale).
    • no change
  • Considered backwards and forwards compatibility issues between client and server.
    • no change
  • Follows our coding standards and GUI style guidelines.
    • no significant ui changes
Actions #4

Updated by Stephen Smith 23 days ago

  • Status changed from New to In Progress
Actions #5

Updated by Lisa Knox 22 days ago

LGTM

Actions #6

Updated by Stephen Smith 22 days ago

  • Status changed from In Progress to Resolved
Actions #7

Updated by Peter Amstutz 17 days ago

  • Release set to 70
Actions

Also available in: Atom PDF