TP-Docs
HTML5 Icon HTML5 Icon HTML5 Icon
TP on Social Media

Recent

Welcome to TinyPortal. Please login or sign up.

December 13, 2024, 12:31:07 AM

Login with username, password and session length
Members
Stats
  • Total Posts: 195,443
  • Total Topics: 21,252
  • Online today: 120
  • Online ever: 6,457 (November 30, 2024, 02:40:09 PM)
Users Online
  • Users: 1
  • Guests: 103
  • Total: 104
  • lurkalot

The NEW Button

Started by Skb, March 25, 2019, 10:59:01 PM

Previous topic - Next topic

0 Members and 2 Guests are viewing this topic.

Skb

I'm using SMF 2.1 RC1 / TP 1.6.2.
I have a Right Panel with the Recent Posts Block.
In this Block clicking the NEW button does not have the anticipated response.
It goes to the first post of the last page of the topic.
Whereas, clicking the NEW button in the SMF Board view takes you to your first unread post.

Anyone else experienced that ?

SMF 2.1.4 / TP 3.0.1

lurkalot

Thanks for the report.  We'll have to check that out and see what's going on.

I think 2.1 has had some issues where this stuff is pointing to the wrong place, including the alerts.  Not sure if they have sorted all this yet.

@rjen

#2
It seems that you have a point...

The new link from the unread view uses another message id then the link from the recent block...

Recent block uses the message ID of the new message, SMF uses that of the last message read... not sure what is causing this and why it did not pop-up before..

Unread link SMF:
https://test2.fjr-club.nl/index.php?topic=23259.msg327172;topicseen#new

Unread link Recent block:
https://test2.fjr-club.nl/index.php?topic=23259.msg327173;topicseen#new

Will have to check it out later...

Just did some checks on 2.0 and even though the same links are used there in 2.0.15 both links bring you to the first new post. In 2.1 the recent block drops you on the first message on the page that holds the new post...
Running Latest TP on SMF2.1 at: www.fjr-club.nl

Skb

On this site if I click the NEW button in the Recent Posts block to the left, it takes me to the first unread post, which is how it should be.

SMF 2.1.4 / TP 3.0.1

@rjen

True, it is only not working correctly on 2.1.

This site is on 2.0.15
Running Latest TP on SMF2.1 at: www.fjr-club.nl

@rjen

Caused by changes in SMF's SSI.php

function ssi_recentTopics

The field id_msg is populated differently in 2.1 then in 2.0...
No idea if this is something that is a bug or intentional...
Running Latest TP on SMF2.1 at: www.fjr-club.nl

tino

If it's in SSI.php please raise it with SMF. It's not a issue with TinyPortal.

@rjen

It must be: we are using the href link generated by the SSI function.

It seems to be different with 2.1 from 2.0.
Can't put my finger on the exact cause though ..,
Running Latest TP on SMF2.1 at: www.fjr-club.nl

@rjen

Running Latest TP on SMF2.1 at: www.fjr-club.nl

lurkalot

I assume this is related to this problem? https://github.com/SimpleMachines/SMF2.1/pull/5478/files

Though it's scheduled for RC3

@rjen

Good catch, did not see that one...

Just checked the fix, but even though it is very similar, the fix proposed there does NOT fix this issue from SSI...
Running Latest TP on SMF2.1 at: www.fjr-club.nl

lurkalot

Quote from: @rjen on March 29, 2019, 07:34:04 AM
Good catch, did not see that one...

Just checked the fix, but even though it is very similar, the fix proposed there does NOT fix this issue from SSI...

I did wonder, but I know 2.1 has the same problem with going to the wrong place, usually the top of page on a page containing a new post, instead of the actual new post.  Think this is what that commit fixes, thought it might have been related.

Skb

The issue seems to be sorted in RC2.

SMF 2.1.4 / TP 3.0.1

@rjen

Quote from: Skb on March 31, 2019, 06:37:09 AM
The issue seems to be sorted in RC2.

Not on my RC2 test site... :-\
Running Latest TP on SMF2.1 at: www.fjr-club.nl

Skb

Yes, you are right. It has not been sorted.
I checked a post, and it happened to be the first one on a new page. I was misled into thinking that it was done.

SMF 2.1.4 / TP 3.0.1

VladTepes

Well there you go... a reason to look forward to RC3  :2funny: