search

finos / a11y-theme-builder

DesignOps toolchain theme builder for accessibility inclusion using Atomic Design.
Apache License 2.0
46 stars 69 forks source link

[TB] Minimum Target Area atom broken #990

Closed aaronreed708 closed 3 months ago

aaronreed708 commented 3 months ago

Problem/Concern

When I bring up the Minimum Target Area atom, I see stack exceptions in the display.

Screenshot 2024-08-09 at 11 09 08 AM

This appears to be an issue where the index on an array is incorrect when trying to build the list of selectable values. Probably introduced when I made all of those lint changes.

Proposed Solution

Fix the index being used in the code to build the list of selectable values.

toni-sharpe commented 3 months ago

The merged PR suggests this issue can be closed.

In the PR decrioption, try Fixes #<issue_no> to have the PR close automatically close the issue

aaronreed708 commented 3 months ago

verified on dev today. Closing. Thanks for the reminder, @toni-sharpe!