Skip to content

구현 과제 - 로그인/회원가입 모달 - 유진#36

Open
yujin1228 wants to merge 2 commits intomainfrom
11-yuJin
Open

구현 과제 - 로그인/회원가입 모달 - 유진#36
yujin1228 wants to merge 2 commits intomainfrom
11-yuJin

Conversation

@yujin1228
Copy link
Copy Markdown

로그인/회원가입 모달 과제를 구현 했습니다.

@YennieJ
Copy link
Copy Markdown
Collaborator

YennieJ commented Aug 8, 2023

와우, 유진님
모달로 하셨군요..!!!!👍👍
button에 타입만 추가하시고 저도 구현중이긴한데, focus-visible도 신경써보시면 좋을꺼같아요!
js는 제가 구동은 안시켜봤지만 X버튼 누를때랑 똑같이 만들면 되지 않을까요?
그리고 모달 창이 닫힐 때, 주소를 원래대로 돌리는 것도 좋을꺼 같습니다 :)
고생하셨습니다👍

@yujin1228
Copy link
Copy Markdown
Author

와우, 유진님 모달로 하셨군요..!!!!👍👍 button에 타입만 추가하시고 저도 구현중이긴한데, focus-visible도 신경써보시면 좋을꺼같아요! js는 제가 구동은 안시켜봤지만 X버튼 누를때랑 똑같이 만들면 되지 않을까요? 그리고 모달 창이 닫힐 때, 주소를 원래대로 돌리는 것도 좋을꺼 같습니다 :) 고생하셨습니다👍

코멘트 너무 감사해요 예진님! focus-visible을 생각못했네요...! 그부분도 포함해서 접근성측면에서도 한번 더 생각해봐야겠어요 감사해요🥰
혹시 모달 창이 닫힐 때 주소는 어떤걸 말씀하시는걸까요? 어느부분 주소가 바뀌는지 모르겠어서요..!

@YennieJ
Copy link
Copy Markdown
Collaborator

YennieJ commented Aug 8, 2023

와우, 유진님 모달로 하셨군요..!!!!👍👍 button에 타입만 추가하시고 저도 구현중이긴한데, focus-visible도 신경써보시면 좋을꺼같아요! js는 제가 구동은 안시켜봤지만 X버튼 누를때랑 똑같이 만들면 되지 않을까요? 그리고 모달 창이 닫힐 때, 주소를 원래대로 돌리는 것도 좋을꺼 같습니다 :) 고생하셨습니다👍

코멘트 너무 감사해요 예진님! focus-visible을 생각못했네요...! 그부분도 포함해서 접근성측면에서도 한번 더 생각해봐야겠어요 감사해요🥰 혹시 모달 창이 닫힐 때 주소는 어떤걸 말씀하시는걸까요? 어느부분 주소가 바뀌는지 모르겠어서요..!

주소창에 # <= 생길때가 있어서요! js구동을 안시켜봐서 정확하지 않은 내용을 말씀드려서 죄송해유 :(

@yujin1228 yujin1228 added the JS label Aug 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants