부정확한 링크 카운터 값 수정 제안

버그인 줄 알았던 것이 사실은 의도된 설계라는 점을 추가로 말씀드리면, 토픽 링크 카운터가 50에서 멈추는 문제가 있습니다:

토픽에 수천 개의 링크가 있더라도 :pensive_face:

그리고:

이 점을 고려할 때, 사용자에게 부정확한 숫자를 표시하는 대신 토픽에 50개 이상의 링크가 있는 경우 단순히 + 기호를 표시할 수는 없을까요?

예를 들어, 이 경우:

다음과 같이 단순히 50+로 표시되면 됩니다:

이렇게 하면 사용자에게 부정확한 정보를 제공하지 않게 됩니다 :thinking:

7개의 좋아요

@lindsey do you feel this is pr-welcome? Seems like a small and easy change (we would over select by 1 and then use that information to decide if we render 50 or 50+)

3개의 좋아요

Agreed, added a pr-welcome to add the + for accuracy.

2개의 좋아요

Thanks @lindsey :grin:

I gave it a try:

https://github.com/discourse/discourse/pull/32093/files

6개의 좋아요

Wow, that was quick!

Thanks @Arkshine :smiley:

1개의 좋아요

Hey, I have a question: From what I have seen, all the other components, like views and likes, have the full number. For the sake of design consistency, we should either add the plus to the other cases or add the full number to the link count.

1개의 좋아요

Is there a process for getting this PR merged @Arkshine ? :thinking:

1개의 좋아요

An approved PR is not necessarily merged immediately. There might be some internal concern or discussion about it. Since it’s low priority, let’s wait!

1개의 좋아요

Ok, thanks, just wasn’t sure what the process was :slight_smile:

1개의 좋아요

It is merged now, thanks @Arkshine

3개의 좋아요

Thanks again for working on this one @Arkshine :partying_face:

2개의 좋아요