(Low) Notifications link behavior

User avatar
pokken
Posts: 6388
Joined: 4 years ago
Answers: 2
Pronoun: he / him

Post by pokken » 4 years ago

When you over over Notifications, they expand correctly (though on mobile it often disappears really fast and then won't come back until you refresh - I think this issue is connected).

However, if you click on the word Notifications, the notifications will cease expanding until you reload the page (or go to another page).

Steps to replicate
Load any forum url: http://www.mtgnexus.com/viewforum.php?f=35
Hover over Notifications until they expand
Single click the word Notifications

Note that that closes the notifications rollover and then it will not come back.

As noted above I suspect this may be connected to the similar issue on mobile where you often can only hover over notifications once and then it disappears (presumably because mobile doesn't interpret hovers so you have to click to get it to roll out).

User avatar
Feyd_Ruin
Elder Vampire
Posts: 5410
Joined: 5 years ago
Answers: 3
Pronoun: he / him
Contact:

Post by Feyd_Ruin » 4 years ago

pokken wrote:
4 years ago
When you over over Notifications, they expand correctly (though on mobile it often disappears really fast and then won't come back until you refresh - I think this issue is connected).
Adjusted some things.
Can you confirm this is fixed (I wasn't able to replicate)?
To the beaten, the broken, or the damned; the lost, and the wayward: wherever I may be, you will have a home.

User avatar
Rumpy5897
Tuner of Jank
Posts: 1859
Joined: 4 years ago
Pronoun: he / him

Post by Rumpy5897 » 4 years ago

Tested, was able to replicate pokken's setup, still doesn't work. The word Notifications highlights after clicking, in case that helps with any internal tracking.
 
EDH Primers (click me!)
Deck is Kill Club
Show
Hide

User avatar
pokken
Posts: 6388
Joined: 4 years ago
Answers: 2
Pronoun: he / him

Post by pokken » 4 years ago

I am using Version 75.0.3770.142 (Official Build) (64-bit)

There is a javascript error on pageload that doesn't seem likely to be related but it could be

Code: Select all

vrgrb.js?assets_version=587:76 Uncaught TypeError: $(...).easyAutocomplete is not a function
    at Object.init (vrgrb.js?assets_version=587:76)
    at HTMLDocument.<anonymous> (vrgrb.js?assets_version=587:345)
    at i (jquery.min.js:2)
    at Object.fireWith [as resolveWith] (jquery.min.js:2)
    at Function.ready (jquery.min.js:2)
    at HTMLDocument.K (jquery.min.js:2)
init	@	vrgrb.js?assets_version=587:76
(anonymous)	@	vrgrb.js?assets_version=587:345
i	@	jquery.min.js:2
fireWith	@	jquery.min.js:2
ready	@	jquery.min.js:2
K	@	jquery.min.js:2
There is no javascript error and no network activity when I click on Notifications. So presumably it's just some flawed logic in the code for what happens when you click on notifications.

I'll debug the JS if I can and get back to you

User avatar
Feyd_Ruin
Elder Vampire
Posts: 5410
Joined: 5 years ago
Answers: 3
Pronoun: he / him
Contact:

Post by Feyd_Ruin » 4 years ago

I think that might have been the root cause.
(And I wouldn't have caught it without that note, thank you pokken)

The changes to the jquery extensions weren't visible from older cache versions.
(Which is also why I couldn't replicate, since I had the newer version)
I've hopefully forced a refresh for everyone on this.

Does it work now?
To the beaten, the broken, or the damned; the lost, and the wayward: wherever I may be, you will have a home.

User avatar
pokken
Posts: 6388
Joined: 4 years ago
Answers: 2
Pronoun: he / him

Post by pokken » 4 years ago

It behaves as I would expect now, I click it and it expands. It does not expand when hovered over anymore though which I think is fine :P

User avatar
Feyd_Ruin
Elder Vampire
Posts: 5410
Joined: 5 years ago
Answers: 3
Pronoun: he / him
Contact:

Post by Feyd_Ruin » 4 years ago

pokken wrote:
4 years ago
It behaves as I would expect now, I click it and it expands. It does not expand when hovered over anymore though which I think is fine :P
Yay \o/
Thank you, and apologies to all.
I've added a trigger that should hopefully force future changes through.
To the beaten, the broken, or the damned; the lost, and the wayward: wherever I may be, you will have a home.

Post Reply Previous topicNext topic

Return to “Community Software Feedback and Bug Reports”