Skip to content

css reset style #2

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

hsecode
Copy link

@hsecode hsecode commented Aug 23, 2021

안녕하세요. 마크업 가이드 강의 수강중인 수강생입니다.
실습과제를 겸하여 PR 드립니다.

  1. class에 의한 리셋을 추가하였습니다.
    [class]:where(iframe){overflow: hidden;}
    [class]:where(textarea){resize: vertical;}
    Reset by Type selectors 로 img 태그와 함께 추가하려하다 iframe과 textarea는 비교적 적게 사용되는 타입이라고 생각되어
    고민끝에 :where 를 사용하여 추가했습니다.

  2. body에 -webkit-text-size-adjust: none; text-size-adjust: none; 을 추가하였습니다.

이 외에도 추가하고 싶었으나 잘 사용되지 않거나(hr, u, b 등), 모든 프로젝트 들에 보편적으로 적용하기 어렵거나(input type 별 스타일들,:-ms-clear 등), 이미 작성되어있어 위의 항목만 우선적으로 PR드리게 되었습니다.

습관적으로 사용하던 css reset style에 대해 고민해볼 수 있는 기회를 가질 수 있게되어 뜻깊은 챕터였습니다.
마크업 가이드 강의를 들으며 많은 배움을 얻고있습니다.

부족한 부분이나 간과한 부분이 있다면 피드백 부탁드리며,
좋은 강의를 제작해주셔서 감사합니다.

@naradesign
Copy link
Owner

안녕하세요. 마크업 가이드 강의 수강중인 수강생입니다.
실습과제를 겸하여 PR 드립니다.

  1. class에 의한 리셋을 추가하였습니다.
    [class]:where(iframe){overflow: hidden;}
    [class]:where(textarea){resize: vertical;}
    Reset by Type selectors 로 img 태그와 함께 추가하려하다 iframe과 textarea는 비교적 적게 사용되는 타입이라고 생각되어
    고민끝에 :where 를 사용하여 추가했습니다.
  2. body에 -webkit-text-size-adjust: none; text-size-adjust: none; 을 추가하였습니다.

이 외에도 추가하고 싶었으나 잘 사용되지 않거나(hr, u, b 등), 모든 프로젝트 들에 보편적으로 적용하기 어렵거나(input type 별 스타일들,:-ms-clear 등), 이미 작성되어있어 위의 항목만 우선적으로 PR드리게 되었습니다.

습관적으로 사용하던 css reset style에 대해 고민해볼 수 있는 기회를 가질 수 있게되어 뜻깊은 챕터였습니다.
마크업 가이드 강의를 들으며 많은 배움을 얻고있습니다.

부족한 부분이나 간과한 부분이 있다면 피드백 부탁드리며,
좋은 강의를 제작해주셔서 감사합니다.

@hsecode 정성스러운 PR을 보내주셔서 감사합니다. 보내주신 코드는 바로 병합해도 좋을만큼 내용이 좋아 보입니다. 다만 교육 목적으로 만든 저장소라서 PR을 병합하지 않고 있는 점 양해 부탁드립니다. @hsecode 님의 PR은 다른 수강생 분들께도 좋은 영감을 주었을 것으로 기대가 됩니다. 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants