רכיבי נושא כותרות קטגוריות דיון

נהדר! חזר למצב מצוין. תודה על התגובה המדהימה, ועל שהוספת שיפור כחלק מהעסקה!

לייק 1

היי נייתן, אני סקרן, האם עדיף שהקישור “קרא עוד” יהיה מיושר עם התיאור, או מוטמע בסופו?

I think that it would be best inline at the end of the text.

This would be much more compact / efficient, and more consistent with pinned posts (I think):

לייק 1

@nathank I have added this feature!

This PR adds 2 new features:

  1. More paragraphs to the category description. This adds a new setting, show_full_category_description (default false) to show the full post contents of the About category topic.
  2. Inline ‘Read more’ link. This adds a new setting inline_read_more (default false) to append the ‘Read more’ link at the end of the category description, instead of a few lines down.

Things to note:

  • The inline ‘Read more’ link is not added to the end of the category description when show_full_category_description is true
  • There is no override regarding the category descriptions: you can set both show_category_description and show_full_category_description to true, and both descriptions will be shown. This may be addressed in a later PR.

Please post back here if you face any issues.

4 לייקים

מישהו אחר נתקל בבעיות בגרסה 3.5.0 שבהן סמלי קטגוריות ואימוג’ים אינם מוצגים? גם כאשר סמל נעילת הקטגוריה מופעל, הוא אינו מוצג גם כן.

Hi there, could you send screenshots of this, as well as any browser errors if any? This may have to do with version compatibility :thinking:.

You can see it live in action here: Hanfgarten-Contest - Forum | Cannabisanbauen.net
(I tested with tc preview only, exact same behavior…)

לייק 1

Do you mean category logo, as the one you upload in the images section of the category settings, or do you mean the style you configure in the general settings where you can choose square, icon, or emoji?

Okay, do you have:

  • show_site_logo ticked?
  • logo_small site setting?

The lock icon situation is interesting. I’m taking a look.

EDIT: @jrgong I’ve just merged a PR. Thing is, you’re on stable and I don’t think you’ll get the update :thinking:… is the recent widget deprecations also on stable?

I think updating should work. There is no entry in the compatibility file that pins the current stable to an older version.

I sometimes add one when I change something and I don’t want to check if the changes also work on older versions like stable.

לייק 1

Below are my settings:

I updated the TC but the issue persists

האם לקטגוריות כלשהן שלך אין סמל קטגוריה? אם לכולן יש, האם סימון show_site_logo עוזר?

that is the case

Unfortunately not

לייק 1

אם תנסה להשתמש בתמונת PNG במקום SVG, האם זה עובד?

In light mode yes. but in dark mode the category header still displays the light logo. see here in action.

my category image settings:

אני לא בטוח שהתאימות עם לוגואים של קטגוריות במצב כהה כבר מובנית - זו תהיה בקשת תכונה.

האם אוכל להציע לשקול תרומה כספית ל-@NateDhaliwal (או לפחות להציע אותה) כדי שיטפל בזה?

לייק 1

Yes, as Nathan said, the component currently does not use the dark mode image. However, I do see an interesting component:
https://github.com/discourse/discourse/blob/main/app/assets/javascripts/discourse/app/components/light-dark-img.gjs

I’ll have to test a few things to see if it can work.

2 לייקים

I’ve noticed a regression - the category description text isn’t refreshing when navigating between categories.

The icons and title refresh, but not the text for some reason. It is fixed with a page refresh:

לייק 1

gulp I will look into this, thanks for the report.

לייק 1

Looking into it a bit more, my other site with the TC is fine in this regard. Don’t waste your time looking at it until I’ve dug a bit deeper!

לייק 1