In this commit https://github.com/discourse/discourse/commit/e5233f5ce69acc5ecc5edf29fe692d29e73703f2, I used incorrect logic to add the “two-rows” class to the container of the
header-topic-info widget. I was just checking for a category, but many other variables need to be taken into account, which previously happened in the
This change re-arranges some logic, so that the title/contents of the widget are set before
buildClasses returns, so they can be used to calculate the needed className. Then in
html(), the variables are accessed using
The change felt big locally, but it is funny how tiny the diff is.