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

refactor : header, heading, 스크린리더 수정 #74

Merged

Conversation

yejilee0714
Copy link
Collaborator

issue : #72

전달 사항

개발 내용

  • header, heading, 스크린리더 수정

추후 작업

참고 이미지

@yejilee0714 yejilee0714 self-assigned this Jun 26, 2023
Copy link
Member

@ebokyung ebokyung left a comment

Choose a reason for hiding this comment

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

병합할때 문제가 있었나봐요 ㅜㅜ 코멘트 부분 수정해서 다시 올려볼까요 ~~

Comment on lines 24 to 25
<a href="./chat_room.html" class="user-chat-content new-chat">
<span class="a11y-hidden"></span>
Copy link
Member

Choose a reason for hiding this comment

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

맞나요

<button id="upload-btn" class="size-ms btn-save" tabindex="0" disabled>업로드<span class="a11y-hidden">버튼</span></button>
<h1 class="a11y-hidden">게시물 업로드</h1>
<button id="upload-btn" class="size-ms btn-save" disabled>업로드<span class="a11y-hidden">하기</span></button>
<button class="btn-back"><img src="../assets/icon/icon-arrow-left.svg" alt="뒤로가기 버튼"></button>
Copy link
Member

Choose a reason for hiding this comment

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

여기 img 태그 안뺐네용

js/following.js Outdated
Comment on lines 5 to 36
const followers = document.querySelector('.follow-list');
let reqCnt = 0;


// 1. 내가 내 팔로워 목록을 보는지, 내가 다른 사용자 팔로워 목록을 보는지
const viewMyFollowerList = accountname === myAccountname ? true : false;


// 무한 스크롤
window.addEventListener("scroll", async () => {
if (getScrollTop() >= getDocumentHeight() - window.innerHeight) {
console.log('바닥이당! 데이터 불러올게 기다려!')
throttle(makeList(await getFollowingList()), 1000)
if (viewMyFollowerList) {
throttle(makeMyFollowerList(await getFollowerList()), 1000)
} else {
throttle(makeUserFollowerList(await getFollowerList()), 1000)
}
};
})

// 버튼 이벤트
followings.addEventListener('click', async (e) => {
if (e.target.classList.contains('btn-follow')) {
// 클릭한 요소의 사용자계정
const clickedAccount = e.target.closest('li').querySelector('a').href.split('?accountName=')[1];
async function run() {
const data = await getFollowerList();
if (viewMyFollowerList) makeMyFollowerList(data);
else makeUserFollowerList(data);
};

if (e.target.classList.contains('opposite')) {
// 언팔로우 기능
await deleteFollow(clickedAccount);
run()

e.target.classList.remove('opposite');
e.target.innerHTML = `팔로우<span class="a11y-hidden">하기 버튼</span>`;
} else {

// event
if (viewMyFollowerList) {
// (팔로우(하기) 버튼만 가능, 팔로워를 삭제하는 버튼은 있지만 기능은 없음 disabled 처리)
Copy link
Member

Choose a reason for hiding this comment

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

팔로잉.js 파일 전체가 병합 이상하게 됐네요 ㅠㅠ

js/home.js Outdated
}
if (item.image) {
liNode.querySelector('.post-edit a').insertAdjacentHTML('beforeend', `<div class="img-cover"><img class="post-img" src="${checkImageUrl(item.image,'post')}" alt="게시물 사진"></img></div>`)
liNode.querySelector('.post-edit a').insertAdjacentHTML('beforeend', `<div class="img-cover"><img class="post-img" src="${item.image}" alt=""></img></div>`)
Copy link
Member

Choose a reason for hiding this comment

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

checkImageUrl 적용전 코드네요

<a class="profile-img img-cover" href="./profile_info.html?accountName=${userData[i].accountname}">
<img src="${checkImageUrl(userData[i].image,'profile')}" alt="프로필사진">
<span class="a11y-hidden">${userData[i].username}의 프로필 보기</span>
<img src="${userData[i].image}" alt="">
Copy link
Member

Choose a reason for hiding this comment

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

checkImageUrl 적용전 코드네요

js/postDetail.js Outdated
</div>`)
postInfoSec.querySelector('.btn-like').addEventListener('click',handleLike)
}

// 더보기 버튼
const btnOption = document.createElement('button');
btnOption.setAttribute('class', 'btn-option');
btnOption.setAttribute('tabindex', '0');
btnOption.innerHTML = `<img src="../assets/icon/icon-more-vertical.svg" alt="더보기 버튼">`;
btnOption.innerHTML = `<span class="a11y-hidden">게시물 옵션</span><img src="../assets/icon/icon-more-vertical.svg" alt="">`;
Copy link
Member

Choose a reason for hiding this comment

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

이미지 태그 안뺐어요

js/postDetail.js Outdated
<p class="comment-text">${i.content}</p>
<button class="btn-more" tabindex="2" data-commentId=${i.id}><img src="../assets/icon/icon-more-vertical.svg" alt="더보기 버튼"></button>`;
<h3 class="comment-text">${i.content}</h3>
<button class="btn-more" data-commentId=${i.id}><span class="a11y-hidden">댓글 옵션</span><img src="../assets/icon/icon-more-vertical.svg" alt=""></button>`;
Copy link
Member

Choose a reason for hiding this comment

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

이미지 태그 안뺐어요

productName.insertAdjacentText('beforeend',`${item.itemName}`)

const productImg = document.createElement('img')
productImg.className='product-img'
productImg.setAttribute('src',`${checkImageUrl(item.itemImage,'post')}`)
productImg.setAttribute('alt','상품 이미지')
const itemSrc = item.itemImage.includes("https://api.mandarin.weniv.co.kr",) ? item.itemImage : "https://api.mandarin.weniv.co.kr/1687488082591.png"
Copy link
Member

Choose a reason for hiding this comment

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

checkImageUrl 적용전 코드네요


const userImgImg = document.createElement('img');
userImgImg.src = checkImageUrl(item.author.image,'profile');
userImgImg.alt = '프로필사진';
userImgImg.src = item.author.image;
Copy link
Member

Choose a reason for hiding this comment

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

checkImageUrl 적용전 코드네요

@@ -253,7 +248,7 @@ function updatePost(post_data) {
const btnOption = document.createElement('button');
btnOption.className = 'btn-option';
btnOption.innerHTML = `
<img src="../assets/icon/icon-more-vertical.svg" alt="더보기 버튼">
<span class="a11y-hidden">게시물 옵션</span><img src="../assets/icon/icon-more-vertical.svg" alt="">
Copy link
Member

Choose a reason for hiding this comment

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

이미지 태그 안뺐어요

@jypark38
Copy link
Collaborator

고생하셨습니다

@jypark38 jypark38 merged commit 8d90f5d into FRONTENDSCHOOL5:development Jun 27, 2023
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.

None yet

3 participants