Skip to content
This repository has been archived by the owner on Jul 29, 2024. It is now read-only.

[FE] refactor: Button focus 시 스타일링 제거 #504

Closed
wants to merge 1 commit into from

Conversation

yogjin
Copy link
Collaborator

@yogjin yogjin commented Oct 1, 2023

🛠️ Issue

✅ Tasks

  • Button 컴포넌트의 focus 스타일 제거

⏰ Time Difference

  • 0.1

📝 Note

focus 시 테두리 스타일은 box-shadow 속성을 통해 만들고 있는데, 배경색이 달라지면 그에 맞춰 구현하기가 좀 까다로워 집니다.
따라서 현재 variant에 따라 다른 동작을 보입니다.

2023-10-01.9.56.37.mov

또한 현재 버튼의 테두리를 인지 할 새 없이 바로 페이지가 이동하는 경우가 대부분이기 때문에, 삭제해도 UX는 크게 변경점이 없다고 생각했습니다.

추가적으로 Button 컴포넌트 자체에 Ripple 컴포넌트를 적용해도 괜찮을 것 같지만, 마찬가지로

현재 버튼의 테두리를 인지 할 새 없이 바로 페이지가 이동하는 경우가 대부분

의 이유때문에 도입은 고려해야할 것 같습니다.

@yogjin yogjin added 🎨 UI 디자인, UI 🖥️ frontend 프론트엔드 작업 labels Oct 1, 2023
@yogjin yogjin self-assigned this Oct 1, 2023
Copy link
Member

@jeonjeunghoon jeonjeunghoon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

직접 버튼을 이리저리 클릭해보면서 테스트해보니
focus 시 효과가 크게 체감되지 않네요!
후에 Ripple 컴포넌트를 도입해보는 것도 좋은 의견인 것같아요!

Copy link
Collaborator

@nangkyeonglim nangkyeonglim left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

하나 궁금한 게 있는데 button에 focus 시 스타일을 주는 이유가 접근성 때문이 아니였나요?
tab 키로 이동할 때 focus 속성 때문에 어떤 버튼에 지금 focus 되고 있는 지 보이는데 스타일링을 제거 한 후 tab 키로 이동하면 어떤 버튼에 focus 되어있는 지가 보이지 않아서요🥲

@yogjin yogjin closed this Oct 10, 2023
@ezzanzzan ezzanzzan deleted the refactor/remove-button-focus-style-500 branch October 10, 2023 05:18
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
🖥️ frontend 프론트엔드 작업 🎨 UI 디자인, UI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Button 컴포넌트 focus 스타일 제거
3 participants