We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
When a logged in user has read all the posts of a discussion and opens the reply composer, the PostStreamScrubber unnecessarily shows "0 UNREAD".
PostStreamScrubber
"0 UNREAD" should be hidden, like it was in previous flarum versions.
php flarum info
No response
The text was updated successfully, but these errors were encountered:
This is the commit that is causing this issue: 11aa7bb
opacity: unreadPercent ? 1 : 0,
When there aren't any new unread posts, for some reason unreadPercent is evaluated true only when the composer opens..
unreadPercent
true
Sorry, something went wrong.
It seems to evaluate to a negative value for me locally.
That is caused by
const unreadPercent = count ? Math.min(count - this.stream.index, unreadCount) / count : 0;
Where the index is increased by 1 for the draft post, being more than the count itself.
No branches or pull requests
Current Behavior
When a logged in user has read all the posts of a discussion and opens the reply composer, the
PostStreamScrubber
unnecessarily shows "0 UNREAD".Steps to Reproduce
Expected Behavior
"0 UNREAD" should be hidden, like it was in previous flarum versions.
Screenshots
Environment
Output of
php flarum info
No response
Possible Solution
No response
Additional Context
No response
The text was updated successfully, but these errors were encountered: