Patreon Logo Support us on Patreon to keep GamingOnLinux alive. This ensures all of our main content remains free for everyone. Just good, fresh content! Alternatively, you can donate through PayPal Logo PayPal. You can also buy games using our partner links for GOG and Humble Store.
Title: [Fixed] Notification links
Ehvis 20 Apr 2018
User Avatar
  • Supporter Plus
Something appears to have changed with the links from the notifications. It now put me at the last new post instead of the first unread post like it used to do. Since this new behaviour is not really practical, I assume it's a bug.
Liam Dawe 20 Apr 2018
Massive apologies about this, seems it was linked to another issue someone reported to me.

When updating the code a week ago, I was trying to be smart (doh!) and ended bundling together the code that updates notifications, regardless of if you've read them or not it would update the existing row.

I've since split this back as it should, to simply add to the total if you haven't seen it and overwrite it if you have seen it with the new details.
Ehvis 20 Apr 2018
User Avatar
  • Supporter Plus
Thanks! It appears to be back to normal.
Eike 20 Apr 2018
  • Supporter Plus
Thanks for fixing it this fast!
You're really dedicated to your website!
Liam Dawe 20 Apr 2018
Good news, one less thing to deal with ;)
Salvatos 24 Apr 2018
Not sure if related to the same bug, but for at least 2 days now my notification links keep sending me to the very bottom of the page instead of scrolling to the comment in question.
Example link: https://www.gamingonlinux.com/articles/valve-has-removed-the-steam-machine-section-from-steam.11505/page=6#r120557

Looking at the page source there's only one element with that id and it seems to be where it should, so I'm not sure what is going on. Using Firefox 59.0.2, no active addons that should be getting in the way.
Liam Dawe 25 Apr 2018
That's unrelated and will be browser dependent. It's simply using HTML anchor links, so if it's not working there's literally nothing I can do to help really.
tuubi 25 Apr 2018
User Avatar
I mostly fails for me too on firefox, but clicking the address bar and hitting enter jumps to the right comment.

Liam: You could set the scroll position with javascript after everything is loaded. That should work but probably overkill.

EDIT: Also maybe highlight the linked comment?
Ehvis 25 Apr 2018
User Avatar
  • Supporter Plus
Also using FF59, but I have no problems with it. Both on Linux at home and Windows at work.

[edit]

Ok, I need to change that. This one fails consistently:

https://www.gamingonlinux.com/articles/rise-of-the-tomb-raider-is-now-officially-available-on-linux-heres-a-look-at-it-with-benchmarks.11576/page=20#r120609

If can copy/paste this link into a new tab. It will load the page, then a short flash where it will actually be at the correct post after which it will move to the bottom of the page.
Liam Dawe 25 Apr 2018
FYI, to link directly to comments properly, you need to copy the link. As it's different for everyone based on how many they set to display per-page.

So: https://www.gamingonlinux.com/articles/rise-of-the-tomb-raider-is-now-officially-available-on-linux-heres-a-look-at-it-with-benchmarks.11576/comment_id=120609

I can now confirm the issue, but only in Firefox, works 100% of the time in Chrome. Will investigate.
Salvatos 25 Apr 2018
Could it be related to the "Click to view long quote from..." feature? I assume the page loads all content then it gets shrunk with JS, and that throws Firefox off somehow?

Though Ehvis's link above works as expected for me every time and there's long quotes in it too.
Liam Dawe 26 Apr 2018
Quoting: SalvatosCould it be related to the "Click to view long quote from..." feature? I assume the page loads all content then it gets shrunk with JS, and that throws Firefox off somehow?

Though Ehvis's link above works as expected for me every time and there's long quotes in it too.
Ohhhh, didn't think of that. You may very well be right about that! Interesting, will have to think on how to best solve that one.
tuubi 26 Apr 2018
User Avatar
Quoting: liamdawe
Quoting: SalvatosCould it be related to the "Click to view long quote from..." feature? I assume the page loads all content then it gets shrunk with JS, and that throws Firefox off somehow?

Though Ehvis's link above works as expected for me every time and there's long quotes in it too.
Ohhhh, didn't think of that. You may very well be right about that! Interesting, will have to think on how to best solve that one.
I think my suggestion to set the scroll position after everything else is done should work. That's not too hard with jQuery or the experimental [Element.scrollIntoView()](https://caniuse.com/#feat=scrollintoview) method.
Liam Dawe 26 Apr 2018
Okay, this should now be solved, works with 100% replication in both Chrome and Firefox for me now.

I'm now counting the quotes, once the JS loop has finished, it will then use the scrollIntoView() method :)
Ehvis 26 Apr 2018
User Avatar
  • Supporter Plus
Appears to work beautifully now!
Salvatos 26 Apr 2018
Seems fixed for me too! Good job and thanks!
Ehvis 18 May 2018
User Avatar
  • Supporter Plus
It appears something got broken again. Clicking the comment links in the side bar now scrolls two far (title and approx. two lines are clipped).

It's fine when you click links from the "see more comments" page though. Apparently the ?page=...#id version broke, but the /comment_id=... version is still ok.
Liam Dawe 18 May 2018
Actually, nothing broke, I just never updated the sidebar :P

Done, fixed.
Ehvis 18 May 2018
User Avatar
  • Supporter Plus
Thanks!

Something must have changed though. It worked when you fixed that last problem. Anyway, good to have the permanent solution in there!
While you're here, please consider supporting GamingOnLinux on:

Reward Tiers: Patreon Logo Patreon. Plain Donations: PayPal Logo PayPal.

This ensures all of our main content remains totally free for everyone! Patreon supporters can also remove all adverts and sponsors! Supporting us helps bring good, fresh content. Without your continued support, we simply could not continue!

You can find even more ways to support us on this dedicated page any time. If you already are, thank you!
Login / Register