mui / mui/material-ui

[Button] Focus effect padding

Open
#33,303 13 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

design: material scope: button type: bug
Dominant language
JavaScript
Stars
99.1k
Forks
32.5k
Avg merge
2d 17h
Merged PRs (30d)
106

Description

Duplicates
  • I have searched the existing issues
Latest version
  • I have tested the latest version
Current behavior 😯

The focus effect does not cover the hole button

Captura de Pantalla 2022-06-26 a la(s) 09 50 58
Expected behavior 🤔

Focus must be full width

Steps to reproduce 🕹

Steps:

  1. Add a button
  2. Focus by using the tab key
Context 🔦

No response

Your environment 🌎

No response

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Reproduce the issue with a Material UI Button and the tab key, comparing the focus effect with the button's full bounds. Locate the Button focus styling from that reproduction and verify that the effect covers the complete button width when the change is done.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript, react
Domain
accessibility, frontend
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.