pfaffman
(Jay Pfaffman)
1
모더레이터 아래에 기존 about 통계를 표시하고 우측 사이드바의 새 통계를 숨깁니다.
이 내용을 작성했을 당시 우측 사이드바의 통계가 단일 클래스로 감싸져 있지 않아 last-child를 사용하여 <h3> 헤더를 숨겨야 했습니다. 이것이 얼마나 취약할지 확실하지 않습니다.
이 테마 컴포넌트의 개발 및 첫 1년간의 지속적인 유지보수를 위해 비용을 지불해 주신 @Richie와 Grey Arrows Drone Club에 감사드립니다.
16개의 좋아요
pfaffman
(Jay Pfaffman)
3
아는 사람들에게는 그렇지 않죠. 특히 저보다 더 잘 아는 분들에게는 더더욱요! 여기서 도움을 요청하지 않고 제가 무언가를 끝까지 해본 적이 거의 없거든요. 심지어 결국 도움을 주는 쪽이 바로 제가 될 때도요. 
7개의 좋아요
첫째, "레거시"에서 실제로 유용한 것이 하나 있다는 것 자체가 좋은데, 무슨 이유에서인지 제거되는 경우가 많거든요
이걸 유지해 주셔서 감사합니다.
둘째, 저도 그 경험을 해 봤습니다 … 제가 질문을 올린 후 하루 뒤에야 스스로 문제를 해결하는 경우가 많아서, 그 마음 잘 이해합니다!
9개의 좋아요
Richie
(Richie Rich)
5
4개의 좋아요
ToddZ
6
좋네요 – 저는 last-child 대신 nth-of-type을 사용해 접근했습니다. 새로운 통계는 숨기되 기존 통계를 복원하지 않으려는 분들을 위해, 여기 작은 CSS 클립(현재)이 작동합니다:
pfaffman
(Jay Pfaffman)
7
고맙습니다, @ToddZ! CSS에 대해 알았다면 nth- 버전을 사용했을 텐데, 내일 아니면 다음 주에 당신의 코드를 통합할 거예요.
아마도 적절한 클래스를 포함해서 PR을 제출하면, 이런 CSS 트릭이 필요 없을지도 모르겠네요.
ToddZ
8
해킹이 하나만 더 나은지 모르겠어요. 잘만 작동하면 되는 거니까요 
하지만 전체를 감싸는 stats 클래스가 있으면 훨씬 편할 거예요.
2개의 좋아요
Lilly
(Lillian )
9
좋은 작업입니다! 
설명을 “새로운 정보 통계를 클래식 뷰로 대체” 또는 이와 유사한 문구로 변경하는 것이 좋을 것 같습니다.
2개의 좋아요
Richie
(Richie Rich)
10
아마 이렇게 하는 게 좋겠네요:
새로운 통계 정보 페이지를 클래식 통계 테이블로 대체
즉, "새로운"이라는 단어를 빼는 겁니다. 새로운 정보 페이지가 오래 ‘새로운’ 상태로 유지되지는 않으니까요 
2개의 좋아요
putty
(putty)
11
안녕하세요 @pfaffman,
이 컴포넌트에 이 변경 사항으로 인해 비권장(deprecation) 경고가 표시되기 시작했습니다. 업데이트를 위한 PR을 검토하고 승인해 주시겠습니까?
2개의 좋아요
pfaffman
(Jay Pfaffman)
13
discourse-theme-skeleton에서 최신 내용을 가져와서 린팅 오류를 수정했습니다. 동작하는지 테스트하지는 않았는데, 주로 기존 Classic Statistics가 어떤 모습이어야 하는지 잘 모르기 때문입니다. 
Richie
(Richie Rich)
14
필요하시면 확인하실 수 있도록, 저는 여전히 자부심을 갖고 Discourse에서 이걸 운영하고 있습니다 
2개의 좋아요
pfaffman
(Jay Pfaffman)
15
좋은 정보네요! (아, 근데 지금은 관리자가 아니라서 그냥 믿을 수밖에 없겠네요!) 방금 제가 바꾼 게 더 나아졌는지 나빠졌는지 직접 확인해 보세요. 
Richie
(Richie Rich)
16
방금 업데이트를 완료했습니다. 다행히 문제가 없었고 보고할 변경 사항도 없습니다 
1개의 좋아요