Show "updated topics above this line" in the topic list?

[Mittineague]

@cpradio

your PR should be good to go

The only suggestion I can make is due to my anticipation of “bike shedding” over the copy.

i.e. Is “new” above or below?

Maybe bracket the I18n with unicode up arrows (\2191) ?

[cpradio]

@Mittineague

The only suggestion I can make is due to my anticipation of “bike shedding” over the copy.

i.e. Is “new” above or below?

Nah, that is simple enough for people to change via the Admin > Customize > Text area. So no need to bikeshed that before it is implemented. If they see a lot of people leaning towards a certain phrase, updating the default won’t be difficult.

[Falco]

@awole20

Here’s a quick css test in chrome using the following:

.last-visit + tr::after {
content: “new”;
position: absolute;
left: 50%;
margin-top: -10px;
padding: 0 10px;
}

@Mittineague

Looks like a winner to me !

@cpradio

That should actually do nicely.

Looks like it’s a winner!

[codinghorror (Jeff Atwood)]

Nice work! Looks like localization is the only stumbling block now, but this should be good enough to get started!

[cpradio]

@codinghorror

Nice work! Looks like localization is the only stumbling block now, but this should be good enough to get started!

@Falco’s PR handles localization too using a data-attribute

[Falco]

@codinghorror

Nice work! Looks like localization is the only stumbling block now, but this should be good enough to get started!

Yeah! Actually it already is localizable since it reads from a data attributte from the tr element.

If @sam can just point me a general direction to get this string set we’re done.

PS: I think it’s around here:

https://github.com/discourse/discourse/blob/master/app/assets/javascripts/discourse/components/topic-list-item.js.es6#L51

[cpradio]

Yep, And I think you can get away with changing

  attributeBindings: ['data-topic-id'],
  'data-topic-id': Em.computed.alias('topic.id'),

to

  attributeBindings: ['data-topic-id', 'data-last-visit-text'],
  'data-topic-id': Em.computed.alias('topic.id'),
  'data-last-visit-text': I18n.t('your_variable_here'),

[Falco]

Nice! I’m gonna test this later and put into the PR.

[Zoltan_Szathmary]

Ok, this is nice and all but is there a way to remove it? We really don’t need that red line and might be confusing to our users as they don’t know what it’s for.

[codinghorror (Jeff Atwood)]

You can set it to whatever you want via CSS of course.

[featheredtoast (Jeff Wong)]

Any update on this? I’ve been following on GH and it appears as though it’s diverged a little bit, in which @sam is reviewing the PR before it’s been updated with the above suggestions…

[Kevin_McKinney]

@Zoltan_Szathmary

Ok, this is nice and all but is there a way to remove it? We really don’t need that red line and might be confusing to our users as they don’t know what it’s for.

I’m not very proficient with CSS but this seemed to remove it on our site (in Settings > Customize > CSS/HTML)

.topic-list>tbody>tr.last-visit {
    border-bottom: inherit;
}

If anyone who knows more about CSS and customization (probably everyone looking at this post) sees anything wrong with that, feel free to correct me!

[Falco]

Text is merged in master! Thanks @tgxworld for the tr approach, works wonders!

[elijah (Eli the Bearded)]

Somehow I have two of them now.

[codinghorror (Jeff Atwood)]

Related to recent work @falco did – it can also show up in Suggested Topics accidentally now.

[sam (Sam Saffron)]

should be fixed per:

https://github.com/discourse/discourse/commit/601a96333ae5c40dcb79edf704460813d633889a

[sam (Sam Saffron)]

Been completed for quite a while!