search

namu-theseed / theseed-skin-liberty

Mirror of Liberty skin
GNU General Public License v3.0
10 stars 18 forks source link

Fix syntax #97

Closed navyCarpet closed 7 months ago

navyCarpet commented 7 months ago

currentTheme state가 삭제되었는데 앞으로 삭제되는 것인가요?

namu-theseed commented 7 months ago

currentTheme state가 삭제되었는데 앞으로 삭제되는 것인가요?

삭제되지 않았습니다.

navyCarpet commented 7 months ago

@namu-theseed Client side rendering 상황에는 정상적으로 동작하는 것을 확인했으나 Server side rendering 상황에는 currentTheme state를 기본적으로 제공하지 않아(초기 정의가 안되어있음) data 변동에 따른 rerendering이 되지 않는 상황입니다.

view-source:https://theseed.io/w/%EB%8D%94%EC%8B%9C%EB%93%9C%EC%9C%84%ED%82%A4:%EB%8C%80%EB%AC%B8 (currentTheme가 없음)

navyCarpet commented 7 months ago

+SSR 페이지에서 HTML 렌더링이 정상적으로 되지 않고 있습니다.

navyCarpet commented 7 months ago

다른 스킨에는 문제가 없는 걸 보니 revSelector에 window 때문에 렌더링 오류가 발생하는 것 같기도 한데 오류 로그를 확인할 수가 없어서 확인해주시면 감사하겠습니다..

theseeduser commented 7 months ago

@namu-theseed 나무위키에도 리버티 스킨 색상을 나무위키에 맞게 변경해 주셔야 될 것 같습니다. 모바일에서 상단이 검정색입니다.

navyCarpet commented 7 months ago

@theseeduser umanle측에 스킨 config 설정을 요청하시면 될 것 같습니다.