Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sandbox mode: cursor for debug logging hover state indicates debug logging can be disabled #8138

Closed
csmcneill opened this issue Feb 6, 2024 · 1 comment · Fixed by #8431
Assignees
Labels
category: core WC Payments core related issues, where it’s obvious. component: settings Issues related to Settings focus: account lifecycle priority: medium The issue/PR is medium priority—non-critical functionality loss, minimal effect on usability size: small The issue is sized small. type: bug The issue is a confirmed bug.

Comments

@csmcneill
Copy link
Contributor

Describe the bug

When a merchant tries to disable the Credit / Debit card payment method, the mouse cursor changes to one with a no symbol, which indicates that they are unable to modify this setting.

When WooPayments is in sandbox mode, there is an additional item that cannot be disabled: debug logging. However, the cursor does not change to reflect that this is an option that cannot be disabled.

To Reproduce

  1. Onboard in sandbox mode.
  2. Navigate to Payments > Settings.
  3. Hover over the checkbox for Credit / Debit card (Required).
  4. Note the cursor used.
  5. Scroll down and hover over the checkbox for Sandbox mode is active so logging is on by default..
  6. Note that the cursor is different than step 4.
  7. Hover over another option that can be enabled/disabled (e.g., Enable Multi-Currency)
  8. Note that the cursor is the same as when hovering over Sandbox mode is active so logging is on by default..

Actual behavior

When in sandbox mode, the cursor does not change when hovering over the debug logging option, which is inconsistent with other options that are enabled but cannot be disabled.

Screenshots

CleanShot 2024-02-05 at 17 58 33

Expected behavior

Hovering over the debug logging option while in sandbox mode displays a mouse cursor that mirrors other options that cannot be enabled.

@csmcneill csmcneill added type: bug The issue is a confirmed bug. priority: low The issue/PR is low priority—not many people are affected or there’s a workaround, etc. component: settings Issues related to Settings labels Feb 6, 2024
@zmaglica zmaglica added the category: core WC Payments core related issues, where it’s obvious. label Feb 6, 2024
@zmaglica
Copy link
Contributor

zmaglica commented Feb 8, 2024

This issue impacts <team responsibility WooPayments Settings page, so assigning to Moltres (based on team responsibilities Pc2DNy-3z-p2) @daquinons .
Assigning as part of Gamma Triage process PcreKM-yM-p2.

@daquinons daquinons added needs prioritisation Triage finished and issues are ready for the following processing. and removed priority: low The issue/PR is low priority—not many people are affected or there’s a workaround, etc. labels Feb 9, 2024
@vladolaru vladolaru added priority: medium The issue/PR is medium priority—non-critical functionality loss, minimal effect on usability and removed needs prioritisation Triage finished and issues are ready for the following processing. labels Feb 15, 2024
@dmallory42 dmallory42 assigned dmallory42 and unassigned dmallory42 Feb 26, 2024
@anu-rock anu-rock added the size: small The issue is sized small. label Mar 15, 2024
@elazzabi elazzabi self-assigned this Mar 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: core WC Payments core related issues, where it’s obvious. component: settings Issues related to Settings focus: account lifecycle priority: medium The issue/PR is medium priority—non-critical functionality loss, minimal effect on usability size: small The issue is sized small. type: bug The issue is a confirmed bug.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

8 participants