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

Hitbox focus on components #4633

Closed
7 tasks done
geospatialem opened this issue May 27, 2022 · 12 comments
Closed
7 tasks done

Hitbox focus on components #4633

geospatialem opened this issue May 27, 2022 · 12 comments
Assignees
Labels
4 - verified Issues that have been released and confirmed resolved. a11y Issues related to Accessibility fixes or improvements. ArcGIS Living Atlas Issues logged by ArcGIS Living Atlas team members. design Issues that need design consultation prior to development. enhancement Issues tied to a new feature or request. estimate - 5 A few days of work, definitely requires updates to tests. p - medium Issue is non core or affecting less that 60% of people using the library ready for dev Issues ready for development implementation.

Comments

@geospatialem
Copy link
Member

geospatialem commented May 27, 2022

Summary

Complete an audit of all the components to reduce block elements in focus. For instance, the color picker selector.

This came up as an action as part of #1359.

Actual Behavior

Some components elements are depicted as block elements when focused.

Expected Behavior

Focus highlights the extent of the element.

Components

Reproduction Sample

https://developers.arcgis.com/calcite-design-system/components/color-picker/#sample

Reproduction Steps

  1. Open the sample page.
  2. Navigate into the color picker using keyboard keys (tab) to see the focus highlighting a box for the color picker selector.

Reproduction Version

beta.82

Working W3C Example/Tutorial

N/A

Relevant Info

N/A

Regression?

No response

@geospatialem geospatialem added bug Bug reports for broken functionality. Issues should include a reproduction of the bug. a11y Issues related to Accessibility fixes or improvements. p - high Issue should be addressed in the current milestone, impacts component or core functionality 0 - new New issues that need assignment. needs triage Planning workflow - pending design/dev review. labels May 27, 2022
@geospatialem geospatialem added this to the Back burner milestone Jun 1, 2022
@geospatialem geospatialem removed the needs triage Planning workflow - pending design/dev review. label Jun 2, 2022
@geospatialem geospatialem added the estimate - 3 A day or two of work, likely requires updates to tests. label Dec 1, 2022
@geospatialem geospatialem self-assigned this Dec 1, 2022
@geospatialem geospatialem added 1 - assigned Issues that are assigned to a sprint and a team member. and removed 0 - new New issues that need assignment. labels Dec 7, 2022
@geospatialem
Copy link
Member Author

Audit results:

1. color-picker

Includes boxy focus on the following elements of the component (displayed in one image below for clarity):

  • color handle
  • swatch handle and
  • RGB and HSV groupings (depicted in a 1px black rectangle)
    color picker screenshot with multiple focus combinations

2. link

The hitbox needs a bit more padding, it's a bit hard to discern on the left/right, includes spacing where links could extend multiple lines.
link focus screenshot simple
link focus screenshot multiple lines

3. pagination cc #5113

The focus and selection conflict, maybe a differing focus, such as a circle would help differentiate?
pagination focus screenshot

4. rating

There is a boxed focus so noting as part of the audit, but it seems to be the best look for the component.
rating focus screenshot

5. stepper and stepper-item

There is a boxed focus so noting as part of the audit, but it seems like a low priority to mitigate, if at all.
stepper focus screenshot

6. tabs

There is a boxed focus so noting as part of the audit, but it seems to be the best look for the component.
tabs focus screenshot

cc @driskull @macandcheese - Based on the above results, should we aim to accomplish any of the above for the January release? Thinking link could be a potential candidate (the others likely post 1.0).

@geospatialem geospatialem added 0 - new New issues that need assignment. and removed 1 - assigned Issues that are assigned to a sprint and a team member. labels Jan 16, 2023
@geospatialem geospatialem removed their assignment Jan 16, 2023
@geospatialem
Copy link
Member Author

Adding to the February priorities, but the milestone could change depending on prioritization to tackle the above components hitbox focus.

@geospatialem geospatialem changed the title Audit: Hitbox focus on components Hitbox focus on components Jan 16, 2023
@geospatialem
Copy link
Member Author

Adding one additional component per feedback from Living Atlas:

7. switch

Boxed focus, maybe could be similar to FAB's focus (radius)?
image

@geospatialem geospatialem added ArcGIS Living Atlas Issues logged by ArcGIS Living Atlas team members. design Issues that need design consultation prior to development. 1 - assigned Issues that are assigned to a sprint and a team member. and removed 0 - new New issues that need assignment. labels Feb 10, 2023
@geospatialem
Copy link
Member Author

@ashetland @SkyeSeitz We've gotten some feedback from a few teams regarding our focus hitbox. I performed a rough audit late last year, but would be good to get a design eye on this one, too. Could you both take a peek, and determine if the above and any other components we could the focus states on in the next few weeks?

@geospatialem geospatialem added estimate - 5 A few days of work, definitely requires updates to tests. and removed estimate - 8 Requires input from team, consider smaller steps. labels Jul 3, 2023
@anveshmekala anveshmekala self-assigned this Jul 18, 2023
@geospatialem geospatialem added 1 - assigned Issues that are assigned to a sprint and a team member. 2 - in development Issues that are actively being worked on. and removed 0 - new New issues that need assignment. 1 - assigned Issues that are assigned to a sprint and a team member. labels Jul 24, 2023
anveshmekala added a commit that referenced this issue Jul 31, 2023
**Related Issue:** #4633 

## Summary

This PR will replace the block focus outline of `calcite-switch` with a
rounded one.

Before the change:


![25C47646-4B10-45A7-94E3-3CD884D35370_4_5005_c](https://github.com/Esri/calcite-design-system/assets/88453586/02313b88-8c2e-4f51-9b45-7b9631dff59f)


After the change:


![92B6D3BC-5B7E-4041-B424-BAB2F89E3C32_4_5005_c](https://github.com/Esri/calcite-design-system/assets/88453586/4d47a41f-dd8d-43f2-8390-1b89a9120440)
anveshmekala added a commit that referenced this issue Aug 1, 2023
**Related Issue:** #4633 

## Summary
This PR will replace the existing focus outline for thumb in
`calcite-color-picker` to rounded.

After the change:

![494C307B-315E-4842-93C5-911C6F3810A2](https://github.com/Esri/calcite-design-system/assets/88453586/02b8daed-05c8-4d98-ad1b-504fe9d58fd3)

![ED840D87-BFA5-4602-8D1C-51FF78CD04E2](https://github.com/Esri/calcite-design-system/assets/88453586/4599dc2a-aad4-4a48-b6fd-57277a9b0884)
@geospatialem
Copy link
Member Author

geospatialem commented Aug 1, 2023

Most of the above have been tackled, with rating being prioritized for the August release.

Components verified in 1.5.0-next.30:

  • color-picker:
    verify-color-picker-hitbox-focus

  • link:
    image

  • switch:
    image

benelan pushed a commit that referenced this issue Aug 3, 2023
**Related Issue:** #4633 

## Summary

This PR will replace the block focus outline of `calcite-switch` with a
rounded one.

Before the change:


![25C47646-4B10-45A7-94E3-3CD884D35370_4_5005_c](https://github.com/Esri/calcite-design-system/assets/88453586/02313b88-8c2e-4f51-9b45-7b9631dff59f)


After the change:


![92B6D3BC-5B7E-4041-B424-BAB2F89E3C32_4_5005_c](https://github.com/Esri/calcite-design-system/assets/88453586/4d47a41f-dd8d-43f2-8390-1b89a9120440)
benelan pushed a commit that referenced this issue Aug 3, 2023
**Related Issue:** #4633 

## Summary
This PR will replace the existing focus outline for thumb in
`calcite-color-picker` to rounded.

After the change:

![494C307B-315E-4842-93C5-911C6F3810A2](https://github.com/Esri/calcite-design-system/assets/88453586/02b8daed-05c8-4d98-ad1b-504fe9d58fd3)

![ED840D87-BFA5-4602-8D1C-51FF78CD04E2](https://github.com/Esri/calcite-design-system/assets/88453586/4599dc2a-aad4-4a48-b6fd-57277a9b0884)
anveshmekala added a commit that referenced this issue Aug 26, 2023
**Related Issue:** #4633 ,
#6642

## Summary

This PR will add focus outline for `calcite-rating` component on click.
@anveshmekala anveshmekala added the 3 - installed Issues that have been merged to master branch and are ready for final confirmation. label Aug 26, 2023
@github-actions
Copy link
Contributor

Installed and assigned for verification.

@geospatialem geospatialem added 4 - verified Issues that have been released and confirmed resolved. and removed 2 - in development Issues that are actively being worked on. 3 - installed Issues that have been merged to master branch and are ready for final confirmation. labels Aug 27, 2023
@geospatialem
Copy link
Member Author

The final component, rating has been mitigated on click similar to keyboard use, which has been verified on the main branch.

Elijbet pushed a commit that referenced this issue Aug 30, 2023
**Related Issue:** #4633 ,
#6642

## Summary

This PR will add focus outline for `calcite-rating` component on click.
@brittneytewks brittneytewks added the design Issues that need design consultation prior to development. label Dec 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4 - verified Issues that have been released and confirmed resolved. a11y Issues related to Accessibility fixes or improvements. ArcGIS Living Atlas Issues logged by ArcGIS Living Atlas team members. design Issues that need design consultation prior to development. enhancement Issues tied to a new feature or request. estimate - 5 A few days of work, definitely requires updates to tests. p - medium Issue is non core or affecting less that 60% of people using the library ready for dev Issues ready for development implementation.
Projects
None yet
Development

No branches or pull requests

7 participants