크리스마스 장식 컴포넌트 🎄

@meghna 안녕하세요, 버그 몇 가지를 수정한 pull request를 만들었습니다.

5개의 좋아요

@Canapin 감사합니다. PR이 병합되었습니다. :+1:

4개의 좋아요

훌륭한 테마 컴포넌트를 제공해 주셔서 감사합니다.

사이드바가 활성화되어 있을 때 장식 이미지가 너무 일찍 나타나는 것 같지 않으셨나요? 1450px 폭에서 표시되는 것처럼 보입니다. 사이드바가 있는 경우 1700px 정도가 더 좋을 것 같습니다. 그것이 가능한지 잘 모르겠습니다.

2개의 좋아요

이제 나만의 커스터마이징 대신 이 컴포넌트를 사용 중인데, 둥근 전구를 더 좋아하긴 해요 :blush:

다만 컴포넌트에 여전히 약간의 문제가 있습니다. 전구 밑 부분이 헤더 뒤에 가려져 있어요:

이를 수정하기 위해 PR을 만들었습니다.


둥근 전구를 선호하는 분들을 위해, 다음 CSS를 테마에 추가할 수 있습니다:

.lightrope li {
        width: 12px;
        height: 15px;
    }
}

5개의 좋아요

@Canapin 감사합니다, PR을 병합했습니다. :heart:

5개의 좋아요

스마트폰의 [수치가] 이렇게 낮은 것이 정상인가요?

1개의 좋아요

좋은 지적이에요! 확인해볼게요.


수정: PR을 만들었어요.

3개의 좋아요

배너가 숨겨져 있더라도 이 테마를 Welcome Banner 테마 컴포넌트와 함께 사용하면 수직 위치가 잘못 표시됩니다.

참고:

https://meta.discourse.org/t/welcome-link-banner/218743/61?u=canapin


이 문제를 우회하는 가장 간단한 방법은 다음 CSS를 테마에 추가하는 것입니다:

.above-main-container-outlet.welcome-link-banner-connectors {
    margin-bottom: 0;
}
3개의 좋아요

이 문제를 방지하기 위해 환영 링크 배너 컴포넌트를 업데이트했습니다. @Canapin님, 다시 한번 감사드립니다!

3개의 좋아요

오늘 이 컴포넌트에 업데이트가 있어서, 크리스마스가 끝났기 때문에 여기서는 비활성화되어 있지만 업데이트를 진행했습니다. 그런데 이제 다음과 같은 내용을 확인합니다:

Error: Invalid CSS after "...hadow: 0px math": expected expression (e.g. 1px, bold), was ".div($globe-height," on line 44 of ../../../tmp/discourse_theme_acdcea7663ae5fb94fa2525e54768195/discourse-christmas-decorations/stylesheets/lights.scss from line 2 of common.scss >> box-shadow: 0px math.div($globe-height, 6) $globe-width*2 $globe-sprea --------------------------^

지금 뭔가 깨진 것 같습니다. :wink:

1개의 좋아요

최신 버전의 Discourse를 사용 중이신가요?

1개의 좋아요

여기서는 3.1.0.beta2 [e64d1c4105]를 실행하고 있습니다. 따라서 베타 버전은 맞지만, 최신 커밋은 확실히 아닙니다.

1개의 좋아요

[7eb6223b04]도 마찬가지입니다 :wink:

1개의 좋아요

일단 커밋을 되돌리고, @Falco와 함께 올바른 수정을 진행하겠습니다.

1개의 좋아요

테스트를 하던 중 모바일 뷰에서 모자가 표시되지 않는 문제를 발견했습니다.

@meghna
이 예쁜 디자인 감사해요!

하나만 제안하자면, 양쪽 종에 색을 입히는 건 어떨까요? 지금은 흑백이라 좀 연말 분위기가 부족한 것 같아요. ㅎㅎ 물론 메그나의 선택에 맡기겠습니다…

1개의 좋아요

모바일 뷰에서는 기여자 목록이 없는데, 각 토픽 작성자에게 모자를 씌우면 너무 복잡해 보입니다.
지금은 모바일에서 괜찮아 보이네요. 제 의견입니다.

모든 색상 테마에서 잘 보이도록 흑백으로 선택했습니다. 그래도 PR은 환영합니다 :slight_smile:

3개의 좋아요

또 다른 제안입니다.
라이트 배너 이미지를 변경할 수 있는 설정을 추가하면 더 좋을 것 같아요. 여러 옵션을 제공하거나, 이미지를 랜덤으로 변경하는 방식도 가능합니다.
예를 들어, 지금 이 컴포넌트를 켜도 크리스마스까지 아직 몇 주 남았으니, 이미지를 조금씩 바꿔줄 수 있으면 더 좋겠네요. :slight_smile:

1개의 좋아요

모바일용 모자를 수정해 주실 수 있나요? :grinning:

2개의 좋아요