moonsangjin / clique Goto Github PK

View Code? Open in Web Editor NEW

7.0 7.0 1.0 24.45 MB

📌 스크롤 위치를 기억하는 북마크 서비스 클리크(Clique)

JavaScript 98.48% HTML 1.11% CSS 0.09% TypeScript 0.32%

clique's Introduction

clique's People

Contributors

Stargazers

Watchers

Forkers

goldfiish

clique's Issues

bug : NewTab에서 signout 동작시 folderRow가 그대로 보여지는 것

합친 정보들 폴더 api와 연결

각 탭에서 scrollY 값 popUp으로 가져오기

favicon bug

아래처럼 image url이 아니고, 데이터를 직접 string으로 저장하는 형태의 경우
서버에 url field로 저장되지 않는다.

data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAACAAAAAgCAYAAABzenr0AAAHTElEQVRYR8WXe3BU1R3Hv/e5j+wmmxchIQltIQkJEEmoQCQyFhGKgA80SjvYWqcF/7BS2qnUFut0rAxOp8y0apVBOhakjAijUyG0IE2M5WELiUMk5EGUuCEJu3nsI3v3Ps69p3POBqpl2

Popup.js 파일을 리팩토링한다.

Popup.js

파일 이름 수정해주세요. (저희가 사용하는 IconMenu.js는 어떤가요?)
불필요한 주석 제거해주세요. (message passing 코드 기록이 필요하면, 노션 등 다른 채널 활용해주세요.)
질문: styles component에서 변수 이름을 소문자로 하면 코드 작성할 때도 태그 이름을 소문자로 작성하게 되는거죠?

const Wrapper = styled.div``;

line 24: 변수 이름 제안

const [openPageInfoList, setOpenPageInfoList] = useState([]);

line 26: searchUrl함수 refactoring 제안

const getOpenPageInfo = () => {
    chrome.tabs.query({ lastFocusedWindow: true }, (openPages) => {
      const openPageInfoList = openPages.map((page) => {
        favIconUrl: page.favIconUrl,
        title: page.title,
        url: page.url,
      });

      setTabUrl(openPageInfoList);
    });
  };